kallithea Changeset - 491961dfd0c8

Changeset - 491961dfd0c8

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Thomas De Schampheleire - 10 years ago 2016-02-10 18:30:27
thomas.de.schampheleire@gmail.com

pytest migration: files: convert to TestControllerPytest and parametrize

Migrate the 'files' tests to pytest and its parametrize decorator.
The syntax of this decorator is slightly different: a first argument is a
comma-separated string with the method argument names for which the values
are provided as second argument.

1 file changed with 5 insertions and 5 deletions:

kallithea/tests/functional/test_files.py

0 comments (0 inline, 0 general)

kallithea/tests/functional/test_files.py

➞

Show inline comments

@@ @@ -24,7 +24,7 @@ def _set_downloads(repo_name, set_to): @@
     Session().commit()
-class TestFilesController(TestController):
+class TestFilesController(TestControllerPytest):
     def test_index(self):
         self.log_user()
@@ @@ -347,7 +347,7 @@ removed extra unicode conversion in diff @@
         self.checkSessionFlash(response, 'No filename')
-    @parameterized.expand([
+    @parametrize('location,filename', [
         ('/abs', 'foo'),
         ('../rel', 'foo'),
         ('file/../foo', 'foo'),
@@ @@ -367,7 +367,7 @@ removed extra unicode conversion in diff @@
         self.checkSessionFlash(response, 'Location must be relative path and must not contain .. in path')
-    @parameterized.expand([
+    @parametrize('cnt,location,filename', [
         (1, '', 'foo.txt'),
         (2, 'dir', 'foo.rst'),
         (3, 'rel/dir', 'foo.bar'),
@@ @@ -423,7 +423,7 @@ removed extra unicode conversion in diff @@
         self.checkSessionFlash(response, 'No filename')
-    @parameterized.expand([
+    @parametrize('location,filename', [
         ('/abs', 'foo'),
         ('../rel', 'foo'),
         ('file/../foo', 'foo'),
@@ @@ -443,7 +443,7 @@ removed extra unicode conversion in diff @@
         self.checkSessionFlash(response, 'Location must be relative path and must not contain .. in path')
-    @parameterized.expand([
+    @parametrize('cnt,location,filename', [
         (1, '', 'foo.txt'),
         (2, 'dir', 'foo.rst'),
         (3, 'rel/dir', 'foo.bar'),

0 comments (0 inline, 0 general)