From d7cde7d621b6d2fc607207b3035175217b5453b4 Mon Sep 17 00:00:00 2001 From: ramki-bruno Date: Tue, 20 May 2025 00:28:30 +0530 Subject: [PATCH] Fix failing test case when the selected env is Local --- packages/bruno-tests/collection/environments/Local.bru | 2 ++ packages/bruno-tests/collection/environments/Prod.bru | 2 ++ .../collection/scripting/api/bru/getEnvName.bru | 2 +- .../bruno-tests/collection/scripting/api/bru/getEnvVar.bru | 7 +++---- .../bruno-tests/collection/scripting/api/req/getUrl.bru | 2 +- .../collection/scripting/api/res/getResponseTime.bru | 3 ++- 6 files changed, 11 insertions(+), 7 deletions(-) diff --git a/packages/bruno-tests/collection/environments/Local.bru b/packages/bruno-tests/collection/environments/Local.bru index a5ebb4da7..b69a82d3d 100644 --- a/packages/bruno-tests/collection/environments/Local.bru +++ b/packages/bruno-tests/collection/environments/Local.bru @@ -16,6 +16,8 @@ vars { foo: bar testSetEnvVar: bruno-29653 echo-host: https://echo.usebruno.com + env_name: Local + test_get_env_var_key1: foo client_id: client_id_1 client_secret: client_secret_1 auth_url: http://localhost:8080/api/auth/oauth2/authorization_code/authorize diff --git a/packages/bruno-tests/collection/environments/Prod.bru b/packages/bruno-tests/collection/environments/Prod.bru index 294ef8338..0270ea504 100644 --- a/packages/bruno-tests/collection/environments/Prod.bru +++ b/packages/bruno-tests/collection/environments/Prod.bru @@ -16,4 +16,6 @@ vars { foo: bar testSetEnvVar: bruno-29653 echo-host: https://echo.usebruno.com + env_name: Prod + test_get_env_var_key1: foo } diff --git a/packages/bruno-tests/collection/scripting/api/bru/getEnvName.bru b/packages/bruno-tests/collection/scripting/api/bru/getEnvName.bru index 4e7c37ec3..698253da6 100644 --- a/packages/bruno-tests/collection/scripting/api/bru/getEnvName.bru +++ b/packages/bruno-tests/collection/scripting/api/bru/getEnvName.bru @@ -18,6 +18,6 @@ script:pre-request { tests { test("should get env name in scripts", function() { const testEnvName = bru.getVar("testEnvName"); - expect(testEnvName).to.equal("Prod"); + expect(testEnvName).to.equal(bru.getEnvVar("env_name")); }); } diff --git a/packages/bruno-tests/collection/scripting/api/bru/getEnvVar.bru b/packages/bruno-tests/collection/scripting/api/bru/getEnvVar.bru index 6b0276415..0aae00143 100644 --- a/packages/bruno-tests/collection/scripting/api/bru/getEnvVar.bru +++ b/packages/bruno-tests/collection/scripting/api/bru/getEnvVar.bru @@ -10,10 +10,9 @@ get { auth: none } - tests { test("should get env var in scripts", function() { - const host = bru.getEnvVar("host") - expect(host).to.equal("https://testbench-sanity.usebruno.com"); + const host = bru.getEnvVar("test_get_env_var_key1") + expect(host).to.equal("foo"); }); -} \ No newline at end of file +} diff --git a/packages/bruno-tests/collection/scripting/api/req/getUrl.bru b/packages/bruno-tests/collection/scripting/api/req/getUrl.bru index 155a40b7a..f008434b9 100644 --- a/packages/bruno-tests/collection/scripting/api/req/getUrl.bru +++ b/packages/bruno-tests/collection/scripting/api/req/getUrl.bru @@ -18,6 +18,6 @@ assert { tests { test("req.getUrl()", function() { const url = req.getUrl(); - expect(url).to.equal("https://testbench-sanity.usebruno.com/ping"); + expect(url).to.equal(bru.getEnvVar("host") + "/ping"); }); } diff --git a/packages/bruno-tests/collection/scripting/api/res/getResponseTime.bru b/packages/bruno-tests/collection/scripting/api/res/getResponseTime.bru index 236a5eff1..f3df5b39a 100644 --- a/packages/bruno-tests/collection/scripting/api/res/getResponseTime.bru +++ b/packages/bruno-tests/collection/scripting/api/res/getResponseTime.bru @@ -33,7 +33,8 @@ tests { test("res.getResponseTime()", function() { const responseTime = res.getResponseTime(); expect(typeof responseTime).to.eql("number"); - expect(responseTime > 0).to.be.true; + expect(responseTime >= 0).to.be.true; + // 0ms is possible in with local server and CLI run }); }