Skip to content

Commit

Permalink
tests(wasm): add missing filter tags (#13995)
Browse files Browse the repository at this point in the history
* tests(wasm): fix some eventual assertions

These checks using assert.eventually() could have yielded a false
positive under certain conditions.

* tests(wasm): add missing filter tags

This adds some missing #off and #postgres tags that are used in CI to
filter out certain testing scenarios.

(cherry picked from commit 8f35f67)
  • Loading branch information
flrgh authored and github-actions[bot] committed Dec 9, 2024
1 parent 1f1eb6f commit c8dfab3
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions spec/02-integration/20-wasm/06-clustering_spec.lua
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ describe("#wasm - hybrid mode #postgres" .. " inc_sync=" .. inc_sync, function()
res:read_body()

if res.status ~= 200 then
return {
return nil, {
msg = "bad http status",
exp = 200,
got = res.status,
Expand Down Expand Up @@ -269,7 +269,7 @@ describe("#wasm - hybrid mode #postgres" .. " inc_sync=" .. inc_sync, function()
res:read_body()

if res.status ~= 200 then
return {
return nil, {
msg = "bad http status",
exp = 200,
got = res.status,
Expand Down
6 changes: 3 additions & 3 deletions spec/02-integration/20-wasm/08-declarative_spec.lua
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ local function expect_field_error(res, field, err)
end


describe("#wasm declarative config", function()
describe("#wasm declarative config (db = #off)", function()
local admin
local proxy
local header_name = "x-wasm-dbless"
Expand Down Expand Up @@ -172,7 +172,7 @@ describe("#wasm declarative config", function()
end)


describe("#wasm declarative config (no installed filters)", function()
describe("#wasm declarative config (no installed filters) (db = #off)", function()
local tmp_dir

lazy_setup(function()
Expand Down Expand Up @@ -265,7 +265,7 @@ describe("#wasm declarative config (no installed filters)", function()
end)
end)

describe("#wasm declarative config (wasm = off)", function()
describe("#wasm declarative config (wasm = off) (db = #off)", function()
describe("POST /config", function()
local client

Expand Down
2 changes: 1 addition & 1 deletion spec/02-integration/20-wasm/10-wasmtime_spec.lua
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ for _, v in ipairs({ {"off", "off"}, {"on", "off"}, {"on", "on"}, }) do

for _, role in ipairs({"traditional", "control_plane", "data_plane"}) do

describe("#wasm wasmtime (role: " .. role .. ")", function()
describe("#wasm wasmtime (role: " .. role .. ") (#postgres, #db)", function()
describe("kong prepare", function()
local conf
local prefix = "./wasm"
Expand Down

0 comments on commit c8dfab3

Please sign in to comment.