Skip to content

Run tests against NA server for performance - #166

Merged
mroloux merged 1 commit into
masterfrom
matti/run-tests-na-server
Sep 21, 2026
Merged

mroloux merged 1 commit into
masterfrom
matti/run-tests-na-server

Conversation

@mroloux

@mroloux mroloux commented Sep 21, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 21, 2026 07:35

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.

Copilot review overview

🟢 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates integration tests to run against the North America staging server while keeping usage-report tests on EU staging.

Changes:

  • Switches general tests and system credentials to NA.
  • Adds dedicated EU configuration for usage-report tests.
  • Removes obsolete demo-company helpers.
File Description
tests/​seatsioClientTest.py Uses the generic system API secret variable.
tests/​reports/​usage/​testUsageReports.py Configures usage tests independently.
.github/​workflows/​build.yml Sets NA and EU test environments.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mroloux
mroloux marked this pull request as ready for review September 21, 2026 07:38
@mroloux
mroloux requested a review from a team as a code owner September 21, 2026 07:38
@mroloux
mroloux requested review from bverbeken and schaloner and removed request for a team September 21, 2026 07:38
@mroloux
mroloux merged commit f5bcf83 into master Sep 21, 2026
3 checks passed
@mroloux
mroloux deleted the matti/run-tests-na-server branch September 21, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants