Skip to content

Commit

Permalink
Incorperate feedback
Browse files Browse the repository at this point in the history
Signed-off-by: Kevin Fox <Kevin.Fox@pnnl.gov>
  • Loading branch information
kfox1111 committed Oct 18, 2024
1 parent 3bd1919 commit c7749cc
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions pkg/common/plugin/sshpop/sshpop_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,14 +106,13 @@ func TestNewServer(t *testing.T) {
{
desc: "success",
configString: fmt.Sprintf(`cert_authorities = [%q]
canonical_domain = "local"
agent_path_template = "/{{ .PluginName}}/{{ .Fingerprint }}"`, testCertAuthority),
canonical_domain = "local"`, testCertAuthority),
trustDomain: "foo.test",
requireServer: func(t *testing.T, s *Server) {
require.NotNil(t, s)
require.Equal(t, "foo.test", s.trustDomain.Name())
require.Equal(t, "local", s.canonicalDomain)
require.Equal(t, DefaultAgentPathTemplate, s.agentPathTemplate)
require.Same(t, DefaultAgentPathTemplate, s.agentPathTemplate)
pubkey, _, _, _, err := ssh.ParseAuthorizedKey([]byte(testCertAuthority))
require.NoError(t, err)
require.True(t, s.certChecker.IsHostAuthority(pubkey, ""))
Expand All @@ -128,7 +127,7 @@ func TestNewServer(t *testing.T) {
requireServer: func(t *testing.T, s *Server) {
require.NotNil(t, s)
require.Equal(t, "foo.test", s.trustDomain.Name())
require.Equal(t, DefaultAgentPathTemplate, s.agentPathTemplate)
require.NotSame(t, DefaultAgentPathTemplate, s.agentPathTemplate)
pubkey := requireParsePubkey(t, testCertAuthority)
pubkey2 := requireParsePubkey(t, testCertAuthority2)
pubkey3 := requireParsePubkey(t, testCertAuthority3)
Expand Down

0 comments on commit c7749cc

Please sign in to comment.