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.

Sign up for GitHub

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

Jump to bottom

[JS] Run tests with @baze/jasmine #11486

Merged

harsha509 merged 3 commits into SeleniumHQ:trunk from harsha509:trunk

Dec 27, 2022

Member

harsha509 commented Dec 26, 2022 •

edited

Description

JS has multiple test runners configured (@bazel/jasmine, jasmine, mocha). as tests runs with @bazel/jasmine in CI, this PR removes initial jasmine configuration

Motivation and Context

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

I have read the contributing document.
My change requires a change to the documentation.
I have updated the documentation accordingly.
I have added tests to cover my changes.
All new and existing tests passed.

harsha509 and others added 2 commits

December 26, 2022 17:37


          [JS] Test runner cleanup. Run tests with @bazel/jasmine

70e6874


          Merge branch 'SeleniumHQ:trunk' into trunk

harsha509 marked this pull request as ready for review

December 26, 2022 12:12


          Merge branch 'trunk' into trunk

cc14aca

sonarcloud bot commented Dec 27, 2022

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

Member Author

harsha509 commented Dec 27, 2022

JS Checks Looks good to me!

harsha509 merged commit 4071737 into SeleniumHQ:trunk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment