Reviewers: rossberg,

Description:
Simplify array iterator tests

[email protected]
BUG=

Please review this at https://codereview.chromium.org/344223006/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+3, -6 lines):
  M test/mjsunit/harmony/array-iterator.js


Index: test/mjsunit/harmony/array-iterator.js
diff --git a/test/mjsunit/harmony/array-iterator.js b/test/mjsunit/harmony/array-iterator.js index 23288761a6afed49700c973638b02cf6d9f02978..a64dfa911a4388a0df3cbdc9234c76d9de57144f 100644
--- a/test/mjsunit/harmony/array-iterator.js
+++ b/test/mjsunit/harmony/array-iterator.js
@@ -175,10 +175,9 @@ function TestForArrayValues() {

   assertEquals(8, buffer.length);

-  for (var i = 0; i < buffer.length - 1; i++) {
+  for (var i = 0; i < buffer.length; i++) {
     assertSame(array[i], buffer[i]);
   }
-  assertTrue(isNaN(buffer[buffer.length - 1]));
 }
 TestForArrayValues();

@@ -210,10 +209,9 @@ function TestForArrayEntries() {

   assertEquals(8, buffer.length);

-  for (var i = 0; i < buffer.length - 1; i++) {
+  for (var i = 0; i < buffer.length; i++) {
     assertSame(array[i], buffer[i][1]);
   }
-  assertTrue(isNaN(buffer[buffer.length - 1][1]));

   for (var i = 0; i < buffer.length; i++) {
     assertEquals(i, buffer[i][0]);
@@ -232,10 +230,9 @@ function TestForArray() {

   assertEquals(8, buffer.length);

-  for (var i = 0; i < buffer.length - 1; i++) {
+  for (var i = 0; i < buffer.length; i++) {
     assertSame(array[i], buffer[i]);
   }
-  assertTrue(isNaN(buffer[buffer.length - 1]));
 }
 TestForArrayValues();



--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to