Migrate fixnum tests to unittest. Fix int32 rollover bug.

pkg/fixnum/test/int_64_vm_test.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // A test to compare the results of the fixnum library with the Dart VM
 
 library int64vmtest;
 
-import "package:expect/expect.dart";
 import 'dart:math' as math;

[justinfagnani, 2013/07/18 19:13:48]

this test is skipped, right? should it just be deleted, or can we make it fast
enough to re-enable

[Chris Bracken, 2013/07/18 19:38:08]

Better to replace it with a well-defined set of deterministic tests. Think it
makes more sense to test against expected intx behaviour than blast it with a a
massive set of tests against int behaviour that may or may not test everything
we need to.

Migrated a couple useful tests over to the int32/int64 tests; deleting this
file.

-
-part 'package:fixnum/src/int32.dart';
-part 'package:fixnum/src/int64.dart';
-part 'package:fixnum/src/intx.dart';
-
-final random = new math.Random();
+import 'package:fixnum/fixnum.dart';
+import "package:unittest/unittest.dart";
 
 void main() {
   int64VMTest test = new int64VMTest();
   test.doTestBinary(new BinaryOp("&", (a, b) => a & b));
   test.doTestBinary(new BinaryOp("|", (a, b) => a | b));
   test.doTestBinary(new BinaryOp("^", (a, b) => a ^ b));
   test.doTestBinary(new BinaryOp("+", (a, b) => a + b));
   test.doTestBinary(new BinaryOp("-", (a, b) => a - b));
   test.doTestBinary(new BinaryOp("*", (a, b) => a * b));
   test.doTestUnary(new UnaryOp("-", (a) => -a));
   test.doTestUnary(new UnaryOp("~", (a) => ~a));
   test.doTestShift(new ShiftOp("<<", (a, n) => a << (n & 63)));
   test.doTestShift(new ShiftOp(">>", (a, n) => a >> (n & 63)));
-  test.doTestBoolean(new BooleanOp("compareTo", (a, b) => a.compareTo(b)));
+  test.doTestComparison(new ComparisonOp("compareTo", (a, b) => a.compareTo(b)));
   test.doTestBoolean(new BooleanOp("==", (a, b) => a == b));
   test.doTestBoolean(new BooleanOp("!=", (a, b) => a != b));
   test.doTestBoolean(new BooleanOp("<", (a, b) => a < b));
   test.doTestBoolean(new BooleanOp("<=", (a, b) => a <= b));
   test.doTestBoolean(new BooleanOp(">", (a, b) => a > b));
   test.doTestBoolean(new BooleanOp(">=", (a, b) => a >= b));
   test.doTestBinary(new BinaryOp("%", (a, b) => a % b));
   test.doTestBinary(new BinaryOp("~/", (a, b) => a ~/ b));
   test.doTestBinary(new BinaryOp("remainder", (a, b) => a.remainder(b)));
 }
 
+final random = new math.Random();
 const int DISCARD = 0;
 
 int64 _randomInt64() {
   int i = 0;
   for (int b = 0; b < 64; b++) {
       double rand = random.nextDouble();
       for (int j = 0; j < DISCARD; j++) {
         rand = random.nextDouble();
       }
       i = (i << 1) | ((rand > 0.5) ? 1 : 0);
@@ skipping 42 matching lines @@
   bool ref(int val0, int val1) => op(val0, val1);
   bool test(int64 val0, int64 val1) => op(val0, val1);
 }
 
 class ShiftOp extends Op {
   ShiftOp(String name, Function op) : super(name, op);
   int ref(int val0, int shift) => trunc64(op(val0, shift));
   int64 test(int64 val0, int shift) => op(val0, shift);
 }
 
+class ComparisonOp extends Op {
+  ComparisonOp(String name, Function op) : super(name, op);
+  int ref(int val0, int val1) => trunc64(op(val0, val1));
+  int test(int64 val0, int64 val1) => op(val0, val1);
+}
+
 class int64VMTest {
   static const int BASE_VALUES = 32;
   static const int RANDOM_TESTS = 32;
   List<int64> TEST_VALUES;
 
   int64VMTest() {
     Set<int64> testSet = new Set<int64>();
-    for (int i = 0; i < BASE_VALUES; i++) {
+    for (int i = 0; i < 64; i++) {
       testSet.add(new int64.fromInt(i));
       testSet.add(new int64.fromInt(-i));
 
       testSet.add(int64.MIN_VALUE + i);
       testSet.add(int64.MAX_VALUE - i);
 
-      testSet.add(new int64.fromInt(i << int64._BITS ~/ 2));
-      testSet.add(new int64.fromInt(i << int64._BITS));
-      testSet.add(new int64.fromInt(i << (3 * int64._BITS) ~/ 2));
-      testSet.add(new int64.fromInt(i << 2 * int64._BITS));
-      testSet.add(new int64.fromInt(i << (5 * int64._BITS) ~/ 2));
+      testSet.add(new int64.fromInt(0xf << i));
     }
 
     int64 one = new int64.fromInt(1);
     int64 three = new int64.fromInt(3);
     int64 ones = int64.parseHex("1111111111111111");
     int64 tens = int64.parseHex("1010101010101010");
     int64 oh_ones = int64.parseHex("0101010101010101");
     int64 digits = int64.parseHex("123456789ABCDEFF");
     for (int i = 0; i < 16; i++) {
       testSet.add(ones * i);
@@ skipping 49 matching lines @@
       TEST_VALUES[index++] = val;
     }
 
     print("VALUES.length = $index");
   }
 
   void _doTestUnary(UnaryOp op, int64 val) {
     int ref = op.ref(val.toInt());
     int64 result64 = op.test(val);
     int result = result64.toInt();
-    if (ref != result) {
-      Expect.fail("${op.name}: val = $val");
-    }
+    expect(ref == result, true, reason: "${op.name}: val = $val");
   }
 
   void doTestUnary(UnaryOp op) {
     print("Testing operator ${op.name}");
     for (int i = 0; i < TEST_VALUES.length; i++) {
       _doTestUnary(op, TEST_VALUES[i]);
     }
     for (int i = 0; i < RANDOM_TESTS; i++) {
       int64 randomLong = _randomInt64();
       _doTestUnary(op, randomLong);
     }
   }
 
   void _doTestBinary(BinaryOp op, int64 val0, int64 val1) {
-    // print("Test val0 = $val0, val1 = $val1");
     var refException = null;
     int ref = -1;
     try {
       ref = op.ref(val0.toInt(), val1.toInt());
     } on Exception catch (e) {
       refException = e;
     }
     var testException = null;
     int result = -2;
     int64 result64;
     try {
-      int64 val0_save = new int64._copy(val0);
-      int64 val1_save = new int64._copy(val1);
+      int val0_save = val0.toInt();
+      int val1_save = val1.toInt();
       result64 = op.test(val0, val1);
       result = result64.toInt();
-      if (val0 != val0_save) {
-        print(
-            "Test altered first argument val0 = $val0, val0_save = $val0_save");
+      if (val0.toInt() != val0_save) {
+        fail("Test altered val0 = $val0, val0_save = $val0_save");
       }
-      if (val1 != val1_save) {
-        print("Test altered second argument");
+      if (val1.toInt() != val1_save) {
+        fail("Test altered val1 = $val1, val1_save = $val1_save");
       }
     } on Exception catch (e) {
       testException = e;
     }
     if (testException is IntegerDivisionByZeroException &&
         refException is IntegerDivisionByZeroException) {
     } else if (testException != null || refException != null) {
-      Expect.fail("${op.name}: val0 = $val0, val1 = $val1, "
+      fail("${op.name}: val0 = $val0, val1 = $val1, "
           "testException = $testException, refException = $refException");
       return;
     } else if (ref != result) {
       if ("%" == op.name && ref < 0) {
-        // print("Dart VM bug: ${op.name}: val0 = $val0, val1 = $val1, "
-        //    "ref = $ref, result64 = $result64, result = $result");
+        fail("Dart VM bug: ${op.name}: val0 = $val0, val1 = $val1, "
+            "ref = $ref, result64 = $result64, result = $result");
       } else {
-        Expect.fail("${op.name}: val0 = $val0, val1 = $val1, "
-            "ref = $ref, result64 = $result64, result = $result");
+        fail("${op.name}: val0 = $val0, val1 = $val1, "
+             "ref = $ref, result64 = $result64, result = $result");
       }
     }
   }
 
   void doTestBinary(BinaryOp op) {
     print("Testing operator ${op.name}");
     for (int i = 0; i < TEST_VALUES.length; i++) {
       int64 randomLong = _randomInt64();
       _doTestBinary(op, TEST_VALUES[i], randomLong);
       _doTestBinary(op, randomLong, TEST_VALUES[i]);
@@ skipping 11 matching lines @@
           longVal1 = -longVal0;
         }
       }
       _doTestBinary(op, longVal0, longVal1);
     }
   }
 
   void _doTestBoolean(BooleanOp op, int64 val0, int64 val1) {
     bool ref = op.ref(val0.toInt(), val1.toInt());
     bool result = op.test(val0, val1);
-    if (ref != result) {
-      Expect.fail("${op.name}: val0 = $val0, val1 = $val1");
-    }
+    expect(ref == result, true,
+        reason: "${op.name}: val0 = $val0, val1 = $val1");
   }
 
   void doTestBoolean(BooleanOp op) {
     print("Testing operator ${op.name}");
     for (int i = 0; i < TEST_VALUES.length; i++) {
       int64 randomLong = _randomInt64();
       _doTestBoolean(op, TEST_VALUES[i], randomLong);
       _doTestBoolean(op, randomLong, TEST_VALUES[i]);
       for (int j = 0; j < TEST_VALUES.length; j++) {
         _doTestBoolean(op, TEST_VALUES[i], TEST_VALUES[j]);
       }
     }
     for (int i = 0; i < RANDOM_TESTS; i++) {
       int64 longVal0 = _randomInt64();
       int64 longVal1 = _randomInt64();
       if (_randomInt(20) == 0) {
         if (_randomInt(2) == 0) {
           longVal1 = longVal0;
         } else {
           longVal1 = -longVal0;
         }
       }
       _doTestBoolean(op, longVal0, longVal1);
     }
   }
 
+  void _doTestComparison(ComparisonOp op, int64 val0, int64 val1) {
+    int ref = op.ref(val0.toInt(), val1.toInt());
+    int result = op.test(val0, val1);
+    expect(ref == result, true,
+        reason: "${op.name}: val0 = $val0, val1 = $val1");
+  }
+
+  void doTestComparison(ComparisonOp op) {
+    print("Testing operator ${op.name}");
+    for (int i = 0; i < TEST_VALUES.length; i++) {
+      int64 randomLong = _randomInt64();
+      _doTestComparison(op, TEST_VALUES[i], randomLong);
+      _doTestComparison(op, randomLong, TEST_VALUES[i]);
+      for (int j = 0; j < TEST_VALUES.length; j++) {
+        _doTestComparison(op, TEST_VALUES[i], TEST_VALUES[j]);
+      }
+    }
+    for (int i = 0; i < RANDOM_TESTS; i++) {
+      int64 longVal0 = _randomInt64();
+      int64 longVal1 = _randomInt64();
+      if (_randomInt(20) == 0) {
+        if (_randomInt(2) == 0) {
+          longVal1 = longVal0;
+        } else {
+          longVal1 = -longVal0;
+        }
+      }
+      _doTestComparison(op, longVal0, longVal1);
+    }
+  }
+
   void _doTestShift(ShiftOp op, int64 val, int shift) {
     int ref = op.ref(val.toInt(), shift);
     int64 result64 = op.test(val, shift);
     int result = result64.toInt();
-    if (ref != result) {
-      Expect.fail("${op.name}: val = $val, shift = $shift");
-    }
+    expect(ref == result, true,
+        reason: "${op.name}: val = $val, shift = $shift");
   }
 
   void doTestShift(ShiftOp op) {
     print("Testing operator ${op.name}");
     for (int i = 0; i < TEST_VALUES.length; i++) {
       for (int shift = -64; shift <= 64; shift++) {
         _doTestShift(op, TEST_VALUES[i], shift);
       }
     }
     for (int i = 0; i < RANDOM_TESTS; i++) {
       int64 randomLong = _randomInt64();
       for (int shift = -64; shift <= 64; shift++) {
         _doTestShift(op, randomLong, shift);
       }
     }
   }
 }