Skip to content

[9.0] Add DiracX Unit Tests CI and fix diracx installs#8200

Merged
fstagni merged 2 commits intoDIRACGrid:integrationfrom
chrisburr:diracx-unit-tests
May 20, 2025
Merged

[9.0] Add DiracX Unit Tests CI and fix diracx installs#8200
fstagni merged 2 commits intoDIRACGrid:integrationfrom
chrisburr:diracx-unit-tests

Conversation

@chrisburr
Copy link
Copy Markdown
Member

@chrisburr chrisburr commented May 20, 2025

JobStatusUtilities is intended to isolate code which is common to DIRAC and DiracX to avoid importing parts of DIRAC which are supposed to not work with DiracX.

In #8192 this isolation was broken which broke DiracX so this PR adds DiracX's unit tests to DIRAC as well as fixing the underlying issue.

BEGINRELEASENOTES

*WorkloadManagement
FIX: Remove accidental dependnecies in JobStateUtility

ENDRELEASENOTES

@chrisburr chrisburr force-pushed the diracx-unit-tests branch 3 times, most recently from 0e9474d to 3295a72 Compare May 20, 2025 08:55
@chrisburr chrisburr force-pushed the diracx-unit-tests branch from 3295a72 to 015f82f Compare May 20, 2025 08:57
@chrisburr chrisburr marked this pull request as ready for review May 20, 2025 09:05
@chrisburr chrisburr requested review from atsareg and fstagni as code owners May 20, 2025 09:05
@fstagni fstagni merged commit 33d5439 into DIRACGrid:integration May 20, 2025
24 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants