-
Notifications
You must be signed in to change notification settings - Fork 689
Closed
Description
As I worked on #1794 I noticed there are a few specs which follow pattern of assert(res,
They seem to pass even when i changed the expected values.
I made a change on the one test in that PR(https://github.com/AlaSQL/alasql/pull/1794/files#diff-45ecd1e6cf603c0d570eae55d8803c4b5af8e3aee3d64d1b061d64814191e146R19).
Should the other specs like shown in this branch https://github.com/AlaSQL/alasql/compare/develop...rameshrajagopalanbayer:alasql:asserts-fix?expand=1 also have the equal or deepEqual added to it,
If this looks to be the case let me know i can make that change and create a PR. Currently i do have 1 spec (uncomment) fail after the change to have equal or deepEqual.