Feature request: warn vs error mode about axe-storybook-testing HOT 7 CLOSED

Yeah that definitely would be helpful! It would also be great to have a top level option similar to skip or instead of skip that could use all existing rules.

Something like

axe: {
  runStatus: 'off'/'warn'/'error'
}

Not tied to that wording at all, had a hard time thinking of a good name there 😂

from axe-storybook-testing.

@ahuth you're amazing! It might be a little while before we're able to fully test this out but I'll let you know how it goes!

from axe-storybook-testing.

Good point. Are you thinking instead of

disabledRules: ['foo', 'bar', 'baz'],

it could be

ruleOverrides: {
  foo: 'off',
  bar: 'warn',
  baz: 'warn',
},

or something like that?

If so, I like it 👍 . We can definitely add that or something similar.

from axe-storybook-testing.

Ah, gotcha. I misunderstood before, but now it's clear. We're talking about marking an entire story as non-failing, but still report its results.

Makes sense to me, and I'll add that.

from axe-storybook-testing.

Thank you @ahuth ! 😄

from axe-storybook-testing.

@backwardok I implemented this in #70 and merged it. Going to do some more testing, and probably publish a release with it tomorrow.

from axe-storybook-testing.

Released in v6.3.0 🚀

How the results are displayed leaves a little bit to be desired, but it's the simplest way I could do it without much deeper refactoring. Lemme know how it works for you!

from axe-storybook-testing.