Update tests for package elapsed time change.

gotestyourself · Aug 28, 2021 · 5ec6ef6 · 5ec6ef6
1 parent ed34d9a
commit 5ec6ef6
Show file tree

Hide file tree

Showing 5 changed files with 130 additions and 140 deletions.
diff --git a/internal/junitxml/testdata/junitxml-report.golden b/internal/junitxml/testdata/junitxml-report.golden
@@ -1,14 +1,14 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <testsuites>
-	<testsuite tests="0" failures="0" time="0.000000" name="github.com/gotestyourself/gotestyourself/testjson/internal/badmain" timestamp="0001-01-01T00:00:00Z">
+	<testsuite tests="0" failures="0" time="0.010000" name="github.com/gotestyourself/gotestyourself/testjson/internal/badmain" timestamp="0001-01-01T00:00:00Z">
 		<properties>
 			<property name="go.version" value="go7.7.7"></property>
 		</properties>
 		<testcase classname="" name="TestMain" time="0.000000">
 			<failure message="Failed" type="">sometimes main can exit 2&#xA;FAIL&#x9;github.com/gotestyourself/gotestyourself/testjson/internal/badmain&#x9;0.010s&#xA;</failure>
 		</testcase>
 	</testsuite>
-	<testsuite tests="18" failures="0" time="0.020000" name="github.com/gotestyourself/gotestyourself/testjson/internal/good" timestamp="0001-01-01T00:00:00Z">
+	<testsuite tests="18" failures="0" time="0.000000" name="github.com/gotestyourself/gotestyourself/testjson/internal/good" timestamp="0001-01-01T00:00:00Z">
 		<properties>
 			<property name="go.version" value="go7.7.7"></property>
 		</properties>
@@ -35,7 +35,7 @@
 		<testcase classname="github.com/gotestyourself/gotestyourself/testjson/internal/good" name="TestParallelTheSecond" time="0.010000"></testcase>
 		<testcase classname="github.com/gotestyourself/gotestyourself/testjson/internal/good" name="TestParallelTheFirst" time="0.010000"></testcase>
 	</testsuite>
-	<testsuite tests="28" failures="4" time="0.020000" name="github.com/gotestyourself/gotestyourself/testjson/internal/stub" timestamp="0001-01-01T00:00:00Z">
+	<testsuite tests="28" failures="4" time="0.011000" name="github.com/gotestyourself/gotestyourself/testjson/internal/stub" timestamp="0001-01-01T00:00:00Z">
 		<properties>
 			<property name="go.version" value="go7.7.7"></property>
 		</properties>
@@ -80,7 +80,7 @@
 		<testcase classname="github.com/gotestyourself/gotestyourself/testjson/internal/stub" name="TestParallelTheSecond" time="0.010000"></testcase>
 		<testcase classname="github.com/gotestyourself/gotestyourself/testjson/internal/stub" name="TestParallelTheFirst" time="0.010000"></testcase>
 	</testsuite>
-	<testsuite tests="0" failures="0" time="0.000000" name="gotest.tools/gotestsum/internal/empty" timestamp="0001-01-01T00:00:00Z">
+	<testsuite tests="0" failures="0" time="0.004000" name="gotest.tools/gotestsum/internal/empty" timestamp="0001-01-01T00:00:00Z">
 		<properties>
 			<property name="go.version" value="go7.7.7"></property>
 		</properties>

diff --git a/testjson/dotformat_test.go b/testjson/dotformat_test.go
@@ -105,8 +105,8 @@ func TestFmtDotElapsed(t *testing.T) {
 	for _, tc := range testcases {
 		t.Run(tc.expected, func(t *testing.T) {
 			pkg := &Package{
-				cached: tc.cached,
-				Passed: []TestCase{{Elapsed: tc.elapsed}},
+				cached:  tc.cached,
+				elapsed: tc.elapsed,
 			}
 			actual := fmtDotElapsed(pkg)
 			assert.Check(t, cmp.Equal(utf8.RuneCountInString(actual), 7))

diff --git a/testjson/execution_test.go b/testjson/execution_test.go
@@ -12,22 +12,6 @@ import (
 	"gotest.tools/v3/golden"
 )
 
-func TestPackage_Elapsed(t *testing.T) {
-	pkg := &Package{
-		Failed: []TestCase{
-			{Elapsed: 300 * time.Millisecond},
-		},
-		Passed: []TestCase{
-			{Elapsed: 200 * time.Millisecond},
-			{Elapsed: 2500 * time.Millisecond},
-		},
-		Skipped: []TestCase{
-			{Elapsed: 100 * time.Millisecond},
-		},
-	}
-	assert.Equal(t, pkg.Elapsed(), 3100*time.Millisecond)
-}
-
 func TestExecution_Add_PackageCoverage(t *testing.T) {
 	exec := newExecution()
 	exec.add(TestEvent{
@@ -144,7 +128,7 @@ func TestPackage_AddEvent(t *testing.T) {
 		{
 			name:     "package failed",
 			event:    `{"Action":"fail","Package":"gotest.tools/testing","Elapsed":0.012}`,
-			expected: Package{action: ActionFail},
+			expected: Package{action: ActionFail, elapsed: 12 * time.Millisecond},
 		},
 		{
 			name:  "package is cached",
@@ -157,7 +141,7 @@ func TestPackage_AddEvent(t *testing.T) {
 		{
 			name:     "package pass",
 			event:    `{"Action":"pass","Package":"gotest.tools/testing","Elapsed":0.012}`,
-			expected: Package{action: ActionPass},
+			expected: Package{action: ActionPass, elapsed: 12 * time.Millisecond},
 		},
 	}
 

diff --git a/testjson/format_test.go b/testjson/format_test.go
@@ -105,15 +105,18 @@ var expectedExecution = &Execution{
 				{Test: "TestSkipped"},
 				{Test: "TestSkippedWitLog"},
 			},
+			elapsed: 11 * time.Millisecond,
 			action:  ActionFail,
 			running: map[string]TestCase{},
 		},
 		"github.com/gotestyourself/gotestyourself/testjson/internal/badmain": {
 			action:  ActionFail,
 			running: map[string]TestCase{},
+			elapsed: 10 * time.Millisecond,
 		},
 		"gotest.tools/gotestsum/internal/empty": {
-			action: ActionPass,
+			action:  ActionPass,
+			elapsed: 4 * time.Millisecond,
 		},
 	},
 }
@@ -224,6 +227,7 @@ var expectedCoverageExecution = &Execution{
 				{Test: "TestSkipped"},
 				{Test: "TestSkippedWitLog"},
 			},
+			elapsed:  12 * time.Millisecond,
 			action:   ActionPass,
 			coverage: "coverage: 0.0% of statements",
 			running:  map[string]TestCase{},
@@ -240,13 +244,15 @@ var expectedCoverageExecution = &Execution{
 				{Test: "TestSkipped"},
 				{Test: "TestSkippedWitLog"},
 			},
+			elapsed:  11 * time.Millisecond,
 			action:   ActionFail,
 			coverage: "coverage: 0.0% of statements",
 			running:  map[string]TestCase{},
 		},
 		"gotest.tools/gotestsum/testjson/internal/badmain": {
 			action:  ActionFail,
 			running: map[string]TestCase{},
+			elapsed: time.Millisecond,
 		},
 	},
 }