codecov
test-results-parser
BlogDocsChangelog

fix: remove ValidatedString and TestrunOrSkipped

#85
Comparing
joey/remove-structs
(
3e176e8
) with
main
(
accbb7f
)
CodSpeed Performance Gauge
0%
Untouched
6

Benchmarks

Passed

read_aggregation
benches/binary.rs::benches::binary::binary
CodSpeed Performance Gauge
+1%
28.7 ms28.4 ms
create_and_serialize
benches/binary.rs::benches::binary::binary
CodSpeed Performance Gauge
0%
298.8 ms299.7 ms
merge_rewrite
benches/binary.rs::benches::binary::binary
CodSpeed Performance Gauge
0%
637.8 ms640.1 ms
merge
benches/binary.rs::benches::binary::binary
CodSpeed Performance Gauge
0%
633.9 ms636.3 ms
update_different
benches/binary.rs::benches::binary::binary
CodSpeed Performance Gauge
-1%
378.8 ms381.6 ms
update_same
benches/binary.rs::benches::binary::binary
CodSpeed Performance Gauge
-1%
389.2 ms392.4 ms

Commits

Click on a commit to change the comparison range
Base
main
accbb7f
-0.31%
fix: remove ValidatedString and TestrunOrSkipped I previously added these because we were having issues storing very long test names and I wanted to "validate" the data that came in to make sure that we're not including random strings for test names that might include source code. The main problem was that we weren't able to index that data in timescale and that was causing issues. I've since removed those indexes and if we have to index on names we'll have to use full text indexes next time, but for now we don't need this anymore. In the future, we'll likely have to add some mechanism for users to "ignore" irrelevant tests in the UI, or filter the names of tests. In any case, the scope of this library is to simply parse a JUnit XML file, so we should stick to that here, and avoid trying to do more.
3e176e8
5 months ago
by joseph-sentry
© 2025 CodSpeed Technology
Home Terms Privacy Docs