From 229b502ee1ba3d7c25a422c8a3569f2bd38ae34f Mon Sep 17 00:00:00 2001 From: Manuel Palenzuela Merino Date: Fri, 28 Jun 2024 17:16:21 +0200 Subject: [PATCH] Remove TestCodeOwners test This test depends on a real file of the repo, so if there are any unrelated changes to that file the test could fail. Most of the functions this test "tests" are already covered by other tests. --- internal/civisibility/utils/codeowners_test.go | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/internal/civisibility/utils/codeowners_test.go b/internal/civisibility/utils/codeowners_test.go index 9ede4f4ee0..95852bec4a 100644 --- a/internal/civisibility/utils/codeowners_test.go +++ b/internal/civisibility/utils/codeowners_test.go @@ -13,17 +13,6 @@ import ( "github.com/stretchr/testify/assert" ) -func TestCodeOwners(t *testing.T) { - codeOwners := GetCodeOwners() - assert.NotNil(t, codeOwners) - - appSecMatch, _ := codeOwners.Match("/appsec") - assert.Equal(t, appSecMatch.GetOwnersString(), "[\"@DataDog/asm-go\"]") - - ddtraceMatch, _ := codeOwners.Match("/ddtrace") - assert.Equal(t, ddtraceMatch.GetOwnersString(), "[\"@DataDog/apm-go\"]") -} - func TestNewCodeOwners(t *testing.T) { // Create a temporary file for testing fileContent := `[Section 1]