add case without explicit dns server

remove checking for answer string as it only appears in non authoritative dns responses

Author: QxBytes

test/integration/lrp/lrp_fqdn_test.go

@@ -52,11 +52,16 @@ func TestLRPFQDN(t *testing.T) {
 		countIncreases         bool
 	}{
 		{
-			name:                "nslookup google succeeds",
-			command:             []string{"nslookup", "www.google.com", "10.0.0.10"},
-			expectedMsgContains: "answer:",
-			countIncreases:      true,
-			shouldError:         false,
+			name:           "nslookup google succeeds",
+			command:        []string{"nslookup", "www.google.com", "10.0.0.10"},
+			countIncreases: true,
+			shouldError:    false,
+		},
+		{
+			name:           "nslookup google succeeds without explicit dns server",
+			command:        []string{"nslookup", "www.google.com"},
+			countIncreases: true,
+			shouldError:    false,
 		},
 		{
 			name:                   "wget google succeeds",
@@ -66,11 +71,10 @@ func TestLRPFQDN(t *testing.T) {
 			shouldError:            false,
 		},
 		{
-			name:                "nslookup bing succeeds",
-			command:             []string{"nslookup", "www.bing.com", "10.0.0.10"},
-			expectedMsgContains: "answer:",
-			countIncreases:      true,
-			shouldError:         false,
+			name:           "nslookup bing succeeds",
+			command:        []string{"nslookup", "www.bing.com", "10.0.0.10"},
+			countIncreases: true,
+			shouldError:    false,
 		},
 		{
 			name:                   "wget bing fails but dns succeeds",

test/integration/lrp/lrp_test.go

@@ -174,15 +174,15 @@ func testLRPCase(t *testing.T, ctx context.Context, clientPod corev1.Pod, client
 	t.Log("calling command from client")
 
 	val, errMsg, err := kubernetes.ExecCmdOnPod(ctx, cs, clientPod.Namespace, clientPod.Name, clientContainer, clientCmd, config, false)
-
-	require.Contains(t, string(val), expectResponse)
-	require.Contains(t, string(errMsg), expectErrMsg)
 	if shouldError {
 		require.Error(t, err)
 	} else {
 		require.NoError(t, err)
 	}
 
+	require.Contains(t, string(val), expectResponse)
+	require.Contains(t, string(errMsg), expectErrMsg)
+
 	// in case there is time to propagate
 	time.Sleep(500 * time.Millisecond)
 
@@ -212,7 +212,7 @@ func TestLRP(t *testing.T) {
 
 	testLRPCase(t, ctx, *selectedPod, []string{
 		"nslookup", "google.com", "10.0.0.10",
-	}, "Server:", "", false, true)
+	}, "", "", false, true)
 }
 
 // TakeOne takes one item from the slice randomly; if empty, it returns the empty value for the type