better comments

Author: LucaMarconato

tests/test_cli.py

@@ -28,6 +28,7 @@ def test_view_path_not_exists():
         Viewer.close_all()
 
 
+# added due to this https://github.com/napari/napari/issues/8214#issuecomment-3188565917
 @pytest.mark.usefixtures("mock_app_model")
 def test_view_path_is_dir():
     runner = CliRunner()

tests/test_scatterwidgets.py

@@ -284,6 +284,8 @@ def test_add_roi_to_plot(qtbot, plot_widget, prepare_discrete_test_data):
     assert len(rois) == 1, "There should be exactly one ROI in the scatter_plot"
 
 
+# sometimes this test fails due to a faulty clean-up, for example in the last failing
+# commit here: https://github.com/scverse/napari-spatialdata/pull/360
 def test_remove_roi_double_click(qtbot, plot_widget, prepare_discrete_test_data):
     """Test changing coordinates and removing ROI."""
     roi = pg.RectROI([0, 0], [1, 1])