Skip to content

Updates int tests to BaSyx Go AASX File Server#541

Merged
aaronzi merged 4 commits into
eclipse-basyx:mainfrom
aaronzi:main
May 20, 2026
Merged

Updates int tests to BaSyx Go AASX File Server#541
aaronzi merged 4 commits into
eclipse-basyx:mainfrom
aaronzi:main

Conversation

@aaronzi

@aaronzi aaronzi commented May 20, 2026

Copy link
Copy Markdown
Member
  • Updates int tests to use the BaSyx Go AASX File Server
  • Improves test engine result representation

Closes #536

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the integration-test environment and expectations to run against the BaSyx Go AASX File Server in CI, including required database schema additions.

Changes:

  • Update AASX File Server integration tests to match Go backend behavior (status codes, identifier normalization, unskipped scenarios).
  • Switch CI docker-compose AASX File Server service to the Go image and configure it for PostgreSQL.
  • Extend the CI Postgres schema with AASX File Server tables and compatibility columns.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/integration-tests/aasxFile.integration.test.ts Adjusts integration test expectations and adds identifier normalization for Go server responses.
ci/docker-compose.yml Swaps AASX File Server container to Go implementation and wires it to Postgres + schema init.
ci/basyxschema.sql Adds AASX File Server tables/indexes and schema compatibility alterations for Go services.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/integration-tests/aasxFile.integration.test.ts Outdated
Comment thread ci/basyxschema.sql
Comment thread ci/basyxschema.sql
@aaronzi aaronzi merged commit c010df4 into eclipse-basyx:main May 20, 2026
4 checks passed
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.

Improve Test Engine Result Statements

2 participants