Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

axe.getFrameContexts mutates its input #3045

Closed
WilcoFiers opened this issue Jun 28, 2021 · 2 comments
Closed

axe.getFrameContexts mutates its input #3045

WilcoFiers opened this issue Jun 28, 2021 · 2 comments
Assignees
Labels
core Issues in the core code (lib/core) fix Bug fixes pr A pr has been created for the issue
Milestone

Comments

@WilcoFiers
Copy link
Contributor

This shouldn't fail:

const context = { exclude: [['iframe', 'foo']] }
axe.utils.getFrameContexts(context);
assert.deepEqual(context, { exclude: [['iframe', 'foo']] });
@WilcoFiers WilcoFiers added fix Bug fixes core Issues in the core code (lib/core) labels Jun 28, 2021
@WilcoFiers WilcoFiers added this to the axe-core 4.3 milestone Jun 28, 2021
@WilcoFiers WilcoFiers reopened this Jun 29, 2021
@straker straker added the pr A pr has been created for the issue label Jul 14, 2021
@padmavemulapati
Copy link

Validated with the latest develop branch code base,
on passing the axe.utils.getFrameContexts({exclude: [['iframe','#testFrame']] });
for a test file
"http://qateam.dequecloud.com/attest/api/test.html" , and id of the iframe being #testFrame
and it excluding the frame contexts, below is the screenshot,
image

@straker
Copy link
Contributor

straker commented Sep 28, 2021

No docs

@straker straker closed this as completed Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Issues in the core code (lib/core) fix Bug fixes pr A pr has been created for the issue
Projects
None yet
Development

No branches or pull requests

4 participants