I think it's okay to go ahead and apply this diff, Arthur.

On Thu, May 24, 2018 at 8:18 PM Sudarshan S Chawathe <xxxxxx@eip10.org> wrote:

Minor issue related to tests in the sample implementation of SRFI 113
(and as also included in SRFI 146's implementation).  I'm posting this
note mainly because it may save someone else some time.

The test groups "sets/mismatch" and "bag/mismatch" include several SRFI
64 test-error cases. However, when used for "is an error"
(a.k.a. "anything goes") situations, many of these test-errors (can)
cause test failures in the sample implementation.

My reading of that SRFI 64's test-error is that it should be used only
in situations where the implementation is obligated to detect an error,
not in "is an error" situations.

I encountered this issue while working with the SRFI 146 sample
implementation and Kawa.  I had tested earlier with Chibi which skips
the test-error cases (at least on my setup) so the issue did not arise
then.  (So I should add a caveat that my testing is with the code
included in the SRFI 146 sample implementation, not the original SRFI
113 code.  I also tweaked the implementation a bit to account for Kawa's
restricted call/cc, but I don't think that should matter.)

In any case, I am attaching a diff with the changes I made to SRFI 113's
test.sld file.  (I'm hoping it's small enough to be OK.) Mainly, I put a
few test-expect-fail calls before the problematic test-error cases.  It
also includes some trivial renaming to avoid the duplicate defines I
noted in an earlier message (to the SRFI 146 list).

Regards,

-chaw