🌐 US-Proxy
class="logged-out env-production page-responsive" style="word-wrap: break-word;" >
Skip to content

fix: correctly report the number of remaining tests for io tests - #6263

Merged
pchiusano merged 1 commit into
unisonweb:trunkfrom
etorreborre:etorreborre/fix/io-remaining-tests
Aug 7, 2026
Merged

fix: correctly report the number of remaining tests for io tests#6263
pchiusano merged 1 commit into
unisonweb:trunkfrom
etorreborre:etorreborre/fix/io-remaining-tests

Conversation

@etorreborre

Copy link
Copy Markdown
Contributor

Overview

This test fixes the reporting of io.test.all where the "number of tests left" increases after each execution instead of decreasing.

Implementation approach and notes

I have changed a foldrM to a foldlM in order to execute the tests in order, and zip with 0.. to get correct indices.

Test coverage

I have added a unit test. I would have liked to introduce a transcripts but this this only shows as transient messages, I don't think that it can (yet) be expressed in a transcript.

I otherwise tested manually:

 ✅  1/10 tests left to run, current test: test6 6
    New test results:

    1. test1    ◉ Passed
    2. test10   ◉ Passed
    3. test2    ◉ Passed
    4. test3    ◉ Passed
    5. test4    ◉ Passed
    6. test6    ◉ Passed
    7. test7    ◉ Passed
    8. test8    ◉ Passed
    9. test9    ◉ Passed

    10. test5   ✗
                elements not equal
                ("ok5", "xxx")


  🚫 1 test(s) failing, ✅ 9 test(s) passing

And I could see the number of tests remaining to execute decrease (note that I added the total number of tests to execute).

@etorreborre
etorreborre force-pushed the etorreborre/fix/io-remaining-tests branch 2 times, most recently from 04993e1 to 77317f8 Compare August 6, 2026 13:54
Signed-off-by: Eric Torreborre <etorreborre@yahoo.com>
@etorreborre
etorreborre force-pushed the etorreborre/fix/io-remaining-tests branch from 77317f8 to b3b6d71 Compare August 6, 2026 14:00

@pchiusano pchiusano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@pchiusano
pchiusano added this pull request to the merge queue Aug 7, 2026
Merged via the queue into unisonweb:trunk with commit db60ce2 Aug 7, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants