<div dir="auto">Hi Chris,<div dir="auto">I did not mean we need a longer joined example.</div><div dir="auto">I did mean that the test omitted to check joined length, so an implementation returning 'a|b|c|' would pass...</div><div dir="auto">Also each of abc should be found in joined rather than each of joined found in abc, otherwise 'a|a|a' would pass... I fail to see how you changed that in next version, but it's late, so i have an excuse ;)</div><div dir="auto">Also, for the bag case, i would expect to find the 5 z in joined,  and i do not see it in the new version of the test.</div><div dir="auto">IMO, the unordered case is going to need a separate method, because it already takes too much place relatively to more usual sequenceable case.</div><div dir="auto">The difficulty to write these tests, and relative length of sequenceable/unordered cases speak by themself: it perfectly illustrate non obvioussness of the feature, that's why i asked.</div><div dir="auto"><span style="font-family:sans-serif">So yes, we can easily implement the feature, but should we? IMO, we ain't gonna need it, or so rarely...</span><br></div><div dir="auto">To make more sense out of the test, we could select joined at odd indices, sorted, and check that it equals original collection sorted (because it's  like testing sequenceable shuffled join:)...</div><div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mar. 4 juin 2019 à 23:55, Chris Muller <<a href="mailto:asqueaker@gmail.com">asqueaker@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Okay.  Fixed in CollectionsTests-cmm.313.mcz.<br>
<br>
On Tue, Jun 4, 2019 at 4:13 PM Nicolas Cellier<br>
<<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank" rel="noreferrer">nicolas.cellier.aka.nice@gmail.com</a>> wrote:<br>
><br>
> Note that joined='a|a|a' would pass the test. A longer joined too.<br>
><br>
> Le mar. 4 juin 2019 à 22:32, <<a href="mailto:commits@source.squeak.org" target="_blank" rel="noreferrer">commits@source.squeak.org</a>> a écrit :<br>
>><br>
>> Chris Muller uploaded a new version of CollectionsTests to project The Inbox:<br>
>> <a href="http://source.squeak.org/inbox/CollectionsTests-cmm.312.mcz" rel="noreferrer noreferrer" target="_blank">http://source.squeak.org/inbox/CollectionsTests-cmm.312.mcz</a><br>
>><br>
>> ==================== Summary ====================<br>
>><br>
>> Name: CollectionsTests-cmm.312<br>
>> Author: cmm<br>
>> Time: 4 June 2019, 3:32:07.434178 pm<br>
>> UUID: b63a7b5c-085b-4479-bc8d-8910d2afeeaf<br>
>> Ancestors: CollectionsTests-mt.311<br>
>><br>
>> Additional test for #joinSeparatedBy: describes the expected behavior for unordered collections.<br>
>><br>
>> =============== Diff against CollectionsTests-mt.311 ===============<br>
>><br>
>> Item was added:<br>
>> + ----- Method: CollectionTest>>testJoin (in category 'tests') -----<br>
>> + testJoin<br>
>> +       | joined |<br>
>> +       self assert: #(a b c d e) join = 'abcde'.<br>
>> +       self assert: (#(a b c) joinSeparatedBy: '|') = 'a|b|c'.<br>
>> +       joined := (#(a b c) asSet) joinSeparatedBy: '|'.<br>
>> +       self assert: (#(2 4) allSatisfy: [ : index | (joined at: index) = $| ]).<br>
>> +       self assert: (#(1 3 5) allSatisfy: [ : index | 'abc' includes: (joined at: index) ])!<br>
>><br>
>> Item was removed:<br>
>> - ----- Method: SequenceableCollectionTest>>testJoin (in category 'tests - converting') -----<br>
>> - testJoin<br>
>> -<br>
>> -       self assert: #(a b c d e) join = 'abcde'.<br>
>> -       self assert: (#(a b c) joinSeparatedBy: '|') = 'a|b|c'.!<br>
>><br>
>><br>
><br>
<br>
</blockquote></div>