Skip to content

Conversation

@subkanthi
Copy link
Collaborator

@subkanthi subkanthi commented Feb 12, 2026

closes: #106

  • Added Docker integration tests to run on pre-release before pushing docker image.
  • Moved common functions from DockerScenarioBasedIT and ScenarioBasedIT to RestCatalogITBase.
  • Removed cloudResources and phases from all tests.

@subkanthi subkanthi requested a review from xieandrew February 12, 2026 10:38
Copy link
Collaborator

@xieandrew xieandrew left a comment

Choose a reason for hiding this comment

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

The tests work properly, just a few small comments

*/
public class ScenarioBasedIT extends RESTCatalogTestBase {

/**
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These functions were moved to RESTCatalogTestBase

@subkanthi subkanthi requested a review from xieandrew February 12, 2026 20:40
Copy link
Collaborator

@xieandrew xieandrew left a comment

Choose a reason for hiding this comment

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

LGTM, we should move the docker tests in the verify workflow to a separate job in a future PR

@xieandrew xieandrew changed the title Add Docker integration tests. test: Add Docker integration tests Feb 12, 2026
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.

Class not found error

2 participants