Skip to content

Modified integtests to use the attribute_substitution data class instead of config_substitution…#440

Merged
bieryAtFnal merged 2 commits into
developfrom
kbiery/integtest_changes_for_attribute_substitutions
Sep 4, 2025
Merged

Modified integtests to use the attribute_substitution data class instead of config_substitution…#440
bieryAtFnal merged 2 commits into
developfrom
kbiery/integtest_changes_for_attribute_substitutions

Conversation

@bieryAtFnal
Copy link
Copy Markdown
Collaborator

This is part of providing more configuration substitution options in integtests.

Description

The changes in this PR make use of the new way of specifying configuration substitutions in integration tests, as described in DUNE-DAQ/daq-deliverables#181.

Testing Suggestions

Please see the suggestions in the parent Issue, DUNE-DAQ/daq-deliverables#181.

…ead of config_substitution. This is part of providing more configuration substitution options in integtests.
@bieryAtFnal bieryAtFnal requested a review from eflumerf September 3, 2025 13:40
Copy link
Copy Markdown
Member

@eflumerf eflumerf left a comment

Choose a reason for hiding this comment

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

I ran dbt-build --integtest and things functioned normally (disabled_output_test and insufficient_disk_space_test have issues which are addressed elsewhere)

@eflumerf eflumerf mentioned this pull request Sep 4, 2025
@bieryAtFnal bieryAtFnal merged commit 8abd563 into develop Sep 4, 2025
2 checks passed
@bieryAtFnal bieryAtFnal deleted the kbiery/integtest_changes_for_attribute_substitutions branch September 4, 2025 18:09
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.

3 participants