Skip to content

Conversation

@aholstrup1
Copy link
Contributor

@aholstrup1 aholstrup1 commented Oct 22, 2025

Summary

This PR updates our test setup in BCApps to better match our internal gates. Specifically, the PR adds:

  • A job to run the UnitTests without demo data (W1 only)
  • A job to run the Integration tests with setup demo data (One per country)
  • A job to run the Uncategorized tests on the full demo data (W1 only)

As a result of these changes, we can also enable tests for all countries except DK and CZ.

Left for future work:

  • Move more (ideally all) Uncategorized tests to either be Integration or Unit tests. @haoranpb
  • Enable testing in DK and CZ
  • Add a small gate to ensure we don't add more Uncategorized tests to BCApps @aholstrup1

Work Item(s)

Related to AB#609191

@aholstrup1 aholstrup1 requested a review from a team as a code owner October 22, 2025 13:08
@github-actions github-actions bot added the Build: scripts & configs Build scripts and configuration files label Oct 22, 2025
@github-actions github-actions bot added this to the Version 28.0 milestone Oct 22, 2025
@aholstrup1 aholstrup1 requested a review from a team as a code owner October 23, 2025 09:45
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Oct 23, 2025
@aholstrup1 aholstrup1 changed the title Switch to setup data during integration testing [Draft] Switch to setup data during integration testing Oct 23, 2025
@aholstrup1 aholstrup1 changed the title [Draft] Switch to setup data during integration testing Run integration tests and unit tests Oct 23, 2025
haoranpb
haoranpb previously approved these changes Oct 24, 2025
haoranpb
haoranpb previously approved these changes Oct 24, 2025
Added functionality to run tests with and without isolation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Build: scripts & configs Build scripts and configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants