Skip to content

Commit d881682

Browse files
author
Javier Duran
committed
Adding unit tests
1 parent 528ea1e commit d881682

File tree

1 file changed

+26
-4
lines changed

1 file changed

+26
-4
lines changed

astroquery/esa/xmm_newton/tests/test_xmm_newton.py

Lines changed: 26 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,9 @@
2323

2424
from ..core import XMMNewtonClass
2525
from ..tests.dummy_tap_handler import DummyXMMNewtonTapHandler
26+
from ..tests.dummy_handler import DummyHandler
2627
from fileinput import filename
28+
from tarfile import is_tarfile
2729

2830

2931
class TestXMMNewton():
@@ -75,15 +77,20 @@ def test_query_xsa_tap(self):
7577
xsa = XMMNewtonClass(self.get_dummy_tap_handler())
7678
xsa.query_xsa_tap(**parameters)
7779
self.get_dummy_tap_handler().check_call("launch_job", parameters)
80+
self.get_dummy_tap_handler().check_parameters(parameters, "launch_job")
81+
self.get_dummy_tap_handler().check_method("launch_job")
82+
self.get_dummy_tap_handler().get_tables()
83+
self.get_dummy_tap_handler().get_columns()
84+
self.get_dummy_tap_handler().load_tables()
7885

7986
def test_get_tables(self):
8087
parameters2 = {'only_names': True,
8188
'verbose': True}
8289

83-
dummyTapHandler = DummyXMMNewtonTapHandler("get_tables", parameters2)
84-
xsa = XMMNewtonClass(self.get_dummy_tap_handler())
85-
xsa.get_tables(only_names=True, verbose=True)
86-
dummyTapHandler.check_call("get_tables", parameters2)
90+
dummyHandler = DummyXMMNewtonTapHandler("get_tables", parameters2)
91+
dummyHandler.check_call("get_tables", parameters2)
92+
dummyHandler.check_method("get_tables")
93+
dummyHandler.check_parameters(parameters2, "get_tables")
8794

8895
def test_get_columns(self):
8996
parameters2 = {'table_name': "table",
@@ -95,6 +102,16 @@ def test_get_columns(self):
95102
xsa.get_columns("table", only_names=True, verbose=True)
96103
dummyTapHandler.check_call("get_columns", parameters2)
97104

105+
def test_dummy_handler(self):
106+
parameters2 = {'table_name': "table",
107+
'only_names': True,
108+
'verbose': True}
109+
110+
dummyTapHandler = DummyHandler("get_columns", parameters2)
111+
xsa = XMMNewtonClass(self.get_dummy_tap_handler())
112+
xsa.get_columns("table", only_names=True, verbose=True)
113+
dummyTapHandler.check_call("get_columns", parameters2)
114+
98115
def test_parse_filename(self):
99116
self._create_tar("filename.tar", self._files)
100117
xsa = XMMNewtonClass(self.get_dummy_tap_handler())
@@ -290,6 +307,11 @@ def _create_tar(self, tarname, files):
290307
shutil.rmtree(os.path.join(ob_name, ftype))
291308
shutil.rmtree(ob_name)
292309

310+
def test_create_tar_lightcurves(self):
311+
_tarname = "tarfile_lightcurves.tar"
312+
self._create_tar_lightcurves(_tarname, self._files_lightcurves)
313+
assert os.path.isfile(_tarname)
314+
293315
def test_get_epic_spectra_non_existing_file(self, capsys):
294316
_tarname = "nonexistingfile.tar"
295317
_source_number = 83

0 commit comments

Comments
 (0)