Skip to content

Conversation

@MiguelAAe
Copy link
Contributor

Why

Issues

Motivation

What

Summary of changes

Checklist

  • Added/updated unit tests
  • Added/updated documentation
  • Checked for typos in variable names, comments, etc.
  • Added licences for new files

Testing

@MiguelAAe MiguelAAe force-pushed the exp/bdd-integration-testing branch from f2ff855 to 2d86dd9 Compare November 27, 2025 17:44
@MiguelAAe MiguelAAe force-pushed the exp/bdd-integration-testing branch from c80c7f6 to d335607 Compare December 2, 2025 13:20
@MiguelAAe MiguelAAe force-pushed the exp/bdd-integration-testing branch from ba79efa to 13d559d Compare December 3, 2025 16:28
@MiguelAAe MiguelAAe force-pushed the exp/bdd-integration-testing branch from abb6201 to ddd1a36 Compare December 4, 2025 10:35
// Model Deployments
ctx.Step(`^the model is applied$`, w.ApplyModel)
// Model Assertions
ctx.Step(`^the model (?:should )?eventually become(?:s)? Ready$`, w.ModelReady)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Assertions will grow as we need to check for different types of assertions for model status, gateway status etc might be worth implementing some sort of pattern matcher for this type of steps that match regular expression to assertions that are then executed with the Waitfor found here: https://github.com/SeldonIO/seldon-core/pull/6965/files#diff-08a8e491ec85e75d1ee2c67f0009b9303460abb7856178d9a3f2a87f7e4023baR145

@domsolutions

@domsolutions domsolutions marked this pull request as ready for review December 5, 2025 13:22
@domsolutions domsolutions requested a review from lc525 as a code owner December 5, 2025 13:22
@domsolutions domsolutions merged commit 52e9a29 into SeldonIO:v2 Dec 5, 2025
4 of 5 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.

2 participants