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.
  • Loading branch information
flrgh authored Dec 9, 2024
1 parent 7eecd96 commit 8f35f67
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

1 comment on commit 8f35f67

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bazel Build

Docker image available kong/kong:8f35f670914bce85e10b0fe20cb0a3ff72a5d929
Artifacts available https://github.com/Kong/kong/actions/runs/12245583089

Please sign in to comment.