uutils/coreutils

Pattern Findings

671 matches across 11 categories. Click a row to expand file-level details.

Over-Commented Block621 hits · 618 pts

Severity	File	Line	Snippet
LOW	Cargo.toml	41	## Optional feature for stdbuf
LOW	util/dwr.sh	1	#!/usr/bin/env bash
LOW	util/update-version.sh	1	#!/bin/sh
LOW	util/build-run-test-coverage-linux.sh	1	#!/usr/bin/env bash
LOW	util/android-commands.sh	1	#!/usr/bin/env bash
LOW	util/android-commands.sh	201	# finishes a command run with a summary. It returns with the exit code of the probe if specified as
LOW	util/build-gnu.sh	341	# don't need to replicate that behavior.
LOW	util/GHA-delete-GNU-workflow-logs.sh	1	#!/bin/sh
LOW	tests/test_uudoc.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_env.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_env.rs	1101	///
LOW	tests/by-util/test_uptime.rs	101
LOW	tests/by-util/test_mv.rs	1881
LOW	tests/by-util/test_tee.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_ls.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_ls.rs	1161	#[test]
LOW	tests/by-util/test_ls.rs	1221	/// This test does not really test anything provided by `-l` but the file names and symlinks.
LOW	tests/by-util/test_ls.rs	1241
LOW	tests/by-util/test_ls.rs	1261	// The contents of each tuple are the expected colors and names for the link and target.
LOW	tests/by-util/test_stdbuf.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_cp.rs	2801	let dst_path: &str = &[MOUNTPOINT, USERDIR, "dst"].concat();
LOW	tests/by-util/test_tail.rs	341	.stderr_is("tail: error reading 'standard input': Is a directory\n");
LOW	tests/by-util/test_tail.rs	461
LOW	tests/by-util/test_tail.rs	2421	not(target_os = "windows"),
LOW	tests/by-util/test_tail.rs	4521
LOW	tests/by-util/test_tail.rs	4541	// file data
LOW	tests/by-util/test_tail.rs	4601	.stdout_only(expected_stdout);
LOW	tests/by-util/test_touch.rs	641	assert!(at.file_exists(file));
LOW	tests/by-util/test_sleep.rs	281	.fails()
LOW	tests/by-util/test_eintr_handling.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_mkdir.rs	321	// posix_acl entries are in the form of
LOW	tests/by-util/test_cat.rs	821	child.pipe_in("content");
LOW	tests/by-util/test_id.rs	421	// for now, we print clap's standard response for "conflicts_with" instead of:
LOW	tests/by-util/test_split.rs	781	}
LOW	tests/by-util/test_dd.rs	641	//
LOW	tests/by-util/test_dd.rs	861	.args(&["conv=ebcdic,lcase"])
LOW	tests/by-util/test_cksum.rs	21	#[case::crc("crc")]
LOW	tests/by-util/test_chown.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_chown.rs	261	.fails()
LOW	tests/by-util/test_chown.rs	501	.stderr_contains("invalid spec");
LOW	tests/by-util/test_tac.rs	161	// FIXME: See https://github.com/uutils/coreutils/issues/4204
LOW	tests/by-util/test_tac.rs	181	// "xx".
LOW	tests/by-util/test_tac.rs	201	// separator, since there are no more characters preceding it.
LOW	tests/by-util/test_tac.rs	361	new_ucmd!()
LOW	tests/by-util/test_seq.rs	1001	.stdout_only("0\n1\n");
LOW	tests/by-util/test_seq.rs	1021	// 2. GNU seq uses the %Lg format specifier for printing (see the "get_default_format" function
LOW	tests/by-util/test_seq.rs	1041	//
LOW	tests/by-util/test_seq.rs	1081	fn test_equalize_widths_corner_cases() {
LOW	tests/by-util/test_seq.rs	1101	.stdout_is("01.0\n06.2\n");
LOW	tests/by-util/test_mktemp.rs	541	/// Decide whether a string matches a given template.
LOW	tests/by-util/test_more.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_nice.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/by-util/test_head.rs	561	)]
LOW	tests/by-util/test_head.rs	661	#[test]
LOW	tests/uutests/src/lib/random.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/uutests/src/lib/random.rs	21	/// .sample_iter(AlphanumericNewline)
LOW	tests/uutests/src/lib/random.rs	61	/// use crate::common::random::{AlphanumericNewline, RandomizedString};
LOW	tests/uutests/src/lib/random.rs	101	/// * `dist`: A `u8` [`Distribution`]
LOW	tests/uutests/src/lib/util.rs	1	// This file is part of the uutils coreutils package.
LOW	tests/uutests/src/lib/util.rs	301	}
561 more matches not shown…

Hyper-Verbose Identifiers19 hits · 20 pts

Severity	File	Line	Snippet
LOW	util/test_compare_size_results.py	28	def test_load_sizes_date_keyed(self):
LOW	util/test_compare_size_results.py	48	def test_load_sizes_flat_fallback(self):
LOW	util/test_compare_size_results.py	74	def test_compare_threshold_boundaries(self):
LOW	util/test_compare_size_results.py	99	def test_compare_added_removed_and_totals(self):
LOW	util/test_compare_size_results.py	109	def test_compare_sort_and_zero_reference(self):
LOW	util/test_compare_size_results.py	116	def test_format_report_renders_changes(self):
LOW	util/test_compare_size_results.py	137	def test_main_writes_only_when_significant(self):
LOW	util/test_compare_size_results.py	156	def test_main_missing_reference_is_not_fatal(self):
LOW	util/test_compare_test_results.py	53	def test_log_extension_removal(self):
LOW	util/test_compare_test_results.py	93	def test_with_comments_and_blanks(self):
LOW	util/test_compare_test_results.py	213	def test_new_and_removed_tests(self):
LOW	util/test_compare_test_results.py	249	def test_skip_to_fail_not_regression(self):
LOW	util/test_compare_test_results.py	315	def test_main_exit_code_with_real_regressions(self):
LOW	util/test_compare_test_results.py	351	def test_filter_intermittent_fixes(self):
LOW	util/test_compare_test_results.py	425	def test_console_output_formatting(self, mock_stderr, mock_stdout):
LOW	util/test_compare_test_results.py	468	def test_file_output_generation(self):
LOW	util/test_compare_test_results.py	506	def test_exit_code_with_no_regressions(self):
LOW	util/test_compare_test_results.py	523	def test_exit_code_with_regressions(self):
LOW	util/test_compare_test_results.py	547	def test_github_actions_formatting(self):

AI Slop Vocabulary6 hits · 16 pts

Severity	File	Line	Snippet
LOW	util/test_compare_test_results.py	518	# Instead of assertRaises, just call main() and check its return value
MEDIUM	tests/by-util/test_ls.rs	3549	// Essentially the same test as above, but only test symlinks and directories,
MEDIUM	src/uu/md5sum/Cargo.toml	42	# harness = false
MEDIUM	src/uu/checksum_common/Cargo.toml	32	# harness = false
MEDIUM	src/uucore/src/lib/features/buf_copy.rs	6	//! This module provides several buffer-based copy/write functions that leverage
MEDIUM	src/uucore/src/lib/mods/clap_localization.rs	11	//! instead of parsing error strings, providing a more robust solution.

Redundant / Tautological Comments8 hits · 12 pts

Severity	File	Line	Snippet
LOW	util/compare_gnu_result.py	42	# Check if all failing tests are intermittent based on the environment variable
LOW	util/analyze-gnu-results.py	135	# Check if file arguments were provided
LOW	util/analyze-gnu-results.py	145	# Check if the first argument is an output file (starts with -)
LOW	util/run-gnu-test.sh	80	# Check if the file exists with .sh, .pl extension or without any extension in the $path_GNU directory
LOW	util/build-gnu.sh	81	# Check if all patches are already applied
LOW	.github/workflows/l10n.yml	549	# Check if symlinks exist
LOW	.github/workflows/manpage-lint.yml	66	# Check if manpages were generated
LOW	.github/workflows/manpage-lint.yml	89	# Check if mandoc produced any output (errors only, not style warnings)

Excessive Try-Catch Wrapping4 hits · 6 pts

Severity	File	Line	Snippet
LOW	util/remaining-gnu-error.py	22	except Exception as e:
MEDIUM	util/analyze-gnu-results.py	158	print(f"Error: File '{json_files[0]}' not found.", file=sys.stderr)
LOW	util/gnu-json-result.py	37	except Exception as e:
MEDIUM	util/gnu-json-result.py	38	print(f"Error processing file {path}: {e}", file=sys.stderr)

Verbosity Indicators4 hits · 6 pts

Severity	File	Line	Snippet
LOW	src/uu/touch/src/touch.rs	490	// we need to check if the path is the path to a directory (ends with a separator)
LOW	src/uu/dirname/src/dirname.rs	48	// Step 1: Strip trailing slashes (but not if the entire path is slashes)
LOW	src/uu/dirname/src/dirname.rs	58	// Step 2: Check if it ends with `/.` and strip the `/+.` pattern
LOW	src/uu/dirname/src/dirname.rs	80	// Step 3: Normal dirname - find last / and remove everything after it

Self-Referential Comments1 hit · 3 pts

Severity	File	Line	Snippet
MEDIUM	util/test_compare_test_results.py	409	# Create an ignore file

Deep Nesting3 hits · 3 pts

Severity	File	Line
LOW	util/analyze-gnu-results.py	49
LOW	util/compare_test_results.py	44
LOW	util/run-clippy.py	34

Example Usage Blocks2 hits · 3 pts

Severity	File	Line	Snippet
LOW	src/uucore/src/lib/features/update_control.rs	22	//! # Usage example
LOW	src/uucore/src/lib/features/backup_control.rs	35	//! # Usage example

Fake / Example Data2 hits · 2 pts

Severity	File	Line	Snippet
LOW	tests/by-util/test_basenc.rs	289	let input = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. ".repeat(50);
LOW	tests/by-util/test_basenc.rs	289	let input = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. ".repeat(50);

Unused Imports1 hit · 1 pts

Severity	File	Line	Snippet
LOW	util/run-clippy.py	6

Score History

Severity Breakdown

Pattern Findings