Skip to content

integration-cli: move some tests to integration#51228

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:move_some_integration_cli_tests
Oct 21, 2025
Merged

integration-cli: move some tests to integration#51228
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:move_some_integration_cli_tests

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Oct 20, 2025

Migrates:

  • TestAPIErrorJSON
  • TestContainerAPIInvalidPortSyntax
  • TestContainerAPIRestartPolicyInvalidPolicyName
  • TestContainerAPIRestartPolicyRetryMismatch
  • TestContainerAPIRestartPolicyNegativeRetryCount
  • TestContainerAPIRestartPolicyDefaultRetryCount
  • TestCreateWithTooLowMemoryLimit

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah thaJeztah added status/2-code-review area/testing kind/refactor PR's that refactor, or clean-up code labels Oct 20, 2025
@thaJeztah thaJeztah force-pushed the move_some_integration_cli_tests branch from 4b0542f to a28fc24 Compare October 20, 2025 16:21
@thaJeztah
Copy link
Member Author

Actually; just realised this also covers TestAPIErrorJSON; #51176

Let me add @sameergupta4873 as co-author, then that PR doesn't have to be rebased / updated.

@thaJeztah thaJeztah force-pushed the move_some_integration_cli_tests branch 2 times, most recently from 8568125 to 63c017d Compare October 20, 2025 16:56
@thaJeztah thaJeztah marked this pull request as ready for review October 21, 2025 08:18
@thaJeztah thaJeztah requested a review from robmry October 21, 2025 08:28
Migrates:

- TestAPIErrorJSON
- TestContainerAPIInvalidPortSyntax
- TestContainerAPIRestartPolicyInvalidPolicyName
- TestContainerAPIRestartPolicyRetryMismatch
- TestContainerAPIRestartPolicyNegativeRetryCount
- TestContainerAPIRestartPolicyDefaultRetryCount
- TestCreateWithTooLowMemoryLimit

Co-authored-by: Sameer Gupta <sameergupta4873@gmail.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the move_some_integration_cli_tests branch from 63c017d to 9fe856a Compare October 21, 2025 10:02
@thaJeztah thaJeztah merged commit acc9563 into moby:master Oct 21, 2025
54 checks passed
@thaJeztah thaJeztah deleted the move_some_integration_cli_tests branch October 21, 2025 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants