Skip to content

Commit a1c1685

Browse files
jhkim-piiclaude
andcommitted
Clean up test suite per 3-way code review findings
- Remove temp files: network_gcd_traversal.log, util_report_redirect.log - Delete 19 comment-only C++ test stubs (dead code for removed APIs) - Remove redundant graph_make_verify test (covered by graph_advanced) - Centralize assert_file_nonempty/assert_file_contains into test/helpers.tcl and remove inline copies from 17 verilog test files - Fix Build.sh stray quotes in heredoc output - Fix regression.sh unquoted variable expansions - Update .gitignore: add */test/*.log and Testing/ All 6087 tests pass. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
1 parent 29cea73 commit a1c1685

29 files changed

Lines changed: 25 additions & 2189 deletions

.gitignore

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,3 +30,7 @@ doc/messages.txt
3030

3131
# clangd turds
3232
.cache/
33+
34+
# test artifacts
35+
*/test/*.log
36+
Testing/

etc/Build.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@ elif [[ "$OSTYPE" == "darwin"* ]]; then
1616
numThreads=$(sysctl -n hw.ncpu)
1717
else
1818
cat << EOF
19-
WARNING: Unsupported OSTYPE: cannot determine number of host CPUs"
20-
Defaulting to 2 threads. Use -threads=N to use N threads"
19+
WARNING: Unsupported OSTYPE: cannot determine number of host CPUs
20+
Defaulting to 2 threads. Use -threads=N to use N threads
2121
EOF
2222
numThreads=2
2323
fi

graph/test/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ sta_module_tests("graph"
55
delay_corners
66
delete_modify
77
incremental
8-
make_verify
98
modify
109
operations
1110
timing_edges

graph/test/graph_make_verify.ok

Lines changed: 0 additions & 1 deletion
This file was deleted.

graph/test/graph_make_verify.tcl

Lines changed: 0 additions & 10 deletions
This file was deleted.

0 commit comments

Comments
 (0)