jsenv / core

Develop, test and build JavaScript projects
MIT License
31 stars 4 forks source link

Assert diff #398

Closed dmail closed 6 months ago

dmail commented 9 months ago

The goal of this PR is to change the assertion error format

Before

AssertionError: unequal values
--- found ---
true
--- expected ---
false
--- path ---
actual.a

After

image

github-actions[bot] commented 9 months ago

File size impact

Impact on file sizes when merging assert_diff into main.

dist (+0.01%)
Files new size new gzip size
dist/jsenv_core.js 680 kB (+144 B / +0.02%) 151 kB (+51 B / +0.03%) :arrow_upper_right:
Unmodified (112) 357 kB (0 B / +0%) 99.8 kB (0 B / +0%) :ghost:
Total (113) 1 MB (+144 B / +0.01%) 251 kB (+51 B / +0.02%) :arrow_upper_right:

Generated by @jsenv/file-size-impact during file size impact#8814052466 on c2b5848

github-actions[bot] commented 9 months ago

Performance impact

Impact on 37 metrics when merging assert_diff into main. Before drawing conclusion, keep in mind performance variability.

package metrics (-2%)
Metric Before merge After merge Impact
import duration 0.1 second 0.1 second -0.01 second / -10% :arrow_lower_right:
import memory heap used 5.9 MB 5.9 MB +280 B / +0.005% :arrow_upper_right:
import memory heap total 5.9 MB 5.9 MB :ghost:
import fs read operations 0 0 :ghost:
npm tarball size 309 kB 309 kB :ghost:
npm tarball unpacked size 1.4 MB 1.4 MB :ghost:
npm tarball file count 194 194 :ghost:
dev server metrics (-1%)
Metric Before merge After merge Impact
start duration 0.1 second 0.1 second -0.01 second / -11% :arrow_lower_right:
start memory heap used 6.5 MB 6.5 MB -944 B / -0.01% :arrow_lower_right:
start memory heap total 6.4 MB 6.4 MB :ghost:
start fs read 0 0 :ghost:
start fs write 0 0 :ghost:
time to app display 0.6 second 0.6 second +0.002 second / +0.3% :arrow_upper_right:
serve app memory heap total 18.1 MB 18.1 MB :ghost:
serve app memory heap used 16.6 MB 16.6 MB +416 B / +0.003% :arrow_upper_right:
serve app fs read 0 0 :ghost:
serve app fs write 0 0 :ghost:
time to 2nd app display 0.3 second 0.3 second -0.003 second / -1% :arrow_lower_right:
2nd serve memory heap total 3.1 MB 3.1 MB :ghost:
2nd serve memory heap used 911 kB 901 kB -9.5 kB / -1% :arrow_lower_right:
2nd serve fs read 0 0 :ghost:
2nd serve fs write 0 0 :ghost:
build metrics (-0.1%)
Metric Before merge After merge Impact
build duration 0.9 second 1 second +0.004 second / +0.4% :arrow_upper_right:
build memory heap used 31.1 MB 31.1 MB +13.4 kB / +0.04% :arrow_upper_right:
build memory heap total 55.4 MB 54.9 MB -524 kB / -0.9% :arrow_lower_right:
number of fs read operation 0 0 :ghost:
number of fs write operation 48 48 :ghost:
test metrics (+0.6%)
Metric Before merge After merge Impact
test plan duration 4.2 seconds 4.2 seconds -0.01 second / -0.3% :arrow_lower_right:
test plan memory heap used 39.8 MB 39.7 MB -19.1 kB / -0.05% :arrow_lower_right:
test plan memory heap total 72.9 MB 74.8 MB +1.8 MB / +3% :arrow_upper_right:
number of fs read operation 0 0 :ghost:
number of fs write operation 24 24 :ghost:
test metrics (serie) (-0.3%)
Metric Before merge After merge Impact
test plan duration 5.4 seconds 5.3 seconds -0.05 second / -0.8% :arrow_lower_right:
test plan memory heap used 39.8 MB 39.8 MB +13.7 kB / +0.03% :arrow_upper_right:
test plan memory heap total 75 MB 74.8 MB -262 kB / -0.3% :arrow_lower_right:
number of fs read operation 0 0 :ghost:
number of fs write operation 1608 1608 :ghost:

Generated by @jsenv/performance-impact during performance_impact#8814052467 on c2b5848