a11y tests on adding columns to discover table

[bhavyarm]

meta: #51456

[bhavyarm]

@myasonik I have this test passing in local and failing in CI. Giving it another run. Also fixed type errors.

 it('Click on new to clear the search', async () => {
      await PageObjects.discover.clickNewSearchButton();
      await a11y.testAppSnapshot();
    });

[bhavyarm]

@elasticmachine merge upstream

[bhavyarm]

@elasticmachine merge upstream

[bhavyarm]

@myasonik will wait for your input before I skip this test again:
It passes in local - fails in CI

  1) Discover
       Click on new to clear the search:
     Error: a11y report:

VIOLATION
  [aria-valid-attr-value]: Ensures all ARIA attributes have valid values
    Help: https://dequeuniversity.com/rules/axe/3.5/aria-valid-attr-value?application=axeAPI
    Elements:
      - .dscFieldList--selected
      at Accessibility.testAxeReport (test/accessibility/services/a11y/a11y.ts:85:15)
      at Accessibility.testAppSnapshot (test/accessibility/services/a11y/a11y.ts:58:18)
      at process._tickCallback (internal/process/next_tick.js:68:7)

           └- ✖ fail: "Discover Click on new to clear the search"
           │
         └-> "after all" hook
     │
     │4 passing (1.0m)
     │1 failing

[myasonik]

🤔 That's pretty odd that it passes locally... My only thought is that it might be because of different data used in each environment? But I thought it was the same...

Either way, this will all change with the EUIification so let's just wait till then to deal with it. Let's skip it until that's done. Maybe also make an issue and link it to #47559 and assign it to @kertal and I. We'll deal with it then.

[bhavyarm]

@elasticmachine merge upstream

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: fb48c06

History

• 💔 Build #32583 failed 425867d
• 💔 Build #31562 failed 3da4371
• 💔 Build #31468 failed 2c6ef66
• 💔 Build #31156 failed 325e04a
• 💔 Build #31095 failed 92358c8

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

[bhavyarm]

@myasonik ready!!!!

[myasonik]

A+

[bhavyarm]

@kertal this is the PR with the skipped a11y tests. I am going to create an issue to track along with your pr #47559. Thank you!

[kertal]

thanks @bhavyarm, will make it work in #47559