Skip to content

Add missing integration test for filter ordering - #350

Merged
MartinWheelerMT merged 1 commit into
mainfrom
add-filter-order-int-test
Sep 7, 2026
Merged

MartinWheelerMT merged 1 commit into
mainfrom
add-filter-order-int-test

Conversation

@MartinWheelerMT

@MartinWheelerMT MartinWheelerMT commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

What

  • Add default values for missing required environment variable in application.yml for the integration tests.
  • Add missing integration test for filter ordering.

Why

The current way the filters work is unclear and ungainly. In order to complete some refactoring work on the filters and how they are loaded requires that test are in place to ensure the loading and ordering of the filters is completed correctly.

Type of Change

Please check the option(s) that apply.

  • Internal change (non-breaking change with no effect on functionality affecting end users)

Checklist

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have updated the Changelog with details of my change in the UNRELEASED section, if this change will affect end users

* Add default values for missing required environment variable in `application.yml` for the integration tests.
* Add missing integration test for filter ordering.
@MartinWheelerMT
MartinWheelerMT requested a review from a team as a code owner September 4, 2026 15:40
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Images built and published to ECR using a Build Id of PR-405-21295bc

@MartinWheelerMT
MartinWheelerMT merged commit 5035ef5 into main Sep 7, 2026
7 checks passed
@MartinWheelerMT
MartinWheelerMT deleted the add-filter-order-int-test branch September 7, 2026 13:29
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