diff --git a/hack/.golint_failures b/hack/.golint_failures index bae76b4e63..682576a48a 100644 --- a/hack/.golint_failures +++ b/hack/.golint_failures @@ -687,7 +687,6 @@ test/e2e/framework/providers/aws test/e2e/framework/providers/azure test/e2e/framework/providers/gce test/e2e/framework/providers/kubemark -test/e2e/framework/timer test/e2e/instrumentation test/e2e/instrumentation/logging test/e2e/instrumentation/monitoring diff --git a/test/e2e/framework/timer/timer.go b/test/e2e/framework/timer/timer.go index 37769cb344..9620ae021c 100644 --- a/test/e2e/framework/timer/timer.go +++ b/test/e2e/framework/timer/timer.go @@ -29,7 +29,7 @@ import ( var now = time.Now -// Represents a phase of a test. Phases can overlap. +// Phase represents a phase of a test. Phases can overlap. type Phase struct { sequenceNumber int name string @@ -63,9 +63,8 @@ func (phase *Phase) duration() time.Duration { func (phase *Phase) humanReadable() string { if phase.ended() { return fmt.Sprintf("Phase %s: %v\n", phase.label(), phase.duration()) - } else { - return fmt.Sprintf("Phase %s: %v so far\n", phase.label(), phase.duration()) } + return fmt.Sprintf("Phase %s: %v so far\n", phase.label(), phase.duration()) } // A TestPhaseTimer groups phases and provides a way to export their measurements as JSON or human-readable text. @@ -93,10 +92,12 @@ func (timer *TestPhaseTimer) StartPhase(sequenceNumber int, phaseName string) *P return newPhase } +// SummaryKind returns the summary of test summary. func (timer *TestPhaseTimer) SummaryKind() string { return "TestPhaseTimer" } +// PrintHumanReadable returns durations of all phases. func (timer *TestPhaseTimer) PrintHumanReadable() string { buf := bytes.Buffer{} timer.lock.Lock() @@ -107,6 +108,7 @@ func (timer *TestPhaseTimer) PrintHumanReadable() string { return buf.String() } +// PrintJSON returns durations of all phases with JSON format. func (timer *TestPhaseTimer) PrintJSON() string { data := perftype.PerfData{ Version: "v1",