Skip to content

Commit caa9a1b

Browse files
committed
enhance(testing): improve goprox --test failure output with detailed file size logging (refs #73)
- Add test/deleted/ to .gitignore to prevent tracking of test deleted files - Enhance --test mode to log changed files with filename and size difference - Replace binary diff output with structured error logging for each changed file - Maintain git diff --stat summary for quick reference
1 parent 1912d47 commit caa9a1b

5 files changed

Lines changed: 199 additions & 170 deletions

File tree

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,3 +20,5 @@ output/
2020
# GitHub backup files
2121
*.tar.gz
2222
.env
23+
24+
test/deleted/

goprox

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2797,13 +2797,23 @@ if [ "$test" = true ]; then
27972797
_process_media
27982798

27992799
_info "Comparing test output..."
2800-
git diff --quiet ./test/ || {
2800+
if ! git diff --quiet ./test/; then
28012801
# changes detected
28022802
_error "Test failed!"
28032803
echo $fg[red]
28042804
git diff --stat ./test/
2805+
# For each changed file, log filename and size difference
2806+
while IFS= read -r file; do
2807+
# Only consider files (not directories)
2808+
if [[ -f "$file" ]]; then
2809+
oldsize=$(git show HEAD:"$file" 2>/dev/null | wc -c | tr -d ' ')
2810+
newsize=$(wc -c < "$file" | tr -d ' ')
2811+
if [[ -z "$oldsize" ]]; then oldsize=0; fi
2812+
_error "Changed: $file (size: $oldsize -> $newsize bytes)"
2813+
fi
2814+
done < <(git diff --name-only ./test/)
28052815
exit 1
2806-
}
2816+
fi
28072817
_echo "TESTING successful!"
28082818
exit 0
28092819
fi

scripts/testing/test-framework.zsh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
set -e
1313

1414
# Test framework configuration
15-
TEST_ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
15+
TEST_ROOT="$(cd "$(dirname "${0:A}")/.." && pwd)"
1616
TEST_DIR="${TEST_ROOT}/test"
1717
TEST_OUTPUT_DIR="${TEST_ROOT}/output/test-results"
1818
TEST_TEMP_DIR="${TEST_ROOT}/output/test-temp"

0 commit comments

Comments
 (0)