Improve unit tests of escape-literal/identifier, and remove them from integration tests. #607

Merged
merged 1 commit into from Jul 6, 2014

Projects

None yet

2 participants

@gurjeet
gurjeet commented Jun 15, 2014

Improve the code and clarity of unit tests in escape-tests.js. And
removed the related integration tests since it has been demonstrated in
the unit tests that a connection is not needed for escaping the literals
and identifiers.

@gurjeet gurjeet Improve unit tests of escape-literal/identifier, and remove them from…
… integration tests.

Improve the code and clarity of unit tests in escape-tests.js. And
removed the related integration tests since it has been demonstrated in
the unit tests that a connection is not needed for escaping the literals
and identifiers.
75760c4
@brianc
Owner
brianc commented Jun 17, 2014

Nice! 👏 Thanks

@brianc brianc merged commit c4f9141 into brianc:master Jul 6, 2014

1 check passed

Details continuous-integration/travis-ci The Travis CI build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment