Skip to content

Fix MissingArgumentsPropertyInGenericTestDeprecation warning - #568

Open
a-schot wants to merge 1 commit into
brooklyn-data:mainfrom
a-schot:fix/generic-test-arguments-deprecation
Open

Fix MissingArgumentsPropertyInGenericTestDeprecation warning#568
a-schot wants to merge 1 commit into
brooklyn-data:mainfrom
a-schot:fix/generic-test-arguments-deprecation

Conversation

@a-schot

@a-schot a-schot commented Aug 18, 2026

Copy link
Copy Markdown

Overview

Fix MissingArgumentsPropertyInGenericTestDeprecation warnings that fire when parsing dbt_artifacts models on dbt Core 1.10.5+.

Update type - breaking / non-breaking

  • Minor bug fix

What does this solve?

Resolves #567

Wraps generic test arguments under the arguments: property:

  • 9x dbt_artifacts.is_betweenmin_value/max_value nested under arguments:
  • 2x accepted_valuesvalues nested under arguments:

The arguments: property is recognized by dbt Core >=1.10.5 and is required (with deprecation warning) from 1.10.8+. On older versions the property is silently ignored, so this change is fully backward-compatible with the package's existing require-dbt-version constraint.

Outstanding questions

None.

What databases have you tested with?

  • Snowflake
  • Google BigQuery
  • Databricks
  • Spark
  • N/A

@a-schot
a-schot marked this pull request as draft August 18, 2026 12:59
Wrap generic test arguments under the 'arguments:' property:
- 9x dbt_artifacts.is_between (min_value/max_value)
- 2x accepted_values (values)

Resolves brooklyn-data#567
@a-schot
a-schot force-pushed the fix/generic-test-arguments-deprecation branch from 9485a38 to 8307308 Compare August 18, 2026 13:07
@a-schot
a-schot marked this pull request as ready for review August 18, 2026 13:11
@stefankeidel

Copy link
Copy Markdown

We've encountered the same thing and I was delighted to see an open PR already. ⬆️

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.

[Bug]: MissingArgumentsPropertyInGenericTestDeprecation warning flagged

2 participants