Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

duplicate tests in taint test/unit_tests.js (Object keys) #9

Closed
dmitris opened this Issue Mar 7, 2013 · 1 comment

Comments

Projects
None yet
2 participants
Collaborator

dmitris commented Mar 7, 2013

It seems to me that the tests on
https://github.com/wisec/DOMinator/blob/master/tainttests/unit_tests.js#L27
and
https://github.com/wisec/DOMinator/blob/master/tainttests/unit_tests.js#L42
are identical - is it so?

The comment for the second one says "Yet to implement" but I believe it is implemented and works now...

Unless I'm missing some details, I suggest to remove the second test (and add missing semicolons :) ).

Owner

wisec commented Mar 8, 2013

It seems so,
thanks ! I'll remove it.

@wisec wisec closed this Jan 9, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment