MAINT: fixing datetime deprecrations

Author: bsipocz

astroquery/alma/tests/test_alma_remote.py

@@ -1,5 +1,5 @@
 # Licensed under a 3-clause BSD style license - see LICENSE.rst
-from datetime import datetime
+from datetime import datetime, timezone
 import os
 from pathlib import Path
 from urllib.parse import urlparse
@@ -139,7 +139,7 @@ def test_data_proprietary(self, alma):
         # public
         assert not alma.is_proprietary('uid://A001/X12a3/Xe9')
         IVOA_DATE_FORMAT = "%Y-%m-%dT%H:%M:%S.%f"
-        now = datetime.utcnow().strftime(IVOA_DATE_FORMAT)[:-3]
+        now = datetime.now(timezone.utc).strftime(IVOA_DATE_FORMAT)[:-3]
         query = "select top 1 member_ous_uid from ivoa.obscore where " \
                 "obs_release_date > '{}'".format(now)
         result = alma.query_tap(query)

astroquery/cadc/tests/test_cadctap_remote.py

@@ -9,7 +9,7 @@
 import os
 import requests
 from pathlib import Path
-from datetime import datetime
+from datetime import datetime, timezone
 from astropy.coordinates import SkyCoord
 from astropy.io import fits
 from astropy import units as u
@@ -99,7 +99,7 @@ def test_query(self):
         assert 1000 < result[0][0]
 
         # test that no proprietary results are returned when not logged in
-        now = datetime.utcnow()
+        now = datetime.now(timezone.utc)
         query = "select top 1 * from caom2.Plane where " \
                 "metaRelease>'{}'".format(now.strftime('%Y-%m-%dT%H:%M:%S.%f'))
         result = cadc.exec_sync(query)
@@ -121,7 +121,7 @@ def test_login_with_user_password(self):
         for auth_session in [None, authsession.AuthSession(),
                              requests.Session()]:
             cadc = Cadc(auth_session=auth_session)
-            now = datetime.utcnow()
+            now = datetime.now(timezone.utc)
             query = \
                 "select top 1 * from caom2.Plane where metaRelease>'{}'".\
                 format(now.strftime('%Y-%m-%dT%H:%M:%S.%f'))
@@ -152,7 +152,7 @@ def test_login_with_user_password(self):
     def test_login_with_cert(self):
         for auth_session in [requests.Session()]:
             cadc = Cadc(auth_session=auth_session)
-            now = datetime.utcnow()
+            now = datetime.now(timezone.utc)
             query = \
                 "select top 1 * from caom2.Plane where metaRelease>'{}'".\
                 format(now.strftime('%Y-%m-%dT%H:%M:%S.%f'))
@@ -181,14 +181,14 @@ def test_authsession(self):
         auth_session = requests.Session()
         auth_session.cert = os.environ['CADC_CERT']
         cadc = Cadc(auth_session=auth_session)
-        now = datetime.utcnow()
+        now = datetime.now(timezone.utc)
         query = "select top 1 * from caom2.Plane where " \
                 "metaRelease>'{}'".format(now.strftime('%Y-%m-%dT%H:%M:%S.%f'))
         result = cadc.exec_sync(query)
         assert len(result) == 1
         annon_session = requests.Session()
         cadc = Cadc(auth_session=annon_session)
-        now = datetime.utcnow()
+        now = datetime.now(timezone.utc)
         query = "select top 1 * from caom2.Plane where " \
                 "metaRelease>'{}'".format(now.strftime('%Y-%m-%dT%H:%M:%S.%f'))
         result = cadc.exec_sync(query)

astroquery/esa/jwst/core.py

@@ -15,7 +15,7 @@
 import shutil
 import tarfile
 import zipfile
-from datetime import datetime
+from datetime import datetime, timezone
 from urllib.parse import urlencode
 
 from astropy import log
@@ -1050,7 +1050,7 @@ def __extract_file(self, output_file_full_path, output_dir, files):
 
     def __set_dirs(self, output_file, observation_id):
         if output_file is None:
-            now = datetime.now()
+            now = datetime.now(timezone.utc)
             formatted_now = now.strftime("%Y%m%d_%H%M%S")
             output_dir = os.getcwd() + os.sep + "temp_" + \
                 formatted_now

astroquery/gaia/core.py

@@ -15,7 +15,7 @@
 """
 import zipfile
 import os
-from datetime import datetime
+from datetime import datetime, timezone
 import shutil
 from collections.abc import Iterable
 
@@ -221,7 +221,7 @@ def load_data(self, ids, *, data_release=None, data_structure='INDIVIDUAL',
         -------
         A table object
         """
-        now = datetime.now()
+        now = datetime.now(timezone.utc)
         now_formatted = now.strftime("%Y%m%d_%H%M%S")
         temp_dirname = "temp_" + now_formatted
         downloadname_formated = "download_" + now_formatted

astroquery/ipac/ned/core.py

@@ -5,7 +5,7 @@
 from collections import namedtuple
 from xml.dom.minidom import parseString
 
-from datetime import datetime
+from datetime import datetime, timezone
 
 import astropy.units as u
 import astropy.coordinates as coord
@@ -625,7 +625,7 @@ def get_table_async(self, object_name, *, table='photometry',
                 'yes' if kwargs.get('extended_search') else 'no')
             request_payload['begin_year'] = kwargs.get('from_year', 1800)
             request_payload['end_year'] = kwargs.get('to_year',
-                                                     datetime.now().year)
+                                                     datetime.now(timezone.utc).year)
         if get_query_payload:
             return request_payload
         response = self._request("GET", url=Ned.DATA_SEARCH_URL,

astroquery/query.py

@@ -12,7 +12,7 @@
 import requests
 import textwrap
 
-from datetime import datetime, timedelta
+from datetime import datetime, timezone, timedelta
 from pathlib import Path
 
 from astropy.config import paths
@@ -117,8 +117,8 @@ def from_cache(self, cache_location, cache_timeout):
             if cache_timeout is None:
                 expired = False
             else:
-                current_time = datetime.utcnow()
-                cache_time = datetime.utcfromtimestamp(request_file.stat().st_mtime)
+                current_time = datetime.now(timezone.utc)
+                cache_time = datetime.fromtimestamp(request_file.stat().st_mtime, timezone.utc)
                 expired = current_time-cache_time > timedelta(seconds=cache_timeout)
             if not expired:
                 with open(request_file, "rb") as f:

astroquery/utils/tap/taputils.py

@@ -17,7 +17,7 @@
 
 import re
 import warnings
-from datetime import datetime
+from datetime import datetime, timezone
 
 TAP_UTILS_QUERY_TOP_PATTERN = re.compile(
     r"\s*SELECT\s+(ALL\s+|DISTINCT\s+)?TOP\s+\d+\s+", re.IGNORECASE)
@@ -224,7 +224,7 @@ def get_table_name(full_qualified_table_name):
 
 def get_suitable_output_file(conn_handler, async_job, output_file, headers,
                              is_error, output_format):
-    date_time = datetime.now().strftime("%Y%m%d%H%M%S")
+    date_time = datetime.now(timezone.utc).strftime("%Y%m%d%H%M%S")
     if output_file is None:
         file_name = conn_handler.get_file_from_header(headers)
         if file_name is None:

docs/conf.py

@@ -78,7 +78,7 @@
 project = setup_cfg['name']
 author = setup_cfg['author']
 copyright = '{0}, {1}'.format(
-    datetime.datetime.now().year, setup_cfg['author'])
+    datetime.datetime.now(datetime.UTC).year, setup_cfg['author'])
 
 # The version info for the project you're documenting, acts as replacement for
 # |version| and |release|, also used in various other places throughout the