Skip to content

Commit c8147c0

Browse files
committed
fix uts
1 parent 49b4f72 commit c8147c0

File tree

1 file changed

+21
-26
lines changed

1 file changed

+21
-26
lines changed

pkg/ingresscache/safetrie_test.go

Lines changed: 21 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -448,12 +448,12 @@ func TestSafeTrie(t *testing.T) {
448448
suite.Run(t, new(SafeTrieTestSuite))
449449
}
450450

451-
// --- SingleFunctionNameTestSuite ---
452-
type SingleFunctionNameTestSuite struct {
451+
// --- SingleTargetTestSuite ---
452+
type SingleTargetTestSuite struct {
453453
suite.Suite
454454
}
455455

456-
func (suite *SingleFunctionNameTestSuite) TestContains() {
456+
func (suite *SingleTargetTestSuite) TestContains() {
457457
testCases := []struct {
458458
name string
459459
functionName string
@@ -487,7 +487,7 @@ func (suite *SingleFunctionNameTestSuite) TestContains() {
487487
}
488488
}
489489

490-
func (suite *SingleFunctionNameTestSuite) TestRemoveFunctionName() {
490+
func (suite *SingleTargetTestSuite) TestRemoveFunctionName() {
491491
testCases := []struct {
492492
name string
493493
functionName string
@@ -524,7 +524,7 @@ func (suite *SingleFunctionNameTestSuite) TestRemoveFunctionName() {
524524
}
525525
}
526526

527-
func (suite *SingleFunctionNameTestSuite) TestAddFunctionName() {
527+
func (suite *SingleTargetTestSuite) TestAddFunctionName() {
528528
testCases := []struct {
529529
name string
530530
functionName string
@@ -552,7 +552,7 @@ func (suite *SingleFunctionNameTestSuite) TestAddFunctionName() {
552552
}
553553
}
554554

555-
func (suite *SingleFunctionNameTestSuite) TestToSliceString() {
555+
func (suite *SingleTargetTestSuite) TestToSliceString() {
556556
testCases := []struct {
557557
name string
558558
singleFunctionName string
@@ -579,7 +579,7 @@ func (suite *SingleFunctionNameTestSuite) TestToSliceString() {
579579
}
580580
}
581581

582-
func (suite *SingleFunctionNameTestSuite) TestIsSingleFunctionName() {
582+
func (suite *SingleTargetTestSuite) TestIsSingleFunctionName() {
583583
testCases := []struct {
584584
name string
585585
singleFunctionName *SingleTarget
@@ -600,16 +600,15 @@ func (suite *SingleFunctionNameTestSuite) TestIsSingleFunctionName() {
600600

601601
// TestSingleFunctionNameTestSuite runs the test suite
602602
func TestSingleFunctionNameTestSuite(t *testing.T) {
603-
suite.Run(t, new(SingleFunctionNameTestSuite))
603+
suite.Run(t, new(SingleTargetTestSuite))
604604
}
605605

606-
// --- CanaryFunctionNamesTestSuite ---
607-
type CanaryFunctionNamesTestSuite struct {
606+
// --- CanaryTargetTestSuite ---
607+
type CanaryTargetTestSuite struct {
608608
suite.Suite
609609
}
610610

611-
func (suite *CanaryFunctionNamesTestSuite) TestContains() {
612-
suite.T().Parallel()
611+
func (suite *CanaryTargetTestSuite) TestContains() {
613612
testCases := []struct {
614613
name string
615614
functionName string
@@ -636,15 +635,14 @@ func (suite *CanaryFunctionNamesTestSuite) TestContains() {
636635

637636
for _, testCase := range testCases {
638637
suite.Run(testCase.name, func() {
639-
testCanaryFunctionNames := CanaryTarget{[2]string{"test-function1", "test-function2"}}
638+
testCanaryFunctionNames := &CanaryTarget{[2]string{"test-function1", "test-function2"}}
640639
result := testCanaryFunctionNames.Contains(testCase.functionName)
641640
suite.Equal(testCase.expectedResult, result)
642641
})
643642
}
644643
}
645644

646-
func (suite *CanaryFunctionNamesTestSuite) TestRemoveFunctionName() {
647-
suite.T().Parallel()
645+
func (suite *CanaryTargetTestSuite) TestRemoveFunctionName() {
648646
testCases := []struct {
649647
name string
650648
functionName string
@@ -663,16 +661,15 @@ func (suite *CanaryFunctionNamesTestSuite) TestRemoveFunctionName() {
663661

664662
for _, testCase := range testCases {
665663
suite.Run(testCase.name, func() {
666-
testCanaryFunctionNames := CanaryTarget{[2]string{"test-function1", "test-function2"}}
664+
testCanaryFunctionNames := &CanaryTarget{[2]string{"test-function1", "test-function2"}}
667665
result, err := testCanaryFunctionNames.Delete(testCase.functionName)
668666
suite.Require().NoError(err)
669667
suite.Require().Equal(testCase.expectedResult, result)
670668
})
671669
}
672670
}
673671

674-
func (suite *CanaryFunctionNamesTestSuite) TestAddFunctionName() {
675-
suite.T().Parallel()
672+
func (suite *CanaryTargetTestSuite) TestAddFunctionName() {
676673
testCases := []struct {
677674
name string
678675
functionName string
@@ -695,7 +692,7 @@ func (suite *CanaryFunctionNamesTestSuite) TestAddFunctionName() {
695692

696693
for _, testCase := range testCases {
697694
suite.Run(testCase.name, func() {
698-
testCanaryFunctionNames := CanaryTarget{[2]string{"test-function1", "test-function2"}}
695+
testCanaryFunctionNames := &CanaryTarget{[2]string{"test-function1", "test-function2"}}
699696
result, err := testCanaryFunctionNames.Add(testCase.functionName)
700697
if testCase.expectError {
701698
suite.Require().Error(err)
@@ -709,8 +706,7 @@ func (suite *CanaryFunctionNamesTestSuite) TestAddFunctionName() {
709706
}
710707
}
711708

712-
func (suite *CanaryFunctionNamesTestSuite) TestToSliceString() {
713-
suite.T().Parallel()
709+
func (suite *CanaryTargetTestSuite) TestToSliceString() {
714710
testCases := []struct {
715711
name string
716712
canaryTarget [2]string
@@ -729,15 +725,14 @@ func (suite *CanaryFunctionNamesTestSuite) TestToSliceString() {
729725

730726
for _, testCase := range testCases {
731727
suite.Run(testCase.name, func() {
732-
testCanaryFunctionNames := CanaryTarget{testCase.canaryTarget}
728+
testCanaryFunctionNames := &CanaryTarget{testCase.canaryTarget}
733729
result := testCanaryFunctionNames.ToSliceString()
734730
suite.Equal(testCase.expectedResult, result)
735731
})
736732
}
737733
}
738734

739-
func (suite *CanaryFunctionNamesTestSuite) TestIsSingleFunctionName() {
740-
suite.T().Parallel()
735+
func (suite *CanaryTargetTestSuite) TestIsSingleFunctionName() {
741736
testCases := []struct {
742737
name string
743738
}{
@@ -748,7 +743,7 @@ func (suite *CanaryFunctionNamesTestSuite) TestIsSingleFunctionName() {
748743

749744
for _, testCase := range testCases {
750745
suite.Run(testCase.name, func() {
751-
testCanaryFunctionNames := CanaryTarget{[2]string{"test-function1", "test-function2"}}
746+
testCanaryFunctionNames := &CanaryTarget{[2]string{"test-function1", "test-function2"}}
752747
result := testCanaryFunctionNames.IsSingle()
753748
suite.Require().False(result)
754749
})
@@ -757,5 +752,5 @@ func (suite *CanaryFunctionNamesTestSuite) TestIsSingleFunctionName() {
757752

758753
// TestCanaryFunctionNamesTestSuite runs the test suite
759754
func TestCanaryFunctionNamesTestSuite(t *testing.T) {
760-
suite.Run(t, new(CanaryFunctionNamesTestSuite))
755+
suite.Run(t, new(CanaryTargetTestSuite))
761756
}

0 commit comments

Comments
 (0)