Updates tests

Adding the cert requirement where it makes sense
And using insecure channel by default on most other cases
(via testReceiveCommands) mostly.

windows-agent/internal/proservices/landscape/executor_test.go

@@ -746,6 +746,8 @@ func testReceiveCommand(t *testing.T, distrosettings distroSettings, homedir str
 	ctx, cancel := context.WithCancel(context.Background())
 	t.Cleanup(cancel)
 
+	ctx = context.WithValue(ctx, landscape.InsecureCredentials, true)
+
 	tb.wslMock = wslmock.New()
 	ctx = wsl.WithMock(ctx, tb.wslMock)
 

windows-agent/internal/proservices/landscape/landscape_test.go

@@ -138,23 +138,25 @@ func TestConnect(t *testing.T) {
 		"Error when the first-contact SendUpdatedInfo fails":   {tokenErr: true, wantErr: true},
 		"Error when the config cannot be accessed":             {breakLandscapeClientConfig: true, wantErr: true},
 		"Error when the config cannot be parsed":               {wantErr: true},
-		"Error when the SSL certificate cannot be read":        {wantErr: true},
-		"Error when the SSL certificate is not valid":          {wantErr: true},
+		"Error when the SSL certificate cannot be read":        {requireCertificate: true, wantErr: true},
+		"Error when the SSL certificate is not valid":          {requireCertificate: true, wantErr: true},
 	}
 
 	for name, tc := range testCases {
 		t.Run(name, func(t *testing.T) {
 			ctx, cancel := context.WithCancel(context.Background())
 			defer cancel()
 
-			if wsl.MockAvailable() {
-				t.Parallel()
-				ctx = wsl.WithMock(ctx, wslmock.New())
-			}
-
 			p := ""
 			if tc.requireCertificate {
 				p = certPath
+			} else {
+				ctx = context.WithValue(ctx, landscape.InsecureCredentials, true)
+			}
+
+			if wsl.MockAvailable() {
+				t.Parallel()
+				ctx = wsl.WithMock(ctx, wslmock.New())
 			}
 
 			lis, server, mockService := setUpLandscapeMock(t, ctx, "localhost:", p)
@@ -305,7 +307,7 @@ func TestSendUpdatedInfo(t *testing.T) {
 
 	for name, tc := range testCases {
 		t.Run(name, func(t *testing.T) {
-			ctx := context.Background()
+			ctx := context.WithValue(context.Background(), landscape.InsecureCredentials, true)
 			if wsl.MockAvailable() {
 				t.Parallel()
 				mock := wslmock.New()
@@ -519,7 +521,7 @@ func TestAutoReconnection(t *testing.T) {
 
 	for name, tc := range testCases {
 		t.Run(name, func(t *testing.T) {
-			ctx, cancel := context.WithCancel(context.Background())
+			ctx, cancel := context.WithCancel(context.WithValue(context.Background(), landscape.InsecureCredentials, true))
 			defer cancel()
 
 			if wsl.MockAvailable() {
@@ -760,6 +762,8 @@ func TestReconnect(t *testing.T) {
 				certPath = t.TempDir()
 				testutils.GenerateTempCertificate(t, certPath)
 				lcapeConfig = fmt.Sprintf("%s\nssl_public_key = {{ .CertPath }}/cert.pem", defaultLandscapeConfig)
+			} else {
+				ctx = context.WithValue(ctx, landscape.InsecureCredentials, true)
 			}
 
 			lis, server, mockServerService := setUpLandscapeMock(t, ctx, "localhost:", certPath)