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

Add jasmine-axe for Batch Explorer unit tests #2587

Merged
merged 9 commits into from
Apr 17, 2023
Merged

Add jasmine-axe for Batch Explorer unit tests #2587

merged 9 commits into from
Apr 17, 2023

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 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.

package.json Outdated Show resolved Hide resolved
@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 <cRui861@users.noreply.github.com>
Co-authored-by: David Watrous <509299+dpwatrous@users.noreply.github.com>
Co-authored-by: Waseem Dahman <mhdahman@microsoft.com>
Co-authored-by: Waseem <79171711+mhdahman@users.noreply.github.com>
Co-authored-by: Haopeng Wang <wanghp000@gmail.com>
Co-authored-by: “aibiber” <“aibiber@microsoft.com”>
Co-authored-by: Robert Taft <robtaft@gmail.com>
Co-authored-by: robtaft-ms <robtaft@microsoft.com>
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 <cRui861@users.noreply.github.com>
Co-authored-by: David Watrous <509299+dpwatrous@users.noreply.github.com>
Co-authored-by: Waseem Dahman <mhdahman@microsoft.com>
Co-authored-by: Waseem <79171711+mhdahman@users.noreply.github.com>
Co-authored-by: Haopeng Wang <wanghp000@gmail.com>
Co-authored-by: “aibiber” <“aibiber@microsoft.com”>
Co-authored-by: Robert Taft <robtaft@gmail.com>
Co-authored-by: robtaft-ms <robtaft@microsoft.com>

* Bumps Batch Explorer to version 2.18.0

---------

Co-authored-by: Rena <cRui861@users.noreply.github.com>
Co-authored-by: David Watrous <509299+dpwatrous@users.noreply.github.com>
Co-authored-by: Waseem Dahman <mhdahman@microsoft.com>
Co-authored-by: Waseem <79171711+mhdahman@users.noreply.github.com>
Co-authored-by: Haopeng Wang <wanghp000@gmail.com>
Co-authored-by: “aibiber” <“aibiber@microsoft.com”>
Co-authored-by: Robert Taft <robtaft@gmail.com>
Co-authored-by: robtaft-ms <robtaft@microsoft.com>
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.

None yet

2 participants