Skip to content

Migrate external_repo tests to bazel - #298

Open
furtib wants to merge 2 commits into
Ericsson:mainfrom
furtib:test_into_bazel_003
Open

Migrate external_repo tests to bazel#298
furtib wants to merge 2 commits into
Ericsson:mainfrom
furtib:test_into_bazel_003

Conversation

@furtib

@furtib furtib commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Why:
We want all tests to be run with bazel test //...

What:

  • Migrate external_repository tests to Bazel (This introduces a performance penalty; each external test takes over a minute to run)

Addresses:
#210

@furtib
furtib requested a review from Szelethus August 19, 2026 09:51
@furtib furtib self-assigned this Aug 19, 2026
@furtib furtib added the test ☑️ Adding or refactoring tests label Aug 19, 2026
@Szelethus

Copy link
Copy Markdown
Collaborator

This introduces a performance penalty; each external test takes over a minute to run

I mean that sounds like a big deal.

@furtib

furtib commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Running pytest on the original test takes 55s.
In the Bazel-migrated version, the longest job takes 78s; thanks to parallelization, this is the time for the whole test suite.

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

Labels

test ☑️ Adding or refactoring tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants