Skip to content

Commit cb21af7

Browse files
committed
Fix tests that look at 'Aliases'
Inspect output for 'NetworkSettings.Networks.<network>.Aliases' includes the container's short-id (although it will be removed in API v1.45, in moby 26.0). Signed-off-by: Rob Murray <[email protected]>
1 parent bd164f9 commit cb21af7

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

tests/integration/models_containers_test.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -110,12 +110,12 @@ def test_run_with_networking_config(self):
110110
client.networks.create(net_name)
111111
self.tmp_networks.append(net_name)
112112

113-
test_aliases = ['hello']
113+
test_alias = 'hello'
114114
test_driver_opt = {'key1': 'a'}
115115

116116
networking_config = {
117117
net_name: client.api.create_endpoint_config(
118-
aliases=test_aliases,
118+
aliases=[test_alias],
119119
driver_opt=test_driver_opt
120120
)
121121
}
@@ -132,8 +132,10 @@ def test_run_with_networking_config(self):
132132
assert 'NetworkSettings' in attrs
133133
assert 'Networks' in attrs['NetworkSettings']
134134
assert list(attrs['NetworkSettings']['Networks'].keys()) == [net_name]
135-
assert attrs['NetworkSettings']['Networks'][net_name]['Aliases'] == \
136-
test_aliases
135+
# Expect Aliases to list 'test_alias' and the container's short-id.
136+
# In API version 1.45, the short-id will be removed.
137+
assert attrs['NetworkSettings']['Networks'][net_name]['Aliases'] \
138+
== [test_alias, attrs['Id'][:12]]
137139
assert attrs['NetworkSettings']['Networks'][net_name]['DriverOpts'] \
138140
== test_driver_opt
139141

@@ -190,7 +192,9 @@ def test_run_with_networking_config_only_undeclared_network(self):
190192
assert 'NetworkSettings' in attrs
191193
assert 'Networks' in attrs['NetworkSettings']
192194
assert list(attrs['NetworkSettings']['Networks'].keys()) == [net_name]
193-
assert attrs['NetworkSettings']['Networks'][net_name]['Aliases'] is None
195+
# Aliases should include the container's short-id (but it will be removed
196+
# in API v1.45).
197+
assert attrs['NetworkSettings']['Networks'][net_name]['Aliases'] == [attrs["Id"][:12]]
194198
assert (attrs['NetworkSettings']['Networks'][net_name]['DriverOpts']
195199
is None)
196200

0 commit comments

Comments
 (0)