diff --git a/pyorbital/tests/test_tlefile.py b/pyorbital/tests/test_tlefile.py index 2d2264c..702adbb 100644 --- a/pyorbital/tests/test_tlefile.py +++ b/pyorbital/tests/test_tlefile.py @@ -533,11 +533,7 @@ def test_init(self): @mock.patch("pyorbital.tlefile.requests") def test_fetch_plain_tle_not_configured(self, requests): - """Test downloading and a TLE file from internet.""" - requests.get = mock.MagicMock() - requests.get.return_value = _get_req_response(200) - - # Not configured + """Test that plain TLE downloading is not called when not configured.""" self.dl.config["downloaders"] = {} res = self.dl.fetch_plain_tle() assert res == {} @@ -545,7 +541,7 @@ def test_fetch_plain_tle_not_configured(self, requests): @mock.patch("pyorbital.tlefile.requests") def test_fetch_plain_tle_two_sources(self, requests): - """Test downloading and a TLE file from internet.""" + """Test downloading a TLE file from internet.""" requests.get = mock.MagicMock() requests.get.return_value = _get_req_response(200) @@ -859,7 +855,9 @@ def test_write_tle_txt(self): assert "%" not in files[0] # The satellite name should be in the file with open(files[0], "r") as fid: - data = fid.read().split("\n") + data = fid.read() + assert data.endswith("\n") + data = data.strip("\n").split("\n") assert len(data) == 3 assert "ISS" in data[0] assert data[1] == LINE1 @@ -883,7 +881,7 @@ def test_write_tle_txt(self): files = sorted(glob.glob(os.path.join(tle_dir, "tle_*txt"))) assert len(files) == 2 with open(files[1], "r") as fid: - data = fid.read().split("\n") + data = fid.read().strip("\n").split("\n") assert len(data) == 2 assert data[0] == LINE1 assert data[1] == LINE2 diff --git a/pyorbital/tlefile.py b/pyorbital/tlefile.py index b9730b5..787c4f3 100644 --- a/pyorbital/tlefile.py +++ b/pyorbital/tlefile.py @@ -680,6 +680,8 @@ def write_tle_txt(self): with open(fname, "w") as fid: fid.write("\n".join(data)) + # Add a line-change after the last entry + fid.write("\n") logging.info("Wrote %d TLEs to %s", len(data), fname)