Adds coverage for inject, run, report, and write_results methods and fixes default results file writing on Windows #210
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.
Add thorough tests for Axe class and fix default results file path
I've added a bunch of new tests for the Axe class to make sure all the main methods behave correctly:
inject()
now has tests to check that it executes the script properly and raises an error if the script is missing.run()
has tests for different combinations of context and options, plus error handling when the JS fails.report()
is tested to ensure the output matches expectations for both populated and empty violation lists.write_results()
now has tests for default filename usage, invalid paths, and unserializable data.Also fixed an issue in
write_results()
where Windows was failing becauseos.getcwd()
was not used correctly to get the current working directory. Now it writes the defaultresults.json
file as expected across platforms.