Changeset 22028

Timestamp:

08/24/08 12:51:19 (3 months ago)

Author:

bacek

Message:

[spec] Str.rindex: Replace comparition to -1 with checking defined() according to spec

Files:

• t/spec/S29-str/rindex.t (modified) (4 diffs)

t/spec/S29-str/rindex.t

@@ -11 +11 @@
 is(rindex("Hello World", "l"), 9, "One char, in the middle");
 is(rindex("Hello World", "d"), 10, "One char, in the end");
-is(rindex("Hello World", "x"), -1, "One char, no match");
+ok(!defined(rindex("Hello World", "x")), "One char, no match");
 
 is(rindex("Hello World", "l", 10), 9, "One char, first match, pos @ end");
@@ -17 +17 @@
 is(rindex("Hello World", "l", 8), 3, "- 2. match");
 is(rindex("Hello World", "l", 2), 2, "- 3. match");
-is(rindex("Hello World", "l", 1), -1, "- no more matches");
+ok(!defined(rindex("Hello World", "l", 1)), "- no more matches");
 
 # Simple - with a string
@@ -24 +24 @@
 is(rindex("Hello World", "o W"), 4, "Substr, in the middle");
 is(rindex("Hello World", "World"), 6, "Substr, at the end");
-is(rindex("Hello World", "low"), -1, "Substr, no match");
+ok(!defined(rindex("Hello World", "low")), "Substr, no match");
 is(rindex("Hello World", "Hello World"), 0, "Substr eq Str");
 
@@ -31 +31 @@
 is(rindex("Hello World", ""), 11, "Substr is empty");
 is(rindex("", ""), 0, "Both strings are empty");
-is(rindex("", "Hello"), -1, "Only main-string is empty");
+ok(!defined(rindex("", "Hello")), "Only main-string is empty");
 is(rindex("Hello", "", 3), 3, "Substr is empty, pos within str");
 is(rindex("Hello", "", 5), 5, "Substr is empty, pos at end of str");