naoNao89:stty-add-unit-tests - Branch - uutils/coreutils

Added unit tests for stty.rs to improve test coverage

#9094

Comparing

naoNao89:stty-add-unit-tests

(

70c9158

) with

main

(

7c62885

)

Untouched: 127

Skipped: 6

Benchmarks

Skipped (6)

These benchmarks were skipped, so their baseline results are used instead. If they were deleted in your codebase, archive them to remove them from the performance reports.
Learn more about archiving benchmarks

du_balanced_tree[(5, 4, 10)]

src/uu/du/benches/du_bench.rs

Skipped

9.2 ms*

du_all_balanced_tree[(4, 3, 10)]

src/uu/du/benches/du_bench.rs

Skipped

1.6 ms*

factor_multiple_u128s[18446744073709551616]

src/uu/factor/benches/factor_bench.rs

Skipped

327.9 ms*

factor_multiple_big_uint

src/uu/factor/benches/factor_bench.rs

Skipped

18.6 ms*

du_human_balanced_tree[(5, 4, 10)]

src/uu/du/benches/du_bench.rs

Skipped

10.6 ms*

factor_table

src/uu/factor/benches/factor_bench.rs

Skipped

189.5 ms*

Passed

tsort_input_parsing_heavy[5000]

src/uu/tsort/benches/tsort_bench.rs

+1%

83.8 ms83.1 ms

cksum_crc32b

src/uu/cksum/benches/cksum_bench.rs

13.9 ms13.8 ms

cksum_multiple_files

src/uu/cksum/benches/cksum_bench.rs

26 ms25.9 ms

cksum_default

src/uu/cksum/benches/cksum_bench.rs

17.5 ms17.4 ms

cksum_raw_output

src/uu/cksum/benches/cksum_bench.rs

17.5 ms17.5 ms

sort_accented_data[500000]

src/uu/sort/benches/sort_bench.rs

365.1 ms363.6 ms

cksum_crc

src/uu/cksum/benches/cksum_bench.rs

17.5 ms17.5 ms

b64_decode_ignore_garbage_synthetic

src/uu/base64/benches/base64_bench.rs

166.8 µs166.7 µs

sort_german_locale

src/uu/sort/benches/sort_locale_bench.rs

39.3 ms39.3 ms

mv_multiple_to_dir

src/uu/mv/benches/mv_bench.rs

13.1 ms13.1 ms

cp_preserve_metadata[(5, 4, 10)]

src/uu/cp/benches/cp_bench.rs

75.1 ms75.1 ms

cp_recursive_deep_tree[(120, 4)]

src/uu/cp/benches/cp_bench.rs

13.7 ms13.7 ms

cksum_blake2b

src/uu/cksum/benches/cksum_bench.rs

193 ms192.9 ms

du_all_wide_tree[(5000, 500)]

src/uu/du/benches/du_bench.rs

18.2 ms18.2 ms

factor_multiple_u64s[2]

src/uu/factor/benches/factor_bench.rs

213.2 ms213.1 ms

b64_decode_synthetic

src/uu/base64/benches/base64_bench.rs

168.8 µs168.7 µs

shuf_lines[100000]

src/uu/shuf/benches/shuf_bench.rs

26.9 ms26.9 ms

mv_single_file

src/uu/mv/benches/mv_bench.rs

136 ms135.9 ms

tsort_complex_dag[50000]

src/uu/tsort/benches/tsort_bench.rs

84.3 ms84.3 ms

sort_german_c_locale

src/uu/sort/benches/sort_locale_bench.rs

38.5 ms38.5 ms

mv_force_overwrite

src/uu/mv/benches/mv_bench.rs

138.4 ms138.3 ms

rm_single_file

src/uu/rm/benches/rm_bench.rs

119.7 ms119.6 ms

hashsum_md5

src/uu/hashsum/benches/hashsum_bench.rs

32 ms32 ms

cp_recursive_balanced_tree[(5, 4, 10)]

src/uu/cp/benches/cp_bench.rs

72.4 ms72.3 ms

ls_recursive_mixed_tree

src/uu/ls/benches/ls_bench.rs

1.4 ms1.4 ms

tsort_tree_dag[(10, 3)]

src/uu/tsort/benches/tsort_bench.rs

34.2 ms34.2 ms

hashsum_sha512

src/uu/hashsum/benches/hashsum_bench.rs

110.8 ms110.8 ms

shuf_input_range[1000000]

src/uu/shuf/benches/shuf_bench.rs

229.1 ms229.1 ms

cksum_sha2

src/uu/cksum/benches/cksum_bench.rs

1.7 s1.7 s

split_number_chunks

src/uu/split/benches/split_bench.rs

288.1 µs288.1 µs

cksum_sha256

src/uu/cksum/benches/cksum_bench.rs

1.7 s1.7 s

tsort_wide_dag[100000]

src/uu/tsort/benches/tsort_bench.rs

134.1 ms134.1 ms

cksum_sha3

src/uu/cksum/benches/cksum_bench.rs

1.8 s1.8 s

tsort_linear_chain[1000000]

src/uu/tsort/benches/tsort_bench.rs

1.5 s1.5 s

cut_bytes

src/uu/cut/benches/cut_bench.rs

8.5 ms8.5 ms

uniq_heavy_duplicates[10000]

src/uu/uniq/benches/uniq_bench.rs

2 ms2 ms

sort_numeric

src/uu/sort/benches/sort_locale_bench.rs

23.2 ms23.2 ms

mv_directory

src/uu/mv/benches/mv_bench.rs

3.5 ms3.5 ms

numfmt_to_si_precision[10000]

src/uu/numfmt/benches/numfmt_bench.rs

35.4 ms35.4 ms

rm_force_files

src/uu/rm/benches/rm_bench.rs

2 ms2 ms

rm_multiple_files

src/uu/rm/benches/rm_bench.rs

2.3 ms2.3 ms

numfmt_from_si[10000]

src/uu/numfmt/benches/numfmt_bench.rs

32.7 ms32.7 ms

cut_fields_tab

src/uu/cut/benches/cut_bench.rs

34.1 ms34.1 ms

sort_ascii_c_locale

src/uu/sort/benches/sort_locale_bench.rs

21.7 ms21.7 ms

ls_recursive_long_all_balanced_tree[(6, 4, 15)]

src/uu/ls/benches/ls_bench.rs

134.5 ms134.5 ms

numfmt_to_si[10000]

src/uu/numfmt/benches/numfmt_bench.rs

33.7 ms33.7 ms

numfmt_round_modes[("up", 10000)]

src/uu/numfmt/benches/numfmt_bench.rs

34.2 ms34.2 ms

numfmt_round_modes[("towards-zero", 10000)]

src/uu/numfmt/benches/numfmt_bench.rs

34.2 ms34.2 ms

numfmt_round_modes[("down", 10000)]

src/uu/numfmt/benches/numfmt_bench.rs

34.2 ms34.2 ms

hashsum_md5_check

src/uu/hashsum/benches/hashsum_bench.rs

32 ms32 ms

numfmt_padding[(10000, 50)]

src/uu/numfmt/benches/numfmt_bench.rs

48.2 ms48.2 ms

sort_case_insensitive[500000]

src/uu/sort/benches/sort_bench.rs

279.7 ms279.7 ms

sort_mixed_data[500000]

src/uu/sort/benches/sort_bench.rs

328.5 ms328.5 ms

fold_many_lines[100000]

src/uu/fold/benches/fold_bench.rs

80.1 ms80.1 ms

sort_numeric[500000]

src/uu/sort/benches/sort_bench.rs

1.2 s1.2 s

numfmt_to_iec[10000]

src/uu/numfmt/benches/numfmt_bench.rs

33.7 ms33.7 ms

hashsum_sha1

src/uu/hashsum/benches/hashsum_bench.rs

72.5 ms72.5 ms

hashsum_sha256_check

src/uu/hashsum/benches/hashsum_bench.rs

173.6 ms173.6 ms

hashsum_sha256

src/uu/hashsum/benches/hashsum_bench.rs

173.6 ms173.6 ms

rm_recursive_tree

src/uu/rm/benches/rm_bench.rs

13.6 ms13.6 ms

nl_many_lines[100000]

src/uu/nl/benches/nl_bench.rs

79.8 ms79.8 ms

cksum_sysv

src/uu/cksum/benches/cksum_bench.rs

48.3 ms48.3 ms

wc_bytes_lines_synthetic[2000]

src/uu/wc/benches/wc_bench.rs

120.2 ms120.2 ms

cksum_sm3

src/uu/cksum/benches/cksum_bench.rs

1.6 s1.6 s

cksum_shake256

src/uu/cksum/benches/cksum_bench.rs

1.9 s1.9 s

cksum_shake128

src/uu/cksum/benches/cksum_bench.rs

1.5 s1.5 s

nl_large_file[10]

src/uu/nl/benches/nl_bench.rs

100.9 ms100.9 ms

wc_words_synthetic[2000]

src/uu/wc/benches/wc_bench.rs

11.4 s11.4 s

cksum_sha224

src/uu/cksum/benches/cksum_bench.rs

1.7 s1.7 s

cksum_sha384

src/uu/cksum/benches/cksum_bench.rs

1.1 s1.1 s

unexpand_many_lines[100000]

src/uu/unexpand/benches/unexpand_bench.rs

262 ms262 ms

wc_default_large_line_count[100000]

src/uu/wc/benches/wc_bench.rs

50.4 ms50.4 ms

cksum_bsd

src/uu/cksum/benches/cksum_bench.rs

128.3 ms128.3 ms

unexpand_large_file[10]

src/uu/unexpand/benches/unexpand_bench.rs

549.3 ms549.3 ms

wc_words_large_line_count[100000]

src/uu/wc/benches/wc_bench.rs

48 ms48 ms

numfmt_large_numbers_si[10000]

src/uu/numfmt/benches/numfmt_bench.rs

32.4 ms32.4 ms

sort_ascii_utf8_locale

src/uu/sort/benches/sort_locale_bench.rs

43.5 ms43.5 ms

fold_custom_width[50000]

src/uu/fold/benches/fold_bench.rs

32.9 ms32.9 ms

sort_key_field[500000]

src/uu/sort/benches/sort_bench.rs

703.7 ms703.7 ms

cut_fields_custom_delim

src/uu/cut/benches/cut_bench.rs

40.8 ms40.8 ms

expand_many_lines[100000]

src/uu/expand/benches/expand_bench.rs

149.9 ms149.9 ms

sort_reverse_locale[500000]

src/uu/sort/benches/sort_bench.rs

362.7 ms362.7 ms

seq_formatted

src/uu/seq/benches/seq_bench.rs

97.2 ms97.2 ms

expand_custom_tabstops[50000]

src/uu/expand/benches/expand_bench.rs

37 ms37 ms

wc_lines_large_line_count[500000]

src/uu/wc/benches/wc_bench.rs

2.9 ms2.9 ms

seq_custom_separator

src/uu/seq/benches/seq_bench.rs

28.7 ms28.7 ms

sort_long_line[160000]

src/uu/sort/benches/sort_bench.rs

1.7 ms1.7 ms

seq_integers

src/uu/seq/benches/seq_bench.rs

26.1 ms26.1 ms

ls_recursive_long_all_deep_tree[(100, 4)]

src/uu/ls/benches/ls_bench.rs

3.3 ms3.3 ms

wc_chars_large_line_count[100000]

src/uu/wc/benches/wc_bench.rs

1 ms1 ms

sort_dictionary_order[500000]

src/uu/sort/benches/sort_bench.rs

1.1 s1.1 s

ls_recursive_wide_tree[(10000, 1000)]

src/uu/ls/benches/ls_bench.rs

52.6 ms52.6 ms

wc_bytes_synthetic[500]

src/uu/wc/benches/wc_bench.rs

182.4 µs182.4 µs

seq_with_step

src/uu/seq/benches/seq_bench.rs

13.3 ms13.3 ms

cksum_blake3

src/uu/cksum/benches/cksum_bench.rs

207.8 µs207.8 µs

uniq_with_count[10000]

src/uu/uniq/benches/uniq_bench.rs

1.9 ms1.9 ms

cut_characters

src/uu/cut/benches/cut_bench.rs

7.5 ms7.5 ms

sort_ascii_only[500000]

src/uu/sort/benches/sort_bench.rs

354.8 ms354.8 ms

wc_lines_variable_length[(50, 500)]

src/uu/wc/benches/wc_bench.rs

3.4 ms3.4 ms

sort_mixed_c_locale

src/uu/sort/benches/sort_locale_bench.rs

38.4 ms38.4 ms

ls_recursive_long_all_mixed_tree

src/uu/ls/benches/ls_bench.rs

2.9 ms2.9 ms

split_lines

src/uu/split/benches/split_bench.rs

5.7 ms5.7 ms

ls_recursive_long_all_wide_tree[(15000, 1500)]

src/uu/ls/benches/ls_bench.rs

143.4 ms143.4 ms

ls_recursive_deep_tree[(200, 2)]

src/uu/ls/benches/ls_bench.rs

2.2 ms2.2 ms

cksum_sha512

src/uu/cksum/benches/cksum_bench.rs

1.1 s1.1 s

wc_lines_extreme_line_lengths[(100000, 200)]

src/uu/wc/benches/wc_bench.rs

1.6 ms1.6 ms

seq_large_integers

src/uu/seq/benches/seq_bench.rs

2.2 ms2.2 ms

split_bytes

src/uu/split/benches/split_bench.rs

510.5 µs510.5 µs

shuf_repeat_sampling[50000]

src/uu/shuf/benches/shuf_bench.rs

4.7 ms4.7 ms

uniq_case_insensitive[10000]

src/uu/uniq/benches/uniq_bench.rs

1.8 ms1.8 ms

cksum_sha1

src/uu/cksum/benches/cksum_bench.rs

722.9 ms722.9 ms

ls_recursive_balanced_tree[(6, 4, 15)]

src/uu/ls/benches/ls_bench.rs

55.4 ms55.4 ms

cksum_md5

src/uu/cksum/benches/cksum_bench.rs

317.8 ms317.9 ms

du_wide_tree[(5000, 500)]

src/uu/du/benches/du_bench.rs

11.2 ms11.2 ms

split_numeric_suffix

src/uu/split/benches/split_bench.rs

5.9 ms5.9 ms

b64_encode_synthetic

src/uu/base64/benches/base64_bench.rs

164.3 µs164.4 µs

sort_case_sensitive[500000]

src/uu/sort/benches/sort_bench.rs

174.8 ms174.8 ms

sort_mixed_utf8_locale

src/uu/sort/benches/sort_locale_bench.rs

38.9 ms38.9 ms

cp_recursive_wide_tree[(6000, 800)]

src/uu/cp/benches/cp_bench.rs

189.7 ms189.8 ms

sort_unique_mixed

src/uu/sort/benches/sort_locale_bench.rs

39.5 ms39.5 ms

cp_archive_balanced_tree[(5, 4, 10)]

src/uu/cp/benches/cp_bench.rs

77.8 ms77.9 ms

sort_reverse_mixed

src/uu/sort/benches/sort_locale_bench.rs

39.3 ms39.3 ms

du_summarize_balanced_tree[(5, 4, 10)]

src/uu/du/benches/du_bench.rs

8.4 ms8.4 ms

cp_large_file[16]

src/uu/cp/benches/cp_bench.rs

358.5 µs359.6 µs

du_max_depth_balanced_tree[(6, 4, 10)]

src/uu/du/benches/du_bench.rs

32.8 ms32.9 ms

du_deep_tree[(100, 3)]

src/uu/du/benches/du_bench.rs

-1%

1.2 ms1.2 ms

sort_unique_locale[500000]

src/uu/sort/benches/sort_bench.rs

-1%

495 ms498.6 ms

Commits

Click on a commit to change the comparison range

Base

main

7c62885

+0.01%

stty: Add 101 unit tests and 16 integration tests to improve coverage Add comprehensive test coverage for stty: Unit tests (src/uu/stty/src/stty.rs): - Flag struct methods and builder pattern (7 tests) - Control character parsing and formatting (19 tests) - All combination settings expansion (26 tests) - Termios modification functions (13 tests) - String-to-flag/combo/baud parsing (19 tests) - TermiosFlag trait implementations (5 tests) - Helper and utility functions (10 tests) - Trait implementations (2 tests) Integration tests (tests/by-util/test_stty.rs): - Help and version output validation (2 tests) - Invalid argument handling (3 tests) - Control character overflow validation (2 tests) - Grouped flag removal validation (1 test) - File argument error handling (1 test) - Conflicting print modes (1 test) - Additional TTY-dependent tests (6 tests, ignored in CI) Unit test coverage improved from 0% to 43.76% (207/473 lines). Integration tests validate argument parsing and error handling. Addresses #9061

4c9058a

2 months ago

by naoNao89

-31.87%

stty: Add essential unit tests and integration tests to improve coverage - Added 11 essential unit tests for complex internal functions: * Control character parsing (string_to_control_char) * Control character formatting (control_char_to_string) * Combination settings expansion (combo_to_flags) * Terminal size parsing with overflow handling (parse_rows_cols) * Sane control character defaults (get_sane_control_char) - Added 16 integration tests for command behavior: * Help/version output validation * Invalid argument handling * Control character overflow validation * Grouped flag removal validation * File argument error handling * Conflicting print modes * TTY-dependent tests (marked as ignored for CI) Unit tests focus on complex parsing logic that's difficult to test via integration tests. Integration tests validate actual command behavior. Coverage improved from 0% to 43.76% (207/473 lines). Fixes #9061

faf10db

2 months ago

by naoNao89

stty: Add comprehensive unit and integration tests for error handling - Add unit tests for parse_rows_cols() with edge cases and wraparound - Add unit tests for string_to_baud() with platform-specific handling - Add unit tests for string_to_combo() with all combo modes - Add 17 integration tests for missing arguments and invalid inputs - Enhance test_invalid_arg() with better error message assertions - Update coverage script for improved reporting Coverage improved from 22.26% to 23.14% regions.

dc369f8

2 months ago

by naoNao89

stty: Add Debug and PartialEq derives for test assertions - Add #[derive(Debug, PartialEq)] to AllFlags enum - Add PartialEq to Flag struct derives - Enables assert_eq! macro usage in unit tests

7a9cd6b

2 months ago

by naoNao89

stty: Fix formatting and clippy warnings in tests - Replace assert_eq! with assert! for boolean comparisons - Fix line wrapping for long logical expressions - Use inline format string syntax (e.g., {err} instead of {}) - All 25 unit tests pass - No clippy warnings

fde8525

2 months ago

by naoNao89

+31.87%

stty: Add inline spell-checker ignores for test strings - Add spell-checker:ignore comments for test data (notachar, notabaud, susp) - Add spell-checker:ignore comments for French error strings (Valeur, entier, invalide) - Fixes cspell validation without modifying global config

670b14b

2 months ago

by naoNao89

+0.02%

perf: gate PartialEq derive to test builds only The PartialEq derive was being compiled into release builds even though it's only used in test code. This caused a 3.33% performance regression in the du_human_balanced_tree benchmark due to increased binary size affecting CPU cache efficiency. Changes: - stty.rs: Gate PartialEq derive on Flag<T> with #[cfg_attr(test, derive(PartialEq))] - flags.rs: Gate PartialEq derive on AllFlags enum with #[cfg_attr(test, derive(PartialEq))] This eliminates the performance regression while keeping all test code functional and unchanged.

483f247

2 months ago

by naoNao89

-0.01%

Merge branch 'main' into stty-add-unit-tests

70c9158

11 days ago

by sylvestre

Home Terms Privacy Docs