Small implementation improvement in linexprs.py

[linpy.git] / pypol / tests / test_linexprs.py

diff --git a/pypol/tests/test_linexprs.py b/pypol/tests/test_linexprs.py
index c55c842..1966b2a 100644 (file)
--- a/pypol/tests/test_linexprs.py
+++ b/pypol/tests/test_linexprs.py
@@ -117,11 +117,15 @@ class TestExpression(unittest.TestCase):
         self.assertEqual(self.expr * 0, 0)
         self.assertEqual(0 * self.expr, 0)
         self.assertEqual(self.expr * 2, 2*self.x - 4*self.y + 6)
         self.assertEqual(self.expr * 0, 0)
         self.assertEqual(0 * self.expr, 0)
         self.assertEqual(self.expr * 2, 2*self.x - 4*self.y + 6)

+        with self.assertRaises(TypeError):
+            self.x * self.x

 
     def test_truediv(self):
         with self.assertRaises(ZeroDivisionError):
             self.expr / 0
         self.assertEqual(self.expr / 2, self.x / 2 - self.y + Fraction(3, 2))
 
     def test_truediv(self):
         with self.assertRaises(ZeroDivisionError):
             self.expr / 0
         self.assertEqual(self.expr / 2, self.x / 2 - self.y + Fraction(3, 2))

+        with self.assertRaises(TypeError):
+            self.x / self.x

 
     def test_eq(self):
         self.assertEqual(self.expr, self.expr)
 
     def test_eq(self):
         self.assertEqual(self.expr, self.expr)


@@ -271,7 +275,7 @@ class TestRational(unittest.TestCase):
     def setUp(self):
         self.zero = Rational(0)
         self.one = Rational(1)
     def setUp(self):
         self.zero = Rational(0)
         self.one = Rational(1)

-        self.pi = Rational(Fraction(22, 7))
+        self.pi = Rational(22, 7)

 
     def test_new(self):
         self.assertEqual(Rational(), self.zero)
 
     def test_new(self):
         self.assertEqual(Rational(), self.zero)


@@ -279,6 +283,10 @@ class TestRational(unittest.TestCase):
         self.assertEqual(Rational(self.pi), self.pi)
         self.assertEqual(Rational('22/7'), self.pi)
 
         self.assertEqual(Rational(self.pi), self.pi)
         self.assertEqual(Rational('22/7'), self.pi)
 

+    def test_hash(self):
+        self.assertEqual(hash(self.one), hash(1))
+        self.assertEqual(hash(self.pi), hash(Fraction(22, 7)))
+

     def test_isconstant(self):
         self.assertTrue(self.zero.isconstant())
 
     def test_isconstant(self):
         self.assertTrue(self.zero.isconstant())
 


@@ -286,13 +294,6 @@ class TestRational(unittest.TestCase):
         self.assertFalse(self.zero)
         self.assertTrue(self.pi)
 
         self.assertFalse(self.zero)
         self.assertTrue(self.pi)
 

-    def test_fromstring(self):
-        self.assertEqual(Rational.fromstring('22/7'), self.pi)
-        with self.assertRaises(ValueError):
-            Rational.fromstring('a')
-        with self.assertRaises(TypeError):
-            Rational.fromstring(1)
-

     def test_repr(self):
         self.assertEqual(repr(self.zero), '0')
         self.assertEqual(repr(self.one), '1')
     def test_repr(self):
         self.assertEqual(repr(self.zero), '0')
         self.assertEqual(repr(self.one), '1')