rm qtbot, add back everything else

src/yt_napari/_tests/test_widget_reader.py

@@ -1,8 +1,6 @@
-import sys
 from functools import partial
 
 import numpy as np
-import pytest
 
 from yt_napari._ds_cache import dataset_cache
 from yt_napari._widget_reader import ReaderWidget, SelectionEntry
@@ -30,8 +28,6 @@ def test_widget_reader_add_selections(make_napari_viewer, yt_ugrid_ds_fn):
     assert sel.selection_type == "Slice"
 
 
-#@pytest.mark.skipif(sys.platform == "win32", reason="does not run on windows")
-#def test_widget_reader(make_napari_viewer, yt_ugrid_ds_fn, caplog, qtbot):
 def test_widget_reader(make_napari_viewer, yt_ugrid_ds_fn, caplog):
 
     # make_napari_viewer is a pytest fixture. It takes any keyword arguments
@@ -40,45 +36,44 @@ def test_widget_reader(make_napari_viewer, yt_ugrid_ds_fn, caplog):
     viewer = make_napari_viewer()
 
     r = ReaderWidget(napari_viewer=viewer)
-   # qtbot.addWidget(r)
 
     r.ds_container.filename.value = yt_ugrid_ds_fn
     r.add_new_button.click()
     sel = list(r.active_selections.values())[0]
     assert isinstance(sel, SelectionEntry)
-#
-#    mgui_region = sel.selection_container_raw
-#    mgui_region.fields.field_type.value = "gas"
-#    mgui_region.fields.field_name.value = "density"
-#    mgui_region.resolution.value = (10, 10, 10)
-
-#    def rebuild_data(final_shape, data):
-#        # the yt file thats being loaded from the pytest fixture is a saved
-#        # dataset created from an in-memory uniform grid, and the re-loaded
-#        # dataset will not have the full functionality of a ds. so here, we
-#        # inject a correctly shaped random array here. If we start using full
-#        # test datasets from yt in testing, this should be changed.
-#        return np.random.random(final_shape) * data.mean()
-#
-#    rebuild = partial(rebuild_data, mgui_region.resolution.value)
-#    r._post_load_function = rebuild
-#    r.load_data()
-
-#    mgui_region.fields.field_name.value = "temperature"
-#    mgui_region.left_edge.value.value = (0.4, 0.4, 0.4)
-#    mgui_region.right_edge.value.value = (0.6, 0.6, 0.6)
-#    r.load_data()
-#    # should have read from cache, check the log:
-#    assert yt_ugrid_ds_fn in caplog.text
-#    # the viewer should now have two images
-#    assert len(viewer.layers) == 2
-#
-#    temp_layer = viewer.layers[1]
-#    assert temp_layer.metadata["_yt_napari_layer"] is True
-#
-#    r.clear_cache()
-#    assert len(dataset_cache.available) == 0
-#
-#    _ = r.yt_scene
-#    yt_scene = r.yt_scene
-#    assert isinstance(yt_scene, Scene)
+
+    mgui_region = sel.selection_container_raw
+    mgui_region.fields.field_type.value = "gas"
+    mgui_region.fields.field_name.value = "density"
+    mgui_region.resolution.value = (10, 10, 10)
+
+    def rebuild_data(final_shape, data):
+        # the yt file thats being loaded from the pytest fixture is a saved
+        # dataset created from an in-memory uniform grid, and the re-loaded
+        # dataset will not have the full functionality of a ds. so here, we
+        # inject a correctly shaped random array here. If we start using full
+        # test datasets from yt in testing, this should be changed.
+        return np.random.random(final_shape) * data.mean()
+
+    rebuild = partial(rebuild_data, mgui_region.resolution.value)
+    r._post_load_function = rebuild
+    r.load_data()
+
+    mgui_region.fields.field_name.value = "temperature"
+    mgui_region.left_edge.value.value = (0.4, 0.4, 0.4)
+    mgui_region.right_edge.value.value = (0.6, 0.6, 0.6)
+    r.load_data()
+    # should have read from cache, check the log:
+    assert yt_ugrid_ds_fn in caplog.text
+    # the viewer should now have two images
+    assert len(viewer.layers) == 2
+
+    temp_layer = viewer.layers[1]
+    assert temp_layer.metadata["_yt_napari_layer"] is True
+
+    r.clear_cache()
+    assert len(dataset_cache.available) == 0
+
+    _ = r.yt_scene
+    yt_scene = r.yt_scene
+    assert isinstance(yt_scene, Scene)