<div dir="ltr"><div dir="ltr">Hi Luciano,<br></div><div> thank you for the fix. It is integrated now and I added a couple of tests to SequenceableCollectionTest, one when k is cero and another for the normal case.</div><div><br></div><div> When I wrote the test for the normal case I notice that the collection passed as parameter to the block it is always the name and therefore the following test fails:</div><div>testCombinationsAtATimeDoWorksAsExpected<br>        <br>      | combinations |<br>       <br>     combinations := OrderedCollection new.<br>        'abc' combinations: 2 atATimeDo: [ :combination | combinations add: combination].<br>     <br>      self assert: 3 equals: combinations size.<br>     self assert: (combinations includes: #($a $b)).<br>       self assert: (combinations includes: #($a $c)).<br>       self assert: (combinations includes: #($b $c)).<br></div><div><br></div><div>To make it pass I had to make a copy of combination. </div><div>...<br>'abc' combinations: 2 atATimeDo: [ :combination | combinations add: combination <b>copy</b>].<br>....</div><div><br></div><div>Is that the expected behavior? It looks weird to me... I would expect combination to be different collections on each iteration...</div><div><br></div><div>Cheers!</div><div>Hernan.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 23, 2019 at 1:13 PM Luciano Notarfrancesco via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">The method was failing for the corner case of "combinations of 0 elements". Here's the fix.<br></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></span></strong></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></span></strong></span><font face="tahoma, sans-serif" size="2">-4893-2057</font></div><div><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" target="_blank">http://www.10Pines.com</a></span></font></span></span></span></strong></span></div><div><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div></div></div></div></div></div>