[ARVADOS] updated: 2.1.0-2397-g35bb2d982
Git user
git at public.arvados.org
Wed Apr 27 17:33:58 UTC 2022
Summary of changes:
sdk/go/health/aggregator_test.go | 9 +++++++++
services/api/test/integration/errors_test.rb | 2 +-
2 files changed, 10 insertions(+), 1 deletion(-)
via 35bb2d98244d40106c0493385f90cdd198c91447 (commit)
via 1cbf3ce5f6bbb72b9c457e78ae406a557630bebe (commit)
from 588358fc0aa03a5036a9aa21f15b604e9e5c98c7 (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 35bb2d98244d40106c0493385f90cdd198c91447
Author: Tom Clegg <tom at curii.com>
Date: Wed Apr 27 13:33:40 2022 -0400
18794: Fix "check" command test.
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>
diff --git a/sdk/go/health/aggregator_test.go b/sdk/go/health/aggregator_test.go
index 050b2c0fc..f9b6dc6ae 100644
--- a/sdk/go/health/aggregator_test.go
+++ b/sdk/go/health/aggregator_test.go
@@ -245,10 +245,19 @@ func (s *AggregatorSuite) TestCheckCommand(c *check.C) {
confdata = regexp.MustCompile(`Source(Timestamp|SHA256): [^\n]+\n`).ReplaceAll(confdata, []byte{})
err = ioutil.WriteFile(tmpdir+"/config.yml", confdata, 0777)
c.Assert(err, check.IsNil)
+
var stdout, stderr bytes.Buffer
+
exitcode := CheckCommand.RunCommand("check", []string{"-config=" + tmpdir + "/config.yml"}, &bytes.Buffer{}, &stdout, &stderr)
c.Check(exitcode, check.Equals, 0)
c.Check(stderr.String(), check.Equals, "")
+ c.Check(stdout.String(), check.Equals, "")
+
+ stdout.Reset()
+ stderr.Reset()
+ exitcode = CheckCommand.RunCommand("check", []string{"-config=" + tmpdir + "/config.yml", "-yaml"}, &bytes.Buffer{}, &stdout, &stderr)
+ c.Check(exitcode, check.Equals, 0)
+ c.Check(stderr.String(), check.Equals, "")
c.Check(stdout.String(), check.Matches, `(?ms).*(\n|^)health: OK\n.*`)
}
commit 1cbf3ce5f6bbb72b9c457e78ae406a557630bebe
Author: Tom Clegg <tom at curii.com>
Date: Wed Apr 27 13:21:14 2022 -0400
18794: Fix routing test.
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>
diff --git a/services/api/test/integration/errors_test.rb b/services/api/test/integration/errors_test.rb
index a5359278e..8e9929117 100644
--- a/services/api/test/integration/errors_test.rb
+++ b/services/api/test/integration/errors_test.rb
@@ -24,7 +24,7 @@ class ErrorsTest < ActionDispatch::IntegrationTest
# Generally, new routes should appear under /arvados/v1/. If
# they appear elsewhere, that might have been caused by default
# rails generator behavior that we don't want.
- assert_match(/^\/(|\*a|arvados\/v1\/.*|auth\/.*|login|logout|database\/reset|discovery\/.*|static\/.*|sys\/trash_sweep|themes\/.*|assets|_health|metrics\/.*)(\(\.:format\))?$/,
+ assert_match(/^\/(|\*a|arvados\/v1\/.*|auth\/.*|login|logout|database\/reset|discovery\/.*|static\/.*|sys\/trash_sweep|themes\/.*|assets|_health\/.*|metrics)(\(\.:format\))?$/,
route.path.spec.to_s,
"Unexpected new route: #{route.path.spec}")
end
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list