Improvement/new tests #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
toJSON method on indices.
team. Starting the test structure for testing the datastore and all of its methods. Start of setup for the mock database and mock storage driver. Added a fixture db to be used as an initial load to the db file which will be cleared after every test, and a usersIndex file for storing the index of the database
them. Can be done with proper tsconfig setup.
keys/iterate/storeIndex/fetchIndex. Need to write a method for fetching all indexed fields JSON to store. Added transpilation of binary-type-tree during tests since BTT is an es6 module. Added play datastore for testing. updated jest and added Node dev dependency for node storage driver tests.
…index are now implemented on the mock storage driver. Switch out fs library for a mock fs library from jest later. clear now clears all files associated with filepath including all indexed json files. new getPath method that can return 0/null/undefined.
works. Find methods on datastore do not work. cursor does not work Update does not work. These are the final pieces.
Need a new branch to create an Update and add tests for it.
Need a new branch to create an Update and add tests for it. Need to make sure version 0.0.1 can be found compared to 0.0.2
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change Log: