Skip to content

Commit 5d10eb4

Browse files
committed
Revert "test: add --verbose to all e2e test runs to prove full summary output is unchanged"
This reverts the test-only changes from commit 86019bc, removing --verbose from e2e test commands so they exercise the compact summary path. The --details to --verbose code rename is preserved.
1 parent 86019bc commit 5d10eb4

File tree

42 files changed

+135
-900
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+135
-900
lines changed

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
[[e2e]]
44
name = "associate existing cache"
55
steps = [
6-
"vp run --verbose script1 # cache miss",
7-
"vp run --verbose script2 # cache hit, same command as script1",
6+
"vp run script1 # cache miss",
7+
"vp run script2 # cache hit, same command as script1",
88
"json-edit package.json '_.scripts.script2 = \"print world\"' # change script2",
9-
"vp run --verbose script2 # cache miss",
9+
"vp run script2 # cache miss",
1010
]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots/associate existing cache.snap

Lines changed: 4 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -2,56 +2,17 @@
22
source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
33
expression: e2e_outputs
44
---
5-
> vp run --verbose script1 # cache miss
5+
> vp run script1 # cache miss
66
$ print hello
77
hello
8-
9-
10-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
11-
Vite+ Task RunnerExecution Summary
12-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
13-
14-
Statistics: 1 tasks0 cache hits1 cache misses
15-
Performance: 0% cache hit rate
16-
17-
Task Details:
18-
────────────────────────────────────────────────
19-
[1] script1: $ print hello
20-
Cache miss: no previous cache entry found
21-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
22-
> vp run --verbose script2 # cache hit, same command as script1
8+
> vp run script2 # cache hit, same command as script1
239
$ print hellocache hit, replaying
2410
hello
2511

26-
27-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
28-
Vite+ Task RunnerExecution Summary
29-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
30-
31-
Statistics: 1 tasks1 cache hits0 cache misses
32-
Performance: 100% cache hit rate, <duration> saved in total
33-
34-
Task Details:
3512
────────────────────────────────────────────────
36-
[1] script2: $ print hello ✓
37-
→ Cache hit - output replayed - <duration> saved
38-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
13+
[vp run] cache hit, <duration> saved.
3914
> json-edit package.json '_.scripts.script2 = "print world"' # change script2
4015

41-
> vp run --verbose script2 # cache miss
16+
> vp run script2 # cache miss
4217
$ print world ✗ cache miss: args changed, executing
4318
world
44-
45-
46-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
47-
Vite+ Task Runner • Execution Summary
48-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
49-
50-
Statistics: 1 tasks • 0 cache hits • 1 cache misses
51-
Performance: 0% cache hit rate
52-
53-
Task Details:
54-
────────────────────────────────────────────────
55-
[1] script2: $ print world ✓
56-
→ Cache miss: args changed
57-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/snapshots.toml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
[[e2e]]
44
name = "builtin different cwd"
55
steps = [
6-
"cd folder1 && vp run --verbose lint # cache miss in folder1",
7-
"cd folder2 && vp run --verbose lint # cache miss in folder2",
6+
"cd folder1 && vp run lint # cache miss in folder1",
7+
"cd folder2 && vp run lint # cache miss in folder2",
88
"echo 'console.log(1);' > folder2/a.js # modify folder2",
9-
"cd folder1 && vp run --verbose lint # cache hit",
10-
"cd folder2 && vp run --verbose lint # cache miss",
9+
"cd folder1 && vp run lint # cache hit",
10+
"cd folder2 && vp run lint # cache miss",
1111
]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/snapshots/builtin different cwd.snap

Lines changed: 6 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
33
expression: e2e_outputs
44
---
5-
> cd folder1 && vp run --verbose lint # cache miss in folder1
5+
> cd folder1 && vp run lint # cache miss in folder1
66
$ vp lint
77

88
! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -21,21 +21,7 @@ $ vp lint
2121

2222
Found 2 warnings and 0 errors.
2323
Finished in <duration> on 2 files with 90 rules using <n> threads.
24-
25-
26-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
27-
Vite+ Task Runner • Execution Summary
28-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
29-
30-
Statistics: 1 tasks • 0 cache hits • 1 cache misses
31-
Performance: 0% cache hit rate
32-
33-
Task Details:
34-
────────────────────────────────────────────────
35-
[1] lint: $ vp lint ✓
36-
→ Cache miss: no previous cache entry found
37-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
38-
> cd folder2 && vp run --verbose lint # cache miss in folder2
24+
> cd folder2 && vp run lint # cache miss in folder2
3925
$ vp lint ✓ cache hit, replaying
4026

4127
! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -55,22 +41,11 @@ $ vp lint ✓ cache hit, replaying
5541
Found 2 warnings and 0 errors.
5642
Finished in <duration> on 2 files with 90 rules using <n> threads.
5743

58-
59-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
60-
Vite+ Task Runner • Execution Summary
61-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
62-
63-
Statistics: 1 tasks • 1 cache hits • 0 cache misses
64-
Performance: 100% cache hit rate, <duration> saved in total
65-
66-
Task Details:
6744
────────────────────────────────────────────────
68-
[1] lint: $ vp lint ✓
69-
→ Cache hit - output replayed - <duration> saved
70-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
45+
[vp run] cache hit, <duration> saved.
7146
> echo 'console.log(1);' > folder2/a.js # modify folder2
7247

73-
> cd folder1 && vp run --verbose lint # cache hit
48+
> cd folder1 && vp run lint # cache hit
7449
$ vp lint ✗ cache miss: content of input 'folder2/a.js' changed, executing
7550

7651
! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -82,21 +57,7 @@ $ vp lint ✗ cache miss: content of input 'folder2/a.js' changed, executing
8257

8358
Found 1 warning and 0 errors.
8459
Finished in <duration> on 2 files with 90 rules using <n> threads.
85-
86-
87-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
88-
Vite+ Task Runner • Execution Summary
89-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
90-
91-
Statistics: 1 tasks • 0 cache hits • 1 cache misses
92-
Performance: 0% cache hit rate
93-
94-
Task Details:
95-
────────────────────────────────────────────────
96-
[1] lint: $ vp lint ✓
97-
→ Cache miss: content of input 'folder2/a.js' changed
98-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
99-
> cd folder2 && vp run --verbose lint # cache miss
60+
> cd folder2 && vp run lint # cache miss
10061
$ vp lint ✓ cache hit, replaying
10162

10263
! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -109,16 +70,5 @@ $ vp lint ✓ cache hit, replaying
10970
Found 1 warning and 0 errors.
11071
Finished in <duration> on 2 files with 90 rules using <n> threads.
11172

112-
113-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
114-
Vite+ Task Runner • Execution Summary
115-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
116-
117-
Statistics: 1 tasks • 1 cache hits • 0 cache misses
118-
Performance: 100% cache hit rate, <duration> saved in total
119-
120-
Task Details:
12173
────────────────────────────────────────────────
122-
[1] lint: $ vp lint ✓
123-
→ Cache hit - output replayed - <duration> saved
124-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
74+
[vp run] cache hit, <duration> saved.
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[[e2e]]
22
name = "builtin command with non-zero exit does not show cache not updated"
33
steps = [
4-
"vp run --verbose lint -- -D no-debugger",
5-
"vp run --verbose lint -- -D no-debugger",
4+
"vp run lint -- -D no-debugger",
5+
"vp run lint -- -D no-debugger",
66
]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-non-zero-exit/snapshots/builtin command with non-zero exit does not show cache not updated.snap

Lines changed: 2 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
33
expression: e2e_outputs
44
---
5-
[1]> vp run --verbose lint -- -D no-debugger
5+
[1]> vp run lint -- -D no-debugger
66
$ vp lint -D no-debugger
77

88
x eslint(no-debugger): `debugger` statement is not allowed
@@ -14,21 +14,7 @@ $ vp lint -D no-debugger
1414
1515
Found 0 warnings and 1 error.
1616
Finished in <duration> on 1 file with 90 rules using <n> threads.
17-
18-
19-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
20-
Vite+ Task Runner • Execution Summary
21-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
22-
23-
Statistics: 1 tasks • 0 cache hits • 1 cache misses • 1 failed
24-
Performance: 0% cache hit rate
25-
26-
Task Details:
27-
────────────────────────────────────────────────
28-
[1] builtin-non-zero-exit-test#lint: $ vp lint -D no-debugger ✗ (exit code: 1)
29-
→ Cache miss: no previous cache entry found
30-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
31-
[1]> vp run --verbose lint -- -D no-debugger
17+
[1]> vp run lint -- -D no-debugger
3218
$ vp lint -D no-debugger
3319
3420
x eslint(no-debugger): `debugger` statement is not allowed
@@ -40,17 +26,3 @@ $ vp lint -D no-debugger
4026
4127
Found 0 warnings and 1 error.
4228
Finished in <duration> on 1 file with 90 rules using <n> threads.
43-
44-
45-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
46-
Vite+ Task Runner • Execution Summary
47-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
48-
49-
Statistics: 1 tasks • 0 cache hits • 1 cache misses • 1 failed
50-
Performance: 0% cache hit rate
51-
52-
Task Details:
53-
────────────────────────────────────────────────
54-
[1] builtin-non-zero-exit-test#lint: $ vp lint -D no-debugger ✗ (exit code: 1)
55-
→ Cache miss: no previous cache entry found
56-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled/snapshots.toml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@
33
[[e2e]]
44
name = "task with cache disabled"
55
steps = [
6-
"vp run --verbose no-cache-task # cache miss",
7-
"vp run --verbose no-cache-task # cache disabled, runs again",
6+
"vp run no-cache-task # cache miss",
7+
"vp run no-cache-task # cache disabled, runs again",
88
]
99

1010
[[e2e]]
1111
name = "task with cache enabled"
1212
steps = [
13-
"vp run --verbose cached-task # cache miss",
14-
"vp run --verbose cached-task # cache hit",
13+
"vp run cached-task # cache miss",
14+
"vp run cached-task # cache hit",
1515
]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled/snapshots/task with cache disabled.snap

Lines changed: 2 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -2,37 +2,9 @@
22
source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
33
expression: e2e_outputs
44
---
5-
> vp run --verbose no-cache-task # cache miss
5+
> vp run no-cache-task # cache miss
66
$ print-file test.txtcache disabled: no cache config
77
test content
8-
9-
10-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
11-
Vite+ Task RunnerExecution Summary
12-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
13-
14-
Statistics: 1 tasks0 cache hits0 cache misses1 cache disabled
15-
Performance: 0% cache hit rate
16-
17-
Task Details:
18-
────────────────────────────────────────────────
19-
[1] cache-disabled-test#no-cache-task: $ print-file test.txt
20-
Cache disabled in task configuration
21-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
22-
> vp run --verbose no-cache-task # cache disabled, runs again
8+
> vp run no-cache-task # cache disabled, runs again
239
$ print-file test.txtcache disabled: no cache config
2410
test content
25-
26-
27-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
28-
Vite+ Task RunnerExecution Summary
29-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
30-
31-
Statistics: 1 tasks0 cache hits0 cache misses1 cache disabled
32-
Performance: 0% cache hit rate
33-
34-
Task Details:
35-
────────────────────────────────────────────────
36-
[1] cache-disabled-test#no-cache-task: $ print-file test.txt
37-
Cache disabled in task configuration
38-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled/snapshots/task with cache enabled.snap

Lines changed: 3 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -2,37 +2,12 @@
22
source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
33
expression: e2e_outputs
44
---
5-
> vp run --verbose cached-task # cache miss
5+
> vp run cached-task # cache miss
66
$ print-file test.txt
77
test content
8-
9-
10-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
11-
Vite+ Task RunnerExecution Summary
12-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
13-
14-
Statistics: 1 tasks0 cache hits1 cache misses
15-
Performance: 0% cache hit rate
16-
17-
Task Details:
18-
────────────────────────────────────────────────
19-
[1] cache-disabled-test#cached-task: $ print-file test.txt
20-
Cache miss: no previous cache entry found
21-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
22-
> vp run --verbose cached-task # cache hit
8+
> vp run cached-task # cache hit
239
$ print-file test.txtcache hit, replaying
2410
test content
2511

26-
27-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
28-
Vite+ Task RunnerExecution Summary
29-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
30-
31-
Statistics: 1 tasks1 cache hits0 cache misses
32-
Performance: 100% cache hit rate, <duration> saved in total
33-
34-
Task Details:
3512
────────────────────────────────────────────────
36-
[1] cache-disabled-test#cached-task: $ print-file test.txt ✓
37-
→ Cache hit - output replayed - <duration> saved
38-
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
13+
[vp run] cache hit, <duration> saved.

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-command-change/snapshots.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
[[e2e]]
44
name = "cache miss command change"
55
steps = [
6-
"vp run --verbose task # cache miss",
6+
"vp run task # cache miss",
77
"json-edit package.json '_.scripts.task = \"print baz && print bar\"' # change first subtask",
8-
"vp run --verbose task # first: cache miss, second: cache hit",
8+
"vp run task # first: cache miss, second: cache hit",
99
"json-edit package.json '_.scripts.task = \"print bar\"' # remove first subtask",
10-
"vp run --verbose task # cache hit",
10+
"vp run task # cache hit",
1111
]

0 commit comments

Comments
 (0)