Skip to content

Commit

Permalink
Fix tests to match pmichaud's comment, not to mention the behavior of…
Browse files Browse the repository at this point in the history
… every major compiler. Also fudge for niecza.
  • Loading branch information
colomon committed Jan 3, 2012
1 parent 06f52cf commit 48814de
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions S32-list/sort.t
Expand Up @@ -176,6 +176,7 @@ plan 33;

}

#?niecza todo "Niecza's sort is not stable"
{
is (<P e r l 6>.sort: { 0; }).join, 'Perl6',
'sort with arity 0 closure is stable';
Expand All @@ -195,13 +196,12 @@ plan 33;
}
}

# .sort shouldn't work on non-Positionals ... unless it should?
## XXX pmichaud, 2008-07-01: .sort should work on non-list values
#?rakudo skip 'test errors, adverbial block'
{
#?pugs 2 todo 'bug'
dies_ok { 42.sort }, "method form of sort should not work on numbers";
dies_ok { "str".sort },"method form of sort should not work on strings";
is ~42.sort, "42", "method form of sort should work on numbers";
is ~"str".sort, "str", "method form of sort should work on strings";
is ~(42,).sort, "42", "method form of sort should work on parcels";
}

Expand Down Expand Up @@ -231,6 +231,7 @@ plan 33;
my @sorted;

#?rakudo todo 'nom regression'
#?niecza todo 'Is this test actually testing for correct behavior?'
lives_ok { @sorted = (RT71258_1.new, RT71258_1.new).sort },
'sorting by stringified class instance (name and memory address)';

Expand Down

0 comments on commit 48814de

Please sign in to comment.