Skip to content

Commit 86a5640

Browse files
committed
fix(test): add explicit logging for binary test suite execution
Add clear separator and status logging to show which binary is being tested and whether it exists. This resolves confusion in CI logs when multiple binaries are tested sequentially. Output now shows: ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Testing: [Binary Name] Path: [path] ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Binary exists: true/false ✓ Binary found and ready for testing (if exists) This makes it immediately clear in CI logs: - When SEA tests are being attempted after smol - Whether each binary was found in cache - Which tests are actually running vs skipping
1 parent dfe5be7 commit 86a5640

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

packages/cli/test/integration/binary/binary-test-suite.test.mts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,8 +131,16 @@ function runBinaryTestSuite(binaryType: keyof typeof BINARIES) {
131131
let binaryExists = false
132132

133133
beforeAll(async () => {
134+
// Log which binary we're testing.
135+
logger.log('')
136+
logger.log(`━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`)
137+
logger.log(`Testing: ${binary.name}`)
138+
logger.log(`Path: ${binary.path}`)
139+
logger.log(`━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`)
140+
134141
// Check if binary exists.
135142
binaryExists = existsSync(binary.path)
143+
logger.log(`Binary exists: ${binaryExists}`)
136144

137145
if (!binaryExists) {
138146
logger.log('')
@@ -192,6 +200,11 @@ function runBinaryTestSuite(binaryType: keyof typeof BINARIES) {
192200

193201
logger.log(`Binary built successfully: ${binary.path}`)
194202
logger.log('')
203+
} else {
204+
// Binary already exists.
205+
logger.log('')
206+
logger.log(`✓ Binary found and ready for testing`)
207+
logger.log('')
195208
}
196209

197210
// Check authentication.

0 commit comments

Comments
 (0)