Skip to content

Commit

Permalink
Fix ipWhiteList test (#3181)
Browse files Browse the repository at this point in the history
Port change seems to fix a timing issue.
  • Loading branch information
KristjanESPERANTO authored Sep 6, 2023
1 parent ef20fe2 commit 9d49196
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 4 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ _This release is scheduled to be released on 2023-10-01._
- Fix undefined formatTime method in clock module (#3143)
- Fix clientonly startup fails after async added (#3151)
- Fix electron width/heigth when using xrandr under bullseye
- Fix ipWhiteList test (#3179)

## [2.24.0] - 2023-07-01

Expand Down
3 changes: 2 additions & 1 deletion tests/configs/empty_ipWhiteList.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@
* MIT Licensed.
*/
let config = require(`${process.cwd()}/tests/configs/default.js`).configFactory({
ipWhitelist: []
ipWhitelist: [],
port: 8282
});

/*************** DO NOT EDIT THE LINE BELOW ***************/
Expand Down
3 changes: 2 additions & 1 deletion tests/configs/noIpWhiteList.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@
* MIT Licensed.
*/
let config = require(`${process.cwd()}/tests/configs/default.js`).configFactory({
ipWhitelist: ["x.x.x.x"]
ipWhitelist: ["x.x.x.x"],
port: 8181
});

/*************** DO NOT EDIT THE LINE BELOW ***************/
Expand Down
4 changes: 2 additions & 2 deletions tests/e2e/ipWhitelist_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ describe("ipWhitelist directive configuration", () => {
});

it("should return 403", async () => {
const res = await helpers.fetch("http://localhost:8080");
const res = await helpers.fetch("http://localhost:8181");
expect(res.status).toBe(403);
});
});
Expand All @@ -24,7 +24,7 @@ describe("ipWhitelist directive configuration", () => {
});

it("should return 200", async () => {
const res = await helpers.fetch("http://localhost:8080");
const res = await helpers.fetch("http://localhost:8282");
expect(res.status).toBe(200);
});
});
Expand Down

0 comments on commit 9d49196

Please sign in to comment.