Skip to content

Commit

Permalink
test/compose: remove compose v1 code
Browse files Browse the repository at this point in the history
Now that we only test compose v2 remove the special cases from the test
code to simply the tests.

Signed-off-by: Paul Holzinger <[email protected]>
  • Loading branch information
Luap99 committed Apr 18, 2024
1 parent 2c61ee9 commit b1736c4
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 19 deletions.
7 changes: 4 additions & 3 deletions test/compose/README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
Tests for docker-compose
========================
Tests for docker-compose v2
===========================

This directory contains tests for docker-compose under podman.
This directory contains tests for docker-compose v2 under podman.
docker-compose v1 is no longer supported upstream so we no longer test with it.

Each subdirectory must contain one docker-compose.yml file along with
all necessary infrastructure for it (e.g. Containerfile, any files
Expand Down
5 changes: 1 addition & 4 deletions test/compose/etc_hosts/tests.sh
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
# -*- bash -*-

ctr_name="etc_hosts_test_1"
if [ "$TEST_FLAVOR" = "compose_v2" ]; then
ctr_name="etc_hosts-test-1"
fi
ctr_name="etc_hosts-test-1"

podman exec "$ctr_name" sh -c 'grep "foobar" /etc/hosts'
like "$output" "10\.123\.0\." "$testname : no entries are copied from the host"
Expand Down
5 changes: 1 addition & 4 deletions test/compose/ipam_set_ip/tests.sh
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
# -*- bash -*-

ctr_name="ipam_set_ip_test_1"
if [ "$TEST_FLAVOR" = "compose_v2" ]; then
ctr_name="ipam_set_ip-test-1"
fi
ctr_name="ipam_set_ip-test-1"
podman container inspect "$ctr_name" --format '{{ .NetworkSettings.Networks.ipam_set_ip_net1.IPAddress }}'
is "$output" "10.123.0.253" "$testname : ip address is set"
podman container inspect "$ctr_name" --format '{{ .NetworkSettings.Networks.ipam_set_ip_net1.MacAddress }}'
Expand Down
5 changes: 1 addition & 4 deletions test/compose/two_networks/tests.sh
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
# -*- bash -*-

ctr_name="two_networks_con1_1"
if [ "$TEST_FLAVOR" = "compose_v2" ]; then
ctr_name="two_networks-con1-1"
fi
ctr_name="two_networks-con1-1"
podman container inspect "$ctr_name" --format '{{len .NetworkSettings.Networks}}'
is "$output" "2" "$testname : Container is connected to both networks"
podman container inspect "$ctr_name" --format '{{.NetworkSettings.Networks}}'
Expand Down
5 changes: 1 addition & 4 deletions test/compose/uptwice/tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,5 @@ output=$(podman_compose up -d 2>&1)
# Horrible output check here but we really want to make sure that there are
# no unexpected warning/errors and the normal messages are send on stderr as
# well so we cannot check for an empty stderr.
expected="Recreating uptwice_app_1 ... ${CR}${NL}Recreating uptwice_app_1 ... done$CR"
if [ "$TEST_FLAVOR" = "compose_v2" ]; then
expected="Container uptwice-app-1 Recreate${NL}Container uptwice-app-1 Recreated${NL}Container uptwice-app-1 Starting${NL}Container uptwice-app-1 Started"
fi
expected="Container uptwice-app-1 Recreate${NL}Container uptwice-app-1 Recreated${NL}Container uptwice-app-1 Starting${NL}Container uptwice-app-1 Started"
is "$output" "$expected" "no error output in compose up (#15580)"

0 comments on commit b1736c4

Please sign in to comment.