github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixing wrong unit test naming

Browse Source
This commit is contained in:
Guoliang Wang
2018-05-19 08:09:39 +08:00
parent 1e689a8b8c
commit afa2a1cfe5
2 changed files with 37 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
pkg/controller/podautoscaler/legacy_horizontal_test.go
View File

@@ -525,7 +525,7 @@ func (tc *legacyTestCase) runTest(t *testing.T) {
tc.verifyResults(t) tc.verifyResults(t)
} }
func LegacyTestScaleUp(t *testing.T) { func TestLegacyScaleUp(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -540,7 +540,7 @@ func LegacyTestScaleUp(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpUnreadyLessScale(t *testing.T) { func TestLegacyScaleUpUnreadyLessScale(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -557,7 +557,7 @@ func LegacyTestScaleUpUnreadyLessScale(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpUnreadyNoScale(t *testing.T) { func TestLegacyScaleUpUnreadyNoScale(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -574,7 +574,7 @@ func LegacyTestScaleUpUnreadyNoScale(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpDeployment(t *testing.T) { func TestLegacyScaleUpDeployment(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -594,7 +594,7 @@ func LegacyTestScaleUpDeployment(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpReplicaSet(t *testing.T) { func TestLegacyScaleUpReplicaSet(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -614,7 +614,7 @@ func LegacyTestScaleUpReplicaSet(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpCM(t *testing.T) { func TestLegacyScaleUpCM(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -636,7 +636,7 @@ func LegacyTestScaleUpCM(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpCMUnreadyLessScale(t *testing.T) { func TestLegacyScaleUpCMUnreadyLessScale(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -659,7 +659,7 @@ func LegacyTestScaleUpCMUnreadyLessScale(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpCMUnreadyNoScaleWouldScaleDown(t *testing.T) { func TestLegacyScaleUpCMUnreadyNoScaleWouldScaleDown(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -682,7 +682,7 @@ func LegacyTestScaleUpCMUnreadyNoScaleWouldScaleDown(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleDown(t *testing.T) { func TestLegacyScaleDown(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -697,7 +697,7 @@ func LegacyTestScaleDown(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleDownCM(t *testing.T) { func TestLegacyScaleDownCM(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -719,7 +719,7 @@ func LegacyTestScaleDownCM(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleDownIgnoresUnreadyPods(t *testing.T) { func TestLegacyScaleDownIgnoresUnreadyPods(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -841,7 +841,7 @@ func LegacyTestMaxReplicas(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestSuperfluousMetrics(t *testing.T) { func TestLegacySuperfluousMetrics(t *testing.T) {
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
maxReplicas: 6, maxReplicas: 6,
@@ -1023,7 +1023,7 @@ func LegacyTestComputedToleranceAlgImplementation(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleUpRCImmediately(t *testing.T) { func TestLegacyScaleUpRCImmediately(t *testing.T) {
time := metav1.Time{Time: time.Now()} time := metav1.Time{Time: time.Now()}
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,
@@ -1039,7 +1039,7 @@ func LegacyTestScaleUpRCImmediately(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestScaleDownRCImmediately(t *testing.T) { func TestLegacyScaleDownRCImmediately(t *testing.T) {
time := metav1.Time{Time: time.Now()} time := metav1.Time{Time: time.Now()}
tc := legacyTestCase{ tc := legacyTestCase{
minReplicas: 2, minReplicas: 2,

46
pkg/controller/podautoscaler/legacy_replica_calculator_test.go
View File

@@ -227,7 +227,7 @@ func (tc *legacyReplicaCalcTestCase) runTest(t *testing.T) {
} }
} }
func LegacyTestReplicaCalcDisjointResourcesMetrics(t *testing.T) { func TestLegacyReplicaCalcDisjointResourcesMetrics(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 1, currentReplicas: 1,
expectedError: fmt.Errorf("no metrics returned matched known pods"), expectedError: fmt.Errorf("no metrics returned matched known pods"),
@@ -243,7 +243,7 @@ func LegacyTestReplicaCalcDisjointResourcesMetrics(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleUp(t *testing.T) { func TestLegacyReplicaCalcScaleUp(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 5, expectedReplicas: 5,
@@ -260,7 +260,7 @@ func LegacyTestReplicaCalcScaleUp(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleUpUnreadyLessScale(t *testing.T) { func TestLegacyReplicaCalcScaleUpUnreadyLessScale(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 4, expectedReplicas: 4,
@@ -278,7 +278,7 @@ func LegacyTestReplicaCalcScaleUpUnreadyLessScale(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleUpUnreadyNoScale(t *testing.T) { func TestLegacyReplicaCalcScaleUpUnreadyNoScale(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 3, expectedReplicas: 3,
@@ -296,7 +296,7 @@ func LegacyTestReplicaCalcScaleUpUnreadyNoScale(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleUpCM(t *testing.T) { func TestLegacyReplicaCalcScaleUpCM(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 4, expectedReplicas: 4,
@@ -310,7 +310,7 @@ func LegacyTestReplicaCalcScaleUpCM(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleUpCMUnreadyLessScale(t *testing.T) { func TestLegacyReplicaCalcScaleUpCMUnreadyLessScale(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 4, expectedReplicas: 4,
@@ -325,7 +325,7 @@ func LegacyTestReplicaCalcScaleUpCMUnreadyLessScale(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleUpCMUnreadyNoScaleWouldScaleDown(t *testing.T) { func TestLegacyReplicaCalcScaleUpCMUnreadyNoScaleWouldScaleDown(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 3, expectedReplicas: 3,
@@ -340,7 +340,7 @@ func LegacyTestReplicaCalcScaleUpCMUnreadyNoScaleWouldScaleDown(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleDown(t *testing.T) { func TestLegacyReplicaCalcScaleDown(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 5, currentReplicas: 5,
expectedReplicas: 3, expectedReplicas: 3,
@@ -357,7 +357,7 @@ func LegacyTestReplicaCalcScaleDown(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleDownCM(t *testing.T) { func TestLegacyReplicaCalcScaleDownCM(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 5, currentReplicas: 5,
expectedReplicas: 3, expectedReplicas: 3,
@@ -371,7 +371,7 @@ func LegacyTestReplicaCalcScaleDownCM(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcScaleDownIgnoresUnreadyPods(t *testing.T) { func TestLegacyReplicaCalcScaleDownIgnoresUnreadyPods(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 5, currentReplicas: 5,
expectedReplicas: 2, expectedReplicas: 2,
@@ -389,7 +389,7 @@ func LegacyTestReplicaCalcScaleDownIgnoresUnreadyPods(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcTolerance(t *testing.T) { func TestLegacyReplicaCalcTolerance(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 3, expectedReplicas: 3,
@@ -406,7 +406,7 @@ func LegacyTestReplicaCalcTolerance(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcToleranceCM(t *testing.T) { func TestLegacyReplicaCalcToleranceCM(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 3, expectedReplicas: 3,
@@ -420,7 +420,7 @@ func LegacyTestReplicaCalcToleranceCM(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcSuperfluousMetrics(t *testing.T) { func TestLegacyReplicaCalcSuperfluousMetrics(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 4, currentReplicas: 4,
expectedReplicas: 24, expectedReplicas: 24,
@@ -436,7 +436,7 @@ func LegacyTestReplicaCalcSuperfluousMetrics(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetrics(t *testing.T) { func TestLegacyReplicaCalcMissingMetrics(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 4, currentReplicas: 4,
expectedReplicas: 3, expectedReplicas: 3,
@@ -453,7 +453,7 @@ func LegacyTestReplicaCalcMissingMetrics(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcEmptyMetrics(t *testing.T) { func TestLegacyReplicaCalcEmptyMetrics(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 4, currentReplicas: 4,
expectedError: fmt.Errorf("unable to get metrics for resource cpu: no metrics returned from heapster"), expectedError: fmt.Errorf("unable to get metrics for resource cpu: no metrics returned from heapster"),
@@ -468,7 +468,7 @@ func LegacyTestReplicaCalcEmptyMetrics(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcEmptyCPURequest(t *testing.T) { func TestLegacyReplicaCalcEmptyCPURequest(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 1, currentReplicas: 1,
expectedError: fmt.Errorf("missing request for"), expectedError: fmt.Errorf("missing request for"),
@@ -483,7 +483,7 @@ func LegacyTestReplicaCalcEmptyCPURequest(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetricsNoChangeEq(t *testing.T) { func TestLegacyReplicaCalcMissingMetricsNoChangeEq(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 2, currentReplicas: 2,
expectedReplicas: 2, expectedReplicas: 2,
@@ -500,7 +500,7 @@ func LegacyTestReplicaCalcMissingMetricsNoChangeEq(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetricsNoChangeGt(t *testing.T) { func TestLegacyReplicaCalcMissingMetricsNoChangeGt(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 2, currentReplicas: 2,
expectedReplicas: 2, expectedReplicas: 2,
@@ -517,7 +517,7 @@ func LegacyTestReplicaCalcMissingMetricsNoChangeGt(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetricsNoChangeLt(t *testing.T) { func TestLegacyReplicaCalcMissingMetricsNoChangeLt(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 2, currentReplicas: 2,
expectedReplicas: 2, expectedReplicas: 2,
@@ -534,7 +534,7 @@ func LegacyTestReplicaCalcMissingMetricsNoChangeLt(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetricsUnreadyNoChange(t *testing.T) { func TestLegacyReplicaCalcMissingMetricsUnreadyNoChange(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 3, expectedReplicas: 3,
@@ -552,7 +552,7 @@ func LegacyTestReplicaCalcMissingMetricsUnreadyNoChange(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetricsUnreadyScaleUp(t *testing.T) { func TestLegacyReplicaCalcMissingMetricsUnreadyScaleUp(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 3, currentReplicas: 3,
expectedReplicas: 4, expectedReplicas: 4,
@@ -570,7 +570,7 @@ func LegacyTestReplicaCalcMissingMetricsUnreadyScaleUp(t *testing.T) {
tc.runTest(t) tc.runTest(t)
} }
func LegacyTestReplicaCalcMissingMetricsUnreadyScaleDown(t *testing.T) { func TestLegacyReplicaCalcMissingMetricsUnreadyScaleDown(t *testing.T) {
tc := legacyReplicaCalcTestCase{ tc := legacyReplicaCalcTestCase{
currentReplicas: 4, currentReplicas: 4,
expectedReplicas: 3, expectedReplicas: 3,
@@ -591,7 +591,7 @@ func LegacyTestReplicaCalcMissingMetricsUnreadyScaleDown(t *testing.T) {
// TestComputedToleranceAlgImplementation is a regression test which // TestComputedToleranceAlgImplementation is a regression test which
// back-calculates a minimal percentage for downscaling based on a small percentage // back-calculates a minimal percentage for downscaling based on a small percentage
// increase in pod utilization which is calibrated against the tolerance value. // increase in pod utilization which is calibrated against the tolerance value.
func LegacyTestReplicaCalcComputedToleranceAlgImplementation(t *testing.T) { func TestLegacyReplicaCalcComputedToleranceAlgImplementation(t *testing.T) {
startPods := int32(10) startPods := int32(10)
// 150 mCPU per pod. // 150 mCPU per pod.