Skip to content

Conversation

@r-el
Copy link
Owner

@r-el r-el commented Jul 6, 2025

  • Add readAll method
  • Add read(id, callback) method
  • Add test for read and readAll methods
  • Fix constructor test to validate file path correctly
  • Update test scripts to run both basic and read tests
  • Add test-read.js test file
  • Add basic tests for JsonFileCRUD constructor and methods [RENAME test.js to test-basic.js]
  • Update test scripts to reference the correct test files
  • Refactor JsonFileCRUD class structure by organizing methods into regions and removing the unused read method
  • Replace 'read' method to 'findById' method in required methods test
  • Add Test for findById method
  • Add findBy method for filtering items by custom callback
  • Add test for 'findBy' method
  • Add count(callback) method
  • Add test for count method

@r-el r-el merged commit 5f47ba3 into master Jul 6, 2025
@r-el r-el deleted the feature/read-functionality branch July 6, 2025 13:48
r-el pushed a commit that referenced this pull request Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants