Skip to content

Add API base path consistency test#583

Merged
s-heppner merged 1 commit into
eclipse-basyx:developfrom
rwth-iat:improve/add-base-path-consistency-test
Jul 6, 2026
Merged

Add API base path consistency test#583
s-heppner merged 1 commit into
eclipse-basyx:developfrom
rwth-iat:improve/add-base-path-consistency-test

Conversation

@JAB1305

@JAB1305 JAB1305 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Previously the API base path (currently api/v3.1) was hardcoded across interface modules, dockerfiles and tests. Bumping the version risks drift across the occurences.

This change adds a unittest that extracts the version from each of the currently known files via regex and asserts the version matches.

The risk of newly added occurrences not being tracked remains.

Fixes #535

Previously the API base path (currently `api/v3.1`) was hardcoded across interface modules, dockerfiles and tests. Bumping the version risks drift across the occurences.

This change adds a unittest that extracts the version from each of the currently known files via regex and asserts the version matches.

The risk of newly added occurrences not being tracked remains.

Fixes eclipse-basyx#535
@JAB1305 JAB1305 force-pushed the improve/add-base-path-consistency-test branch from a6be013 to 9d11451 Compare July 2, 2026 11:06

@s-heppner s-heppner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you!

@s-heppner s-heppner merged commit 77e709a into eclipse-basyx:develop Jul 6, 2026
15 checks passed
@s-heppner s-heppner deleted the improve/add-base-path-consistency-test branch July 6, 2026 08:59
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