Skip to content

Commit

Permalink
Merge pull request #3296 from thaJeztah/fix_test_create_volume_invali…
Browse files Browse the repository at this point in the history
…d_driver

integration: test_create_volume_invalid_driver allow either 400 or 404
  • Loading branch information
thompson-shaun authored Jan 17, 2025
2 parents 747d23b + fad84c3 commit db7f8b8
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions tests/integration/api_volume_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,16 @@ def test_create_volume(self):
assert result['Driver'] == 'local'

def test_create_volume_invalid_driver(self):
driver_name = 'invalid.driver'
# special name to avoid exponential timeout loop
# https://github.com/moby/moby/blob/9e00a63d65434cdedc444e79a2b33a7c202b10d8/pkg/plugins/client.go#L253-L254
driver_name = 'this-plugin-does-not-exist'

with pytest.raises(docker.errors.NotFound):
with pytest.raises(docker.errors.APIError) as cm:
self.client.create_volume('perfectcherryblossom', driver_name)
assert (
cm.value.response.status_code == 404 or
cm.value.response.status_code == 400
)

def test_list_volumes(self):
name = 'imperishablenight'
Expand Down

0 comments on commit db7f8b8

Please sign in to comment.