Skip to content

refactor: Migrate external_repo tests to bazel - #298

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

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

@furtib furtib changed the title Migrate external_repo tests to bazel chore: Migrate external_repo tests to bazel Aug 26, 2026
@furtib furtib changed the title chore: Migrate external_repo tests to bazel refactor: Migrate external_repo tests to bazel Aug 26, 2026
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