Skip to content

Add jasmine-axe for Batch Explorer unit tests#2587

Merged
cRui861 merged 9 commits intomasterfrom
rechen/axe
Apr 17, 2023
Merged

Add jasmine-axe for Batch Explorer unit tests#2587
cRui861 merged 9 commits intomasterfrom
rechen/axe

Conversation

@cRui861
Copy link
Member

@cRui861 cRui861 commented Nov 4, 2022

Start adding accessibility tests for a few UI components.

@cRui861 cRui861 changed the title Add jasmine-axe for unit tests Add jasmine-axe for Batch Explorer unit tests Nov 4, 2022
@cRui861 cRui861 requested a review from NickKouds as a code owner January 12, 2023 23:38
@cRui861 cRui861 force-pushed the rechen/axe branch 2 times, most recently from 7e01ee9 to e9b0f18 Compare January 18, 2023 21:46
@codecov
Copy link

codecov bot commented Apr 4, 2023

Codecov Report

Merging #2587 (7f42b4b) into master (f983d97) will not change coverage.
The diff coverage is n/a.

❗ Current head 7f42b4b differs from pull request most recent head ab5a56a. Consider uploading reports for the commit ab5a56a to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2587   +/-   ##
=======================================
  Coverage   65.08%   65.08%           
=======================================
  Files         916      916           
  Lines       26477    26477           
  Branches     5237     5237           
=======================================
  Hits        17233    17233           
  Misses       9244     9244           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f983d97...ab5a56a. Read the comment docs.

@cRui861 cRui861 merged commit 52bfee0 into master Apr 17, 2023
@cRui861 cRui861 deleted the rechen/axe branch April 17, 2023 19:26
gingi added a commit that referenced this pull request Apr 28, 2023
* Bumps Batch Explorer to version 2.17.0

* Restores build version string for release packaging

* Fixes AB#1031: Updates vulnerable http-cache-semantics dependency

Also replaced the vulnerable download module with node-downloader-helper.

* Non-home tenants are inactive by default

Users may opt in to authenticate specific tenants by activating them in the authentication settings page.

* Merges release 2.16.1 into master (#2661)

* Restores build version string for release packaging

* Fixes #2659: Removes test code

* Prepare for hotfix release 2.16.1

* Remove custom themes (#2663)

* Fixes #2662: Restores Python backend

Updated Batch CLI extensions (v9.0.0) require `azure-multiapi-storage` as explicit requirement.

* Update python dependencies (#2682)

* Theme colors will only show in dev mode (#2697)

* Add Developer menu item and add theme colors under it
- running into issue with surfacing isDev

* Only show Developer menu item during developer mode
* typo fix

* Switch  hardcoded color values to use theme colors

* Fix unit tests

* Remove fdescribe

* Cloud Service Deprecation Warning on Pool Details (#2702)

* Add banner in pools for cloud service pool deprecation
* small typo fix

* Fix wording

* Update wording

* focus first element after activating ‘Job preparation task’

* focus first interactive control in Storage accounts pane

* focus first element in complex-form afterViewInit

* update form-field prefix and suffix styles

* update monitorChart colors

* update monitor chart tooltip config

* update mat-tab-group labels

* fix @azure/storage-blob usage (#2709)

* Add jasmine-axe for Batch Explorer unit tests (#2587)

* Initial setup for jasmine axe

* Add axe for accessibility testing
* Add accessibility tests for a few UI components

* Update package lock

* Testing to see if pipeline completely passes

* wip

* Update package-lock

* Remove console log statement

* Remove failing test (fix and re-add them later)

* Remove cross-env

* delete metadata row

* Fixes AB#950: Focus not visible for radio buttons

* Fixes AB#865: Focus not visible for multiple controls

* Fixes AB#990: Focus moves to background when pop-up is open

* Fixes AB#702: Focus ring not visible

...when tab order reaches table for the first time without interacting with the table using the arrow keys

* Fixes AB#671: When closing select dropdown with esc key the whole dialog is closed

* Fixes AB#290 removing pointer events on canvas

* Fixes AB#462 - Changing input boxes to bordered boxes.

* Updated changelog and version. (#2719)

---------

Co-authored-by: Rena <[email protected]>
Co-authored-by: David Watrous <[email protected]>
Co-authored-by: Waseem Dahman <[email protected]>
Co-authored-by: Waseem <[email protected]>
Co-authored-by: Haopeng Wang <[email protected]>
Co-authored-by: “aibiber” <“[email protected]”>
Co-authored-by: Robert Taft <[email protected]>
Co-authored-by: robtaft-ms <[email protected]>
gingi added a commit that referenced this pull request May 1, 2023
* Restores build version string for release packaging

* Fixes #2659: Removes test code

* Prepare for hotfix release 2.16.1

* Release 2.17.0 (#2721)

* Bumps Batch Explorer to version 2.17.0

* Restores build version string for release packaging

* Fixes AB#1031: Updates vulnerable http-cache-semantics dependency

Also replaced the vulnerable download module with node-downloader-helper.

* Non-home tenants are inactive by default

Users may opt in to authenticate specific tenants by activating them in the authentication settings page.

* Merges release 2.16.1 into master (#2661)

* Restores build version string for release packaging

* Fixes #2659: Removes test code

* Prepare for hotfix release 2.16.1

* Remove custom themes (#2663)

* Fixes #2662: Restores Python backend

Updated Batch CLI extensions (v9.0.0) require `azure-multiapi-storage` as explicit requirement.

* Update python dependencies (#2682)

* Theme colors will only show in dev mode (#2697)

* Add Developer menu item and add theme colors under it
- running into issue with surfacing isDev

* Only show Developer menu item during developer mode
* typo fix

* Switch  hardcoded color values to use theme colors

* Fix unit tests

* Remove fdescribe

* Cloud Service Deprecation Warning on Pool Details (#2702)

* Add banner in pools for cloud service pool deprecation
* small typo fix

* Fix wording

* Update wording

* focus first element after activating ‘Job preparation task’

* focus first interactive control in Storage accounts pane

* focus first element in complex-form afterViewInit

* update form-field prefix and suffix styles

* update monitorChart colors

* update monitor chart tooltip config

* update mat-tab-group labels

* fix @azure/storage-blob usage (#2709)

* Add jasmine-axe for Batch Explorer unit tests (#2587)

* Initial setup for jasmine axe

* Add axe for accessibility testing
* Add accessibility tests for a few UI components

* Update package lock

* Testing to see if pipeline completely passes

* wip

* Update package-lock

* Remove console log statement

* Remove failing test (fix and re-add them later)

* Remove cross-env

* delete metadata row

* Fixes AB#950: Focus not visible for radio buttons

* Fixes AB#865: Focus not visible for multiple controls

* Fixes AB#990: Focus moves to background when pop-up is open

* Fixes AB#702: Focus ring not visible

...when tab order reaches table for the first time without interacting with the table using the arrow keys

* Fixes AB#671: When closing select dropdown with esc key the whole dialog is closed

* Fixes AB#290 removing pointer events on canvas

* Fixes AB#462 - Changing input boxes to bordered boxes.

* Updated changelog and version. (#2719)

---------

Co-authored-by: Rena <[email protected]>
Co-authored-by: David Watrous <[email protected]>
Co-authored-by: Waseem Dahman <[email protected]>
Co-authored-by: Waseem <[email protected]>
Co-authored-by: Haopeng Wang <[email protected]>
Co-authored-by: “aibiber” <“[email protected]”>
Co-authored-by: Robert Taft <[email protected]>
Co-authored-by: robtaft-ms <[email protected]>

* Bumps Batch Explorer to version 2.18.0

---------

Co-authored-by: Rena <[email protected]>
Co-authored-by: David Watrous <[email protected]>
Co-authored-by: Waseem Dahman <[email protected]>
Co-authored-by: Waseem <[email protected]>
Co-authored-by: Haopeng Wang <[email protected]>
Co-authored-by: “aibiber” <“[email protected]”>
Co-authored-by: Robert Taft <[email protected]>
Co-authored-by: robtaft-ms <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants