1 |
commit: d3e5cf43918fd6134ce258683a757d4f9778917b |
2 |
Author: Brian Dolbec <dolsen <AT> gentoo <DOT> org> |
3 |
AuthorDate: Thu Apr 23 17:35:30 2020 +0000 |
4 |
Commit: Ulrich Müller <ulm <AT> gentoo <DOT> org> |
5 |
CommitDate: Tue May 4 15:48:08 2021 +0000 |
6 |
URL: https://gitweb.gentoo.org/proj/g-sorcery.git/commit/?id=d3e5cf43 |
7 |
|
8 |
test_DBGenerator.py, test_PackageDB.py: Fix test errors |
9 |
|
10 |
The http server was not being shutdown when exceptions were raised |
11 |
This caused other errors in tests run after the failed test. |
12 |
|
13 |
Signed-off-by: Brian Dolbec <dolsen <AT> gentoo.org> |
14 |
Signed-off-by: Ulrich Müller <ulm <AT> gentoo.org> |
15 |
|
16 |
tests/test_DBGenerator.py | 12 +++++++----- |
17 |
tests/test_PackageDB.py | 12 +++++++----- |
18 |
2 files changed, 14 insertions(+), 10 deletions(-) |
19 |
|
20 |
diff --git a/tests/test_DBGenerator.py b/tests/test_DBGenerator.py |
21 |
index 9a47c86..3c28278 100644 |
22 |
--- a/tests/test_DBGenerator.py |
23 |
+++ b/tests/test_DBGenerator.py |
24 |
@@ -4,9 +4,9 @@ |
25 |
""" |
26 |
test_DBGenerator.py |
27 |
~~~~~~~~~~~~~~~~~~~ |
28 |
- |
29 |
+ |
30 |
DBGenerator test suite |
31 |
- |
32 |
+ |
33 |
:copyright: (c) 2013 by Jauhien Piatlicki |
34 |
:license: GPL-2, see LICENSE for more details. |
35 |
""" |
36 |
@@ -81,11 +81,13 @@ class TestDBGenerator(BaseTest): |
37 |
srv = Server(orig_tempdir.name) |
38 |
srv.start() |
39 |
|
40 |
- pkg_db = db_generator(self.tempdir.name, "test_repo", |
41 |
+ try: |
42 |
+ pkg_db = db_generator(self.tempdir.name, "test_repo", |
43 |
common_config = common_config, config = config) |
44 |
|
45 |
- srv.shutdown() |
46 |
- srv.join() |
47 |
+ finally: |
48 |
+ srv.shutdown() |
49 |
+ srv.join() |
50 |
|
51 |
self.assertEqual(set(pkg_db.list_categories()), set(["app-test1", "app-test2"])) |
52 |
self.assertTrue(pkg_db.in_category("app-test1", "test")) |
53 |
|
54 |
diff --git a/tests/test_PackageDB.py b/tests/test_PackageDB.py |
55 |
index 152c605..8be8f8a 100644 |
56 |
--- a/tests/test_PackageDB.py |
57 |
+++ b/tests/test_PackageDB.py |
58 |
@@ -70,11 +70,13 @@ class TestPackageDB(BaseTest): |
59 |
|
60 |
srv = Server(orig_tempdir.name, port=port) |
61 |
srv.start() |
62 |
- self.assertRaises(IntegrityError, test_db.sync, sync_address) |
63 |
- os.system("cd " + orig_tempdir.name + " && mv good.tar.gz dummy.tar.gz") |
64 |
- test_db.sync(sync_address) |
65 |
- srv.shutdown() |
66 |
- srv.join() |
67 |
+ try: |
68 |
+ self.assertRaises(IntegrityError, test_db.sync, sync_address) |
69 |
+ os.system("cd " + orig_tempdir.name + " && mv good.tar.gz dummy.tar.gz") |
70 |
+ test_db.sync(sync_address) |
71 |
+ finally: |
72 |
+ srv.shutdown() |
73 |
+ srv.join() |
74 |
test_db.read() |
75 |
self.assertEqual(orig_db.database, test_db.database) |
76 |
self.assertEqual(orig_db.get_common_data("app-test1"), test_db.get_common_data("app-test1")) |