jorisvandenbossche commented on code in PR #38179:
URL: https://github.com/apache/arrow/pull/38179#discussion_r1526072584
##########
python/pyarrow/tests/test_util.py:
##########
@@ -207,3 +209,19 @@ def test_signal_refcycle():
assert wr() is not None
_break_traceback_cycle_from_frame(sys._getframe(0))
assert wr() is None
+
+
[email protected](sys.platform != "win32",
+ reason="Timezone database is already provided.")
+def test_download_tzdata_on_windows():
+ # Download timezone database
+ download_tzdata_on_windows()
+
+ # Inspect the folder
+ tzdata_path = os.path.expandvars(r"%USERPROFILE%\Downloads\tzdata")
+ tzdata_zones_path = os.path.join(tzdata_path, "windowsZones.xml")
+ assert os.path.exists(tzdata_path)
+ assert os.path.exists(tzdata_zones_path)
+ # 30 files in tzdata (2023) + compressed tzdata.tar.gz + windowsZones.xml
+ assert len(os.listdir(tzdata_path)) == 32
Review Comment:
Yes, either remove completely, or maybe check the presence of a certain file
we are sure will be there (eg `europe`).
You can also verify explicitly the extra `windowsZones.xml` is contained in
the directory
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
