Testing without mocks allows us to look at the code how it is functioning right now so that we can evaluate the intended behavior.

Objects shared by both tests should be moved into the outer block so that they can be shared.

Now the test fails properly and passes properly by introducing callbacks

The failure should be visible to the user and in the current state it is so now by isolating the test it becomes easier to explore solutions that work.

indent a describe block, modify tests to make sure they pass and refactor a function out that easily allows for testing modified parameters.

Add two more tests that address the array case for excludeIndexesFromPath. Also ensure that the conditions which check for these test cases properly ignore exclude from indexes when an array is used

We should not create a nomocks file because we can just group the tests under a unique describe block of test/index.ts.

…xes-with-wildcard

…xes-with-wildcard

…into check-property-existence-for-excludeFromIndexes-with-wildcard

Remove typeof and remove check for string ‘undefined’

…ldcard' of https://github.com/danieljbruce/nodejs-datastore into check-property-existence-for-excludeFromIndexes-with-wildcard

Refactor the tests so that they are more readable. There is a lot of repeated code in each test which makes comparisons hard.

Replace all tests exercising the feature so that they all use the same test script. This will make the tests way easier to read.

Add parameterized testing to eliminate repeated code. These tests need to be easier to read for the sake of the reviewer.

This function is only used once now. We should inline this code in the test.

In the test title include the properties passed in so that it is easy to track what passed/failed.

Similar variables should be used together. Organize the code so that there is less confusion about which variables are related.

The getExpectedConfig does not need to be used anymore because its code can just be inlined in the one place it is used.

Separate the test into blocks and inline some parameters for better test readability.

Stronger typing makes the code easier to read.

Don’t assign to the Datastore variable. This is not necessary. Just use the OriginalDatastore variable directly.

The diff will be easier to read if we do not add a nested if statement and instead apply a condition on the if and change the else to else if.

Apply the first suggestion in the PR to all examples

There is a repeated code fragment for checking if the first path part is undefined that we should refactor.

This reverts commit a101f27.

This reverts commit 4f54d94.

This reverts commit df3f376.

We do four checks to see if the first path part is defined. We should refactor these checks out.

The comment should reflect the next line of code.

…into check-property-existence-for-excludeFromIndexes-with-wildcard

# Conflicts:
#	test/index.ts

Replace comments with description property so that the tests will report the description when running the tests instead of the properties in the description.

…xes-with-wildcard