Skip to content

Commit e1c1015

Browse files
committed
tests/*web*.py: convert misused docstrings to proper comments
1 parent f2eba34 commit e1c1015

File tree

2 files changed

+10
-12
lines changed

2 files changed

+10
-12
lines changed

debsources/tests/test_web_cp.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -199,10 +199,10 @@ def test_checksum_404(self):
199199

200200
def test_api_search_filename_package(self):
201201
# test package requirement
202-
'''rv = json.loads(self.app.get(
203-
"/copyright/api/file/random/debian/copyright/").data)
204-
self.assertEqual(rv['error'], 'File not found')
205-
self.assertEqual(rv['return_code'], 404)'''
202+
# rv = json.loads(self.app.get(
203+
# "/copyright/api/file/random/debian/copyright/").data)
204+
# self.assertEqual(rv['error'], 'File not found')
205+
# self.assertEqual(rv['return_code'], 404)
206206

207207
rv = json.loads(self.app.get(
208208
"/copyright/api/file/gnubg/1.02.000-2/Makefile.am/").data)

debsources/tests/test_webapp.py

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,11 @@
2727
class DebsourcesBaseWebTests(DbTestFixture):
2828
@classmethod
2929
def setUpClass(cls):
30-
"""
31-
We use the class method here. setUpClass is called at the class
32-
creation, and tearDownClass at the class destruction (instead of
33-
setUp and tearDown before and after each test). This is doable
34-
here because the app never modifies the db (so it's useless to
35-
create/destroy it many times), and this a big gain of time.
36-
"""
30+
# We use the class method here. setUpClass is called at the class
31+
# creation, and tearDownClass at the class destruction (instead of
32+
# setUp and tearDown before and after each test). This is doable here
33+
# because the app never modifies the db (so it's useless to
34+
# create/destroy it many times), and this a big gain of time.
3735
cls.db_setup_cls()
3836

3937
# creates an app object, which is used to run queries
@@ -59,7 +57,7 @@ def tearDownClass(cls):
5957
@attr('webapp')
6058
class DebsourcesTestCase(DebsourcesBaseWebTests, unittest.TestCase):
6159
def test_app_config(self):
62-
"""use existing config to initialize app wrapper"""
60+
# use existing config to initialize app wrapper
6361
config = dict(domain="test.debian.test")
6462
app_wrapper = AppWrapper(config=config)
6563
self.assertEqual(app_wrapper.app.config["domain"],

0 commit comments

Comments
 (0)