Skip to content

Commit

Permalink
Remove TestCodeOwners test
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
ManuelPalenzuelaDD committed Jun 28, 2024
1 parent 664d019 commit 229b502
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions internal/civisibility/utils/codeowners_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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]
Expand Down

0 comments on commit 229b502

Please sign in to comment.