TclTLS: View Ticket

2024-05-28
00:40		• Closed ticket [64cdb76212]: tests/all.tcl always succeeds plus 3 other changes artifact: e9be2539c8 user: bohagan
2023-11-20
02:45		• Ticket [64cdb76212]: 5 changes artifact: 3e38d50e19 user: doofus
02:43		• New ticket [64cdb76212]. artifact: 6fa1ec034e user: doofus

Ticket Hash:	64cdb7621260855edf46e074d3520d282d3d8cfd
Title:	tests/all.tcl always succeeds
Status:	Closed	Type:	Code Defect
Severity:	Important	Priority:	Immediate
Subsystem:		Resolution:	Open
Last Modified:	2024-05-28 00:40:10 1.55 years ago	Created:	2023-11-20 02:43:51 2.07 years ago
Version Found In:	1.7.22

User Comments:

doofus added on 2023-11-20 02:43:51:

Even if all tests were to fail, the script all.tcl would still exit with code 0...

That breaks automation, because the caller does not know of any test-failures.

The patch below addresses this problem:

--- tests/all.tcl       2020-10-12 16:39:22.000000000 -0400
+++ tests/all.tcl       2023-11-19 21:19:34.128221000 -0500
@@ -55,5 +55,5 @@
 # cleanup
 puts stdout "\nTests ended at [eval $timeCmd]"
+set failCount [llength $::tcltest::failFiles]
 ::tcltest::cleanupTests 1
-return
-
+exit [expr $failCount > 0]

bohagan added on 2024-05-28 00:40:10:

This is resolved by the changes in [af2c6346c92d5d04].