Skip to content

Commit d7086a6

Browse files
committed
[test/scripts] Reduce odmltordf code redundancy
1 parent bf58187 commit d7086a6

File tree

1 file changed

+14
-29
lines changed

1 file changed

+14
-29
lines changed

test/test_script_odml_to_rdf.py

Lines changed: 14 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,7 @@ def test_script_exit(self):
3535
with self.assertRaises(SystemExit):
3636
odml_to_rdf.main(["--version"])
3737

38-
def test_valid_conversion(self):
39-
# make sure temp dir is empty
40-
self.assertListEqual(os.listdir(self.tmp_dir), [])
41-
42-
# run converter on root directory containing two files
43-
odml_to_rdf.main(["-o", self.tmp_dir, self.dir_files])
44-
38+
def _check_intermediate_dirs(self):
4539
# make sure an odml version conversion output directory has been created
4640
out_dir_lst = os.listdir(self.tmp_dir)
4741
self.assertEqual(len(out_dir_lst), 1)
@@ -54,6 +48,17 @@ def test_valid_conversion(self):
5448
rdf_dir = os.path.join(out_dir, rdf_dir_lst[0])
5549
self.assertTrue(os.path.isdir(rdf_dir))
5650

51+
return rdf_dir
52+
53+
def test_valid_conversion(self):
54+
# make sure temp dir is empty
55+
self.assertListEqual(os.listdir(self.tmp_dir), [])
56+
57+
# run converter on root directory containing two files
58+
odml_to_rdf.main(["-o", self.tmp_dir, self.dir_files])
59+
60+
rdf_dir = self._check_intermediate_dirs()
61+
5762
# make sure two files have been created
5863
file_lst = os.listdir(rdf_dir)
5964
self.assertEqual(len(file_lst), 2)
@@ -70,17 +75,7 @@ def test_invalid(self):
7075
# run converter on directory with invalid files
7176
odml_to_rdf.main(["-o", self.tmp_dir, self.dir_invalid])
7277

73-
# make sure an output directory has been created
74-
out_dir_lst = os.listdir(self.tmp_dir)
75-
self.assertEqual(len(out_dir_lst), 1)
76-
out_dir = os.path.join(self.tmp_dir, out_dir_lst[0])
77-
self.assertTrue(os.path.isdir(out_dir))
78-
79-
# make sure an rdf conversion output directory has been created
80-
rdf_dir_lst = os.listdir(out_dir)
81-
self.assertEqual(len(rdf_dir_lst), 1)
82-
rdf_dir = os.path.join(out_dir, rdf_dir_lst[0])
83-
self.assertTrue(os.path.isdir(rdf_dir))
78+
rdf_dir = self._check_intermediate_dirs()
8479

8580
# make sure no file has been created
8681
self.assertListEqual(os.listdir(rdf_dir), [])
@@ -92,17 +87,7 @@ def test_recursive_conversion(self):
9287
# run converter on root directory containing two files
9388
odml_to_rdf.main(["-r", "-o", self.tmp_dir, self.dir_recursive])
9489

95-
# make sure an odml version conversion output directory has been created
96-
out_dir_lst = os.listdir(self.tmp_dir)
97-
self.assertEqual(len(out_dir_lst), 1)
98-
out_dir = os.path.join(self.tmp_dir, out_dir_lst[0])
99-
self.assertTrue(os.path.isdir(out_dir))
100-
101-
# make sure an rdf conversion output directory has been created
102-
rdf_dir_lst = os.listdir(out_dir)
103-
self.assertEqual(len(rdf_dir_lst), 1)
104-
rdf_dir = os.path.join(out_dir, rdf_dir_lst[0])
105-
self.assertTrue(os.path.isdir(rdf_dir))
90+
rdf_dir = self._check_intermediate_dirs()
10691

10792
# make sure two files have been created
10893
file_lst = os.listdir(rdf_dir)

0 commit comments

Comments
 (0)