-
Notifications
You must be signed in to change notification settings - Fork 14
Conversation
Get the latest update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete pycache/vanity.cpython-35.pyc
@@ -7,48 +7,28 @@ class testNormalize(unittest.TestCase): | |||
""" | |||
A test class for the normalize method. | |||
""" | |||
def testFakePackage(self): | |||
|
|||
def test_FakePackage(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_fake_package
|
||
def testDjango(self): | ||
def test_Django(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_django
normalized = vanity.normalize("dJaNgO") | ||
self.assertEqual(normalized, "Django") | ||
|
||
def testNone(self): | ||
def test_None(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_none
normalized = vanity.normalize(None) | ||
self.assertEqual(normalized, "none") | ||
|
||
def testFlask(self): | ||
def test_Flask(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_flask
normalized = vanity.normalize("fLaSk") | ||
self.assertEqual(normalized, "Flask") | ||
|
||
def testSpaceString(self): | ||
def test_SpaceString(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_space_string
normalized = vanity.normalize(None) | ||
self.assertEqual(normalized, "none") | ||
|
||
def test_Flask(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_flask
normalized = vanity.normalize("fLaSk") | ||
self.assertEqual(normalized, "Flask") | ||
|
||
def test_SpaceString(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_space_string
def test_SpaceString(self): | ||
normalized = vanity.normalize(" Flask ") | ||
self.assertEqual(normalized, " Flask ") | ||
|
||
def testEmpty(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_empty
normalized = vanity.normalize(" Flask ") | ||
self.assertEqual(normalized, " Flask ") | ||
|
||
def test_Empty(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_empty
normalized = vanity.normalize(None) | ||
self.assertEqual(normalized, "none") | ||
|
||
def testEmpty(self): | ||
def test_Flask(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> test_flask
Should I also rename the class name to test_Normalize for consistency? |
@killvung Yes please. |
I actually changed it while asking for it |
And please lowercase it for consistency too. Thank you! |
Done |
I squashed this into a single commit and merged. Thanks! |
Thank you so much |
Sorry about the white spaces problem, I am using Sublime without any plugins, so I don't realized the existence of them.
Also I had fixed some of the problems @hugovk mentioned