Solving : Fix required not defined

[linediconsine]

Hi, This patch aim to solve https://github.com/component-driven/cypress-axe/issues/170

the actual code rely on required be defined in the browser but this is not the case in my tests on Chrome browser:

Actual code

(typeof require?.resolve === 'function'
            ? require.resolve('axe-core/axe.min.js')
            : 'node_modules/axe-core/axe.min.js');

console log

VM87:1 Uncaught ReferenceError: require is not defined
    at <anonymous>:1:1
(anonymous) @ VM87:1

Screenshot on Chrome

Patch proposed :

(typeof require != 'undefined' && typeof require?.resolve === 'function'
            ? require.resolve('axe-core/axe.min.js')
            : 'node_modules/axe-core/axe.min.js');

console log

'node_modules/axe-core/axe.min.js'

tagging last active developers @MehmetYararVX @johnhwhite for a review on this if possible

[linediconsine]

Let me know any feedback on this

thanks

[linediconsine]

On local nom run test give me

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  test.cy.js                               699ms        1        1        -        -        - │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        699ms        1        1        -        -        -  

I see a timeout error on nom run test logs in pipeline testing, is it there a way to re-test?

[courentin]

@linediconsine thanks for your fix! Maybe you can push an empty commit to trigger the CI or create another PR?

@bmordue any chance to merge this one-liner and trigger a new release? This would help us :)

Thanks a lot!

[adamnorbacker]

Any news on this PR? Also have this issue in our project.