make sure default scenario does not yield remote unavailable

elastic · Sep 19, 2023 · a75f74a · a75f74a
1 parent c3ec544
commit a75f74a
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/internal/pkg/agent/application/upgrade/artifact/download/fs/verifier_test.go b/internal/pkg/agent/application/upgrade/artifact/download/fs/verifier_test.go
@@ -173,7 +173,7 @@ func prepareFetchVerifyTests(dropPath, targetDir, targetFilePath, hashTargetFile
 }
 
 func TestVerify(t *testing.T) {
-	log, _ := logger.New("", false)
+	log, obs := logger.NewTesting("TestVerify")
 	targetDir, err := ioutil.TempDir(os.TempDir(), "")
 	if err != nil {
 		t.Fatal(err)
@@ -214,6 +214,10 @@ func TestVerify(t *testing.T) {
 	err = testVerifier.Verify(beatSpec, version, false)
 	require.NoError(t, err)
 
+	// log message informing remote PGP was skipped
+	logs := obs.FilterMessageSnippet("Skipped remote PGP located at")
+	require.Equal(t, 0, logs.Len())
+
 	os.Remove(artifact)
 	os.Remove(artifact + ".sha512")
 	os.RemoveAll(config.DropPath)