Remove the need for explicit call of injectAxe

mwmcode / cypress-axe-core

Test accessibility with axe-core in Cypress.

MIT License

5 stars 0 forks source link

Remove the need for explicit call of injectAxe #9

Open stereobooster opened 3 years ago

stereobooster commented 3 years ago

What do you think about removing explicit call of injectAxe and instead check if axe already injected in checkA11y, if no then inject it, if yes then skip injecting?

Also maybe read file once in the memory rather then reading it on each inject?

stereobooster commented 3 years ago

Is there a need for this:

Enable results logging by defining cy.tasks

Why it needs to be task can't it just directly use console.log, console.table?

mwmcode commented 3 years ago

It'd be a lot easier if we could just cy.checkA11y but the loading/injection of axe-core has to happen before running the running the check function.
You can provide violationsCallback and control how results are logged. In that case you won't need to create those tasks. Take a look here https://github.com/mcha-dev/cypress-axe-core#cyconfigurecypressaxe