Skip to content

Commit 532d1e2

Browse files
committed
BUG/MINOR: e2e: fix dependency between spoe tests
Since setup and teardown functions run before and after every test, one test would fail if running in isolation for spoe tests. Bug fixed for test cases when API should return an error for trying adding existing resource. Additionally, version qs param is now dynamically obtained - previously tests passed because of wrong version number.
1 parent fa8ad65 commit 532d1e2

File tree

4 files changed

+8
-12
lines changed

4 files changed

+8
-12
lines changed

e2e/tests/spoe_agents/add.bats

+2-3
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,9 @@ teardown() {
4949
assert_equal "$SC" 200
5050

5151
assert_equal "$(get_json_path "$BODY" ".data")" "$(get_json_path "$(cat "$BATS_TEST_DIRNAME"/data/post.json)" ".")"
52-
}
5352

54-
@test "spoe_agents: Refuse adding an existing spoe agent" {
55-
run dpa_curl POST "$_SPOE_AGENTS_BASE_PATH?spoe=spoefile_example.cfg&version=2&scope=%5Bip-reputation%5D" /data/post.json
53+
# refuse adding an existing spoe agent
54+
resource_post "$_SPOE_AGENTS_BASE_PATH" "/data/post.json" "spoe=spoefile_example.cfg&scope=%5Bip-reputation%5D"
5655
assert_success
5756

5857
dpa_curl_status_body '$output'

e2e/tests/spoe_groups/add.bats

+2-3
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,9 @@ teardown() {
4949
assert_equal "$SC" 200
5050

5151
assert_equal "$(get_json_path "${BODY}" ".data")" "$(cat ${BATS_TEST_DIRNAME}/data/post.json)"
52-
}
5352

54-
@test "spoe_groups: Refuse adding an existing spoe group" {
55-
run dpa_curl POST "$_SPOE_GROUPS_BASE_PATH?spoe=spoefile_example.cfg&version=2&scope=%5Bip-reputation%5D" /data/post.json
53+
# refuse adding an existing spoe group
54+
resource_post "$_SPOE_GROUPS_BASE_PATH" "/data/post.json" "spoe=spoefile_example.cfg&scope=%5Bip-reputation%5D"
5655
assert_success
5756

5857
dpa_curl_status_body '$output'

e2e/tests/spoe_messages/add.bats

+2-3
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,9 @@ teardown() {
4949
assert_equal "$SC" 200
5050

5151
assert_equal "$(get_json_path "${BODY}" ".data")" "$(cat ${BATS_TEST_DIRNAME}/data/post.json)"
52-
}
5352

54-
@test "spoe_messages: Refuse adding an existing spoe message" {
55-
run dpa_curl POST "$_SPOE_MESSAGES_BASE_PATH?spoe=spoefile_example.cfg&version=2&scope=%5Bip-reputation%5D" /data/post.json
53+
# refuse adding an existing spoe message
54+
resource_post "$_SPOE_MESSAGES_BASE_PATH" "/data/post.json" "spoe=spoefile_example.cfg&scope=%5Bip-reputation%5D"
5655
assert_success
5756

5857
dpa_curl_status_body '$output'

e2e/tests/spoe_scopes/add.bats

+2-3
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,9 @@ teardown() {
4343
@test "spoe_scopes: Add a spoe scope" {
4444
resource_post "$_SPOE_SCOPES_BASE_PATH" "data/add_scope.txt" "spoe=spoefile_example.cfg"
4545
assert_equal "$SC" 201
46-
}
4746

48-
@test "spoe_scopes: Refuse adding an existing spoe scope" {
49-
run dpa_curl POST "/services/haproxy/spoe/spoe_scopes?spoe=spoefile_example.cfg&version=2" /data/add_scope.txt
47+
# refuse adding an existing spoe scope
48+
resource_post "$_SPOE_SCOPES_BASE_PATH" "/data/add_scope.txt" "spoe=spoefile_example.cfg"
5049
assert_success
5150

5251
dpa_curl_status_body '$output'

0 commit comments

Comments
 (0)