Merge pull request #18 from elyezer/master

Do not allow dots on test names

ddt.py

@@ -1,6 +1,7 @@
 import inspect
 import json
 import os
+import re
 from functools import wraps
 
 __version__ = '0.7.1'
@@ -67,12 +68,13 @@ def mk_test_name(name, value):
 
     """
     try:
-        return "{0}_{1}".format(name, value)
+        test_name = "{0}_{1}".format(name, value)
     except UnicodeEncodeError:
         # fallback for python2
-        return "{0}_{1}".format(
+        test_name = "{0}_{1}".format(
             name, value.encode('ascii', 'backslashreplace')
         )
+    return re.sub('\W|^(?=\d)', '_', test_name)
 
 
 def ddt(cls):

test/test_functional.py

@@ -18,6 +18,18 @@ def test_something(self, value):
         return value
 
 
+@ddt
+class DummyInvalidIdentifier():
+    """
+    Dummy class to test the data decorator receiving values invalid characters
+    indentifiers
+    """
+
+    @data('32v2 g #Gmw845h$W b53wi.')
+    def test_data_with_invalid_identifier(self, value):
+        return value
+
+
 @ddt
 class FileDataDummy(object):
     """
@@ -212,9 +224,9 @@ def test_hello(self, val):
                 pass
 
         assert_is_not_none(getattr(mytest, 'test_hello_ascii'))
-        assert_is_not_none(getattr(mytest, 'test_hello_non-ascii-\\u2603'))
+        assert_is_not_none(getattr(mytest, 'test_hello_non_ascii__u2603'))
         assert_is_not_none(
-            getattr(mytest, """test_hello_{u'\\u2603': 'data'}"""))
+            getattr(mytest, """test_hello__u__u2603____data__"""))
 
     elif six.PY3:
 
@@ -225,6 +237,20 @@ def test_hello(self, val):
                 pass
 
         assert_is_not_none(getattr(mytest, 'test_hello_ascii'))
-        assert_is_not_none(getattr(mytest, 'test_hello_non-ascii-\N{SNOWMAN}'))
+        assert_is_not_none(getattr(mytest, 'test_hello_non_ascii__'))
         assert_is_not_none(
-            getattr(mytest, """test_hello_{'\N{SNOWMAN}': 'data'}"""))
+            getattr(mytest, """test_hello________data__"""))
+
+
+def test_feed_data_with_invalid_identifier():
+    """
+    Test that data is fed to the decorated tests
+    """
+    tests = list(filter(is_test, DummyInvalidIdentifier.__dict__))
+    assert_equal(len(tests), 1)
+
+    obj = DummyInvalidIdentifier()
+    method = getattr(obj, tests[0])
+    assert_equal(method.__name__,
+                 'test_data_with_invalid_identifier_32v2_g__Gmw845h_W_b53wi_')
+    assert_equal(method(), '32v2 g #Gmw845h$W b53wi.')