remove namespaces because oss agent response doesnt provide it

hashicorp · Apr 2, 2021 · 9797b44 · 9797b44
1 parent bd4eba7
commit 9797b44
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/connect-inject/endpoints_controller_test.go b/connect-inject/endpoints_controller_test.go
@@ -727,7 +727,8 @@ func TestReconcileCreateEndpoint(t *testing.T) {
 				check, err := consulClient.Agent().ChecksWithFilter(filter)
 				require.NoError(t, err)
 				require.EqualValues(t, len(check), 1)
-				var ignoredFields = []string{"Node", "Definition"}
+				// Ignoring Namespace because the response from ENT includes it and OSS does not.
+				var ignoredFields = []string{"Node", "Definition", "Namespace"}
 				require.True(t, cmp.Equal(check[tt.expectedAgentHealthCheck.CheckID], tt.expectedAgentHealthCheck, cmpopts.IgnoreFields(api.AgentCheck{}, ignoredFields...)))
 			}
 		})
@@ -1482,7 +1483,8 @@ func TestReconcileUpdateEndpoint(t *testing.T) {
 				check, err := consulClient.Agent().ChecksWithFilter(filter)
 				require.NoError(t, err)
 				require.EqualValues(t, len(check), 1)
-				var ignoredFields = []string{"Node", "Definition"}
+				// Ignoring Namespace because the response from ENT includes it and OSS does not.
+				var ignoredFields = []string{"Node", "Definition", "Namespace"}
 				require.True(t, cmp.Equal(check[tt.expectedAgentHealthCheck.CheckID], tt.expectedAgentHealthCheck, cmpopts.IgnoreFields(api.AgentCheck{}, ignoredFields...)))
 			}
 		})