Skip to content

fix(staging): align smoke admin creds and validate label limits#147

Merged
vsxd merged 1 commit intomainfrom
fix/staging-smoke-label-config
Mar 23, 2026
Merged

fix(staging): align smoke admin creds and validate label limits#147
vsxd merged 1 commit intomainfrom
fix/staging-smoke-label-config

Conversation

@vsxd
Copy link
Collaborator

@vsxd vsxd commented Mar 23, 2026

Summary

  • pass the staging bootstrap admin credentials into scripts/smoke-test.sh so make staging uses the same login as the staging compose stack
  • fail fast when skillhub.label.max-definitions or skillhub.label.max-per-skill are configured to non-positive values
  • add domain tests covering the new label limit validation paths

Testing

  • cd server && ./mvnw -q -pl skillhub-domain -am -Dsurefire.failIfNoSpecifiedTests=false -Dtest=LabelDefinitionServiceTest,SkillLabelServiceTest test

@vsxd vsxd merged commit 6c69b96 into main Mar 23, 2026
2 checks 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