[arvados] updated: 2.5.0-4-ga98114557

git repository hosting git at public.arvados.org
Tue Jan 17 22:51:44 UTC 2023

Summary of changes:
 lib/crunchrun/crunchrun_test.go | 1 +
 1 file changed, 1 insertion(+)

       via  a98114557438e69fc7fc088d5d4c19ac2d4c5274 (commit)
      from  458436270ce8fb80d421d55e192236c5ac4a225e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

commit a98114557438e69fc7fc088d5d4c19ac2d4c5274
Author: Tom Clegg <tom at curii.com>
Date:   Tue Jan 17 17:51:23 2023 -0500

    19320: Fix env var leak in test.
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/lib/crunchrun/crunchrun_test.go b/lib/crunchrun/crunchrun_test.go
index 91a46e10e..1e3b2eb5c 100644
--- a/lib/crunchrun/crunchrun_test.go
+++ b/lib/crunchrun/crunchrun_test.go
@@ -2087,6 +2087,7 @@ func (s *TestSuite) TestCalculateCost(c *C) {
 	// hasn't found any data), cost is calculated based on
 	// InstanceType env var
 	os.Setenv("InstanceType", `{"Price":1.2}`)
+	defer os.Unsetenv("InstanceType")
 	cost = cr.calculateCost(now)
 	c.Check(cost, Equals, 1.2)



More information about the arvados-commits mailing list