Skip to content

Conversation

@ogenstad
Copy link
Contributor

@ogenstad ogenstad commented May 13, 2025

This PR corrects an issue where a test was pretty much disabled as we weren't able to merge stable back into develop without it.

The problem came when we were migrating Prefect tasks in #6231 (targeting develop) and at the same time updating this code in #6363 targeting stable. When we merged stable back into develop an old but moved code path was used causing the test not to work (the idea of the test was that the node object store would be populated if the transform was configured to do so.

@github-actions github-actions bot added the group/backend Issue related to the backend (API Server, Git Agent) label May 13, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented May 13, 2025

CodSpeed Performance Report

Merging #6456 will not alter performance

Comparing pog-revert-transform-test-changes-IFC-1502 (2296bd1) with develop (21d3c54)

Summary

✅ 10 untouched benchmarks

@ogenstad ogenstad force-pushed the pog-revert-transform-test-changes-IFC-1502 branch from b326af2 to 2296bd1 Compare May 13, 2025 12:59
@ogenstad ogenstad marked this pull request as ready for review May 13, 2025 16:41
@ogenstad ogenstad requested a review from a team as a code owner May 13, 2025 16:41
@ogenstad ogenstad merged commit b8767cc into develop May 13, 2025
55 of 56 checks passed
@ogenstad ogenstad deleted the pog-revert-transform-test-changes-IFC-1502 branch May 13, 2025 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

group/backend Issue related to the backend (API Server, Git Agent)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants