Skip to content

Add vardok mapping not found tests#396

Merged
mmwinther merged 8 commits intomainfrom
chore/generate-vardef-client
Feb 27, 2026
Merged

Add vardok mapping not found tests#396
mmwinther merged 8 commits intomainfrom
chore/generate-vardef-client

Conversation

@mmwinther
Copy link
Member

@mmwinther mmwinther commented Feb 27, 2026

  • Refactor naming of client methods
  • Remove mocking in some tests
  • Add tests for when the mapping is not found

Ref: DPMETA-1326

@mmwinther mmwinther requested a review from a team as a code owner February 27, 2026 09:29
@github-actions github-actions bot added ci Continuous Integration skip-changelog Changes that should be omitted from the release notes labels Feb 27, 2026
Copy link
Contributor

@tilen1976 tilen1976 left a comment

Choose a reason for hiding this comment

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

Looks great

@mmwinther mmwinther merged commit ff4233c into main Feb 27, 2026
19 checks passed
@mmwinther mmwinther deleted the chore/generate-vardef-client branch February 27, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous Integration skip-changelog Changes that should be omitted from the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants