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

Create test matrix to speedup tests #2458

Merged
merged 1 commit into from
Sep 19, 2022

Conversation

cgarciae
Copy link
Collaborator

@cgarciae cgarciae commented Sep 13, 2022

What does this PR do?

This PR add some options to run_all_tests.sh such that doctest, pytest, and pytype tests can be ran in parallel on CI.

Changes

  • Adds the --no-pytest and --no-doctest flags to run_all_tests.sh
  • Renames run-tests to tests in build.yml and removed test-docs.
  • Creates a test-type matrix for tests with the following options: doctest, pytest, pytype. Each test then uses test-type to set the correct flags for run_all_tests.sh.

@codecov-commenter
Copy link

codecov-commenter commented Sep 13, 2022

Codecov Report

Merging #2458 (88517f6) into main (e320e11) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2458   +/-   ##
=======================================
  Coverage   79.66%   79.66%           
=======================================
  Files          49       49           
  Lines        4982     4982           
=======================================
  Hits         3969     3969           
  Misses       1013     1013           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@copybara-service copybara-service bot merged commit fb8b640 into google:main Sep 19, 2022
@cgarciae cgarciae mentioned this pull request Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants