Replace 'pass:' test output with checkmark icon

t-135·WorkTask·
·
·
·Omni/Bild.hs
Created3 months ago·Updated3 months ago

Description

Edit

Problem

When running tests with bild --test Biz/**/*.hs, verbose "pass:" lines are printed for each test. The user wants only the checkmark icon (✓) that's already used for successful builds, not the separate "pass:" label.

Current Behavior

In Omni/Bild.hs lines 1142-1143 and 1152-1153:

onSuccess = Log.pass ["test", nschunk namespace] >> Log.br

This calls Log.pass which outputs "pass: test: Namespace" (from Omni/Log.hs:77).

Desired Behavior

Test success should update the LineManager status to Success (showing ✓) instead of printing a separate "pass:" line. This keeps output consistent with build success.

Implementation Plan

1. In the test function (Omni/Bild.hs), change onSuccess to call LogC.updateLineState namespace LogC.Success instead of Log.pass 2. Remove the Log.br calls since LineManager handles newlines 3. For loud mode, keep Log.pass behavior for compatibility

Files to Modify

  • Omni/Bild.hs - test function (lines ~1135-1159)

Verification

Run bild --test Omni/Test.hs and verify:

  • No "pass:" lines appear
  • Successful tests show [✓] icon
  • Failed tests still show [x] icon

Timeline (0)

No activity yet.