Skip to content

Conversation

@lkdvos
Copy link
Member

@lkdvos lkdvos commented Oct 8, 2025

This PR separates out some of the parts of the test suite in order to increase parallelism on the github runners, and get slightly better feedback on where things went wrong if they did.

The main idea is to organize the tests into groups, as dictated by a folder structure, and then the QuantumKitHubActions workflow takes care of the rest. Additionally, you can control them from the local command line through:

import Pkg
Pkg.test("TensorKit"; test_args=["--groups", "other", "symmetries"]

which would run only the other/ and symmetries/ tests.

I've made sure that the individual files are all self-contained as well, so a workflow like this also works:

using TestEnv; TestEnv.activate()
include("test/symmetries/fusiontrees.jl")

Obviously the split I chose is arbitrary, and I'm happy to rethink all of the names.


This also tackles some issues we had with the CompatCheck workflow
This also fixes a breaking change we accidentally slipped in with the TensorKitSectors v0.3.1 changes.

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos lkdvos force-pushed the ld-tests branch 2 times, most recently from e75541b to 6d590f0 Compare October 8, 2025 21:51
@lkdvos
Copy link
Member Author

lkdvos commented Oct 8, 2025

Had to patch the parsing due to QuantumKitHub/TensorKitSectors.jl#32

@lkdvos lkdvos force-pushed the ld-tests branch 3 times, most recently from cbc3942 to 745b665 Compare October 17, 2025 16:22
@Jutho
Copy link
Member

Jutho commented Oct 24, 2025

Thanks for this; this looks great. I left a few small comments.

@lkdvos lkdvos enabled auto-merge (squash) October 24, 2025 12:33
@lkdvos
Copy link
Member Author

lkdvos commented Oct 24, 2025

I think all comments have been addressed here, all this needs is an approval and it should merge as soon as the tests pass.

@lkdvos lkdvos disabled auto-merge October 24, 2025 16:02
@lkdvos lkdvos merged commit 9ba3b6c into main Oct 24, 2025
42 checks passed
@lkdvos lkdvos deleted the ld-tests branch October 24, 2025 16:02
lkdvos referenced this pull request Oct 28, 2025
* Bump v0.15.2

* fix small errors in tutorial
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.

2 participants