Skip to content

Commit

Permalink
chore(cli-repl): relax macOS --tlsCertificateSelector test expectat…
Browse files Browse the repository at this point in the history
…ions (#1991)

The actual error message we encounter in CI changed recently (from May 6 to
May 7), likely as the result of changes to build hosts. Allow the more specific
error message here as well, since it covers the same semantics.
  • Loading branch information
addaleax authored May 13, 2024
1 parent 0f45c43 commit 499d803
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/cli-repl/src/tls-certificate-selector.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ describe('arg-mapper.applyTlsCertificateSelector', function () {
return this.skip();
}
expect(() => getTlsCertificateSelector('subject=Foo Bar')).to.throw(
/Could not find a matching certificate/
/Could not find a matching certificate|The specified item could not be found in the keychain/
);
});
});
Expand Down
4 changes: 3 additions & 1 deletion packages/e2e-tests/test/e2e-tls.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -601,7 +601,9 @@ describe('e2e TLS', function () {
'Could not resolve certificate specification'
);
} else if (process.platform === 'darwin') {
shell.assertContainsOutput('Could not find a matching certificate');
shell.assertContainsOutput(
/Could not find a matching certificate|The specified item could not be found in the keychain/
);
} else {
shell.assertContainsOutput(
'tlsCertificateSelector is not supported on this platform'
Expand Down

0 comments on commit 499d803

Please sign in to comment.