Skip to content

tests: add DemoConfig test helper and migrate tests to strict BaseConfig#19

Merged
tomdurrant merged 3 commits intomainfrom
remove_extra
Feb 16, 2026
Merged

tests: add DemoConfig test helper and migrate tests to strict BaseConfig#19
tomdurrant merged 3 commits intomainfrom
remove_extra

Conversation

@tomdurrant
Copy link
Contributor

@tomdurrant tomdurrant commented Feb 16, 2026

Summary:

  • Add tests/test_helpers.py DemoConfig to allow tests to pass arg1/arg2 without relaxing production BaseConfig.
  • Migrate tests that previously constructed BaseConfig(arg1=..., arg2=...) to use DemoConfig.
  • Restore BaseConfig to a single noop call and keep production models strict (extra='forbid').

Notes:

  • DemoConfig is test-only and intentionally not registered in production unions/discriminators.
  • I did not change ModelRun unions or production registration of config types.

Please review and merge when ready.

@tomdurrant tomdurrant merged commit b950d62 into main Feb 16, 2026
1 check passed
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.

1 participant