Skip to content

Conversation

@VsevolodX
Copy link
Member

@VsevolodX VsevolodX commented Oct 30, 2025

remove "results" from executables in swfs and wfs
generate unique ids for swfs and wfs

@VsevolodX VsevolodX changed the base branch from main to feature/SOF-7730 October 30, 2025 21:09
const EXPECTED_SUBWORKFLOW_ID = "233bb8cf-3b4a-5378-84d9-a6a95a2ab43d";
const EXPECTED_UNIT_ID = "9fc7a088-5533-5f70-bb33-f676ec65f565";

describe("workflows", () => {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add test to verify problem is fixed: WFs with the same name and different Apps should give different IDs

Base automatically changed from feature/SOF-7730 to main November 3, 2025 18:09
const EXPECTED_WORKFLOW_ID = "512cb2cc-5cca-52c7-9dd9-1d2b57196188";
const EXPECTED_SUBWORKFLOW_ID = "82673aa1-c4d7-5b2c-b216-7579760fdfaf";
const EXPECTED_WORKFLOW_ID = "464ccdc8-bc26-5fae-813c-ee73fe639101";
const EXPECTED_SUBWORKFLOW_ID = "233bb8cf-3b4a-5378-84d9-a6a95a2ab43d";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add test for the above logic

@timurbazhirov timurbazhirov merged commit b4d1696 into main Nov 4, 2025
4 checks passed
@timurbazhirov timurbazhirov deleted the feature/SOF-7731 branch November 4, 2025 02:45
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.

3 participants