two more tests to get me to 100% coverage :D

Author: tehranian

shortener/baseconv.py

@@ -43,8 +43,8 @@ def from_decimal(self, i):
     def to_decimal(self, s):
         if not isinstance(s, basestring):
             raise DecodingError('%s is not a basestring()' % s)
-        for char in s:
-            if char not in self.digits:
+        for index, char in enumerate(s):
+            if char not in self.digits and not char == '-' and not index == 0:
                 raise DecodingError('Invalid character for encoding: %s' % char)
         return int(self.convert(s, self.digits, self.decimal_digits))
 

shortener/tests.py

@@ -204,9 +204,17 @@ def test_create_with_custom_id(self):
         link = Link.objects.create(id=id, url='http://www.python.org')
         self.assertEqual(link.to_base62(), base62.from_decimal(id))
 
+    def test_unicode(self):
+        """
+        unicode test
+        """
+        url = 'http://www.python.org'
+        link = Link.objects.create(url=url)
+        self.assertTrue(url in unicode(link))
+
 
 class BaseconvTestCase(TestCase):
-    def test_symmetry_int(self):
+    def test_symmetry_positive_int(self):
         """
         symmetry for encoding/decoding values
         """
@@ -215,6 +223,15 @@ def test_symmetry_int(self):
             encoded_int = base62.from_decimal(random_int)
             self.assertEqual(random_int, base62.to_decimal(encoded_int))
 
+    def test_symmetry_negative_int(self):
+        """
+        symmetry for negative numbers
+        """
+        for x in xrange(1000):
+            random_int = random.randint(-1 * sys.maxint - 1, 0)
+            encoded_int = base62.from_decimal(random_int)
+            self.assertEqual(random_int, base62.to_decimal(encoded_int))
+
     def test_encoding_non_int_fails(self):
         """
         calling from_decimal() on letters raises an EncodingError