fix more tests

Author: austin3dickey

extensions/positron-python/python_files/posit/pinned-test-requirements.txt

@@ -47,7 +47,7 @@ torch==2.8.0; python_version < '3.14'
 scipy==1.13.1; python_version == '3.9'
 scipy==1.15.3; python_version == '3.10'
 scipy==1.16.2; python_version >= '3.11'
-snowflake-connector-python==3.17.4
+snowflake-connector-python==3.17.4; python_version < '3.14'
 SQLAlchemy==2.0.43
 
 # putting this last like test-requirements.txt

extensions/positron-python/python_files/posit/positron/tests/test_inspectors.py

@@ -643,11 +643,11 @@ def mutate(x):
 
 @pytest.mark.skipif(geopandas is None, reason="geopandas is not available")
 def test_inspect_geopandas_dataframe() -> None:
-    p1 = Polygon([(0, 0), (1, 0), (1, 1)])
-    p2 = Polygon([(0, 0), (1, 0), (1, 1), (0, 1)])
-    p3 = Polygon([(2, 0), (3, 0), (3, 1), (2, 1)])
+    p1 = Polygon([(0, 0), (1, 0), (1, 1)])  # type: ignore
+    p2 = Polygon([(0, 0), (1, 0), (1, 1), (0, 1)])  # type: ignore
+    p3 = Polygon([(2, 0), (3, 0), (3, 1), (2, 1)])  # type: ignore
 
-    value = geopandas.GeoDataFrame({"g": geopandas.GeoSeries([p1, p2, p3]), "data": [0, 1, 2]})
+    value = geopandas.GeoDataFrame({"g": geopandas.GeoSeries([p1, p2, p3]), "data": [0, 1, 2]})  # type: ignore
 
     rows, cols = value.shape
 
@@ -725,11 +725,11 @@ def mutate(x):
 
 @pytest.mark.skipif(geopandas is None, reason="geopandas is not available")
 def test_inspect_geopandas_series() -> None:
-    value = geopandas.GeoSeries(
+    value = geopandas.GeoSeries(  # type: ignore
         [
-            Polygon([(0, 0), (1, 0), (1, 1)]),
-            Polygon([(0, 0), (1, 0), (1, 1), (0, 1)]),
-            Polygon([(2, 0), (3, 0), (3, 1), (2, 1)]),
+            Polygon([(0, 0), (1, 0), (1, 1)]),  # type: ignore
+            Polygon([(0, 0), (1, 0), (1, 1), (0, 1)]),  # type: ignore
+            Polygon([(2, 0), (3, 0), (3, 1), (2, 1)]),  # type: ignore
         ]
     )
     (rows,) = value.shape
@@ -889,11 +889,11 @@ def test_get_child(value: Any, key: Any, expected: Any) -> None:
 @pytest.mark.skipif(ibis is None, reason="ibis not available")
 def test_inspect_ibis_exprs() -> None:
     # Make sure we don't return an executed repr
-    ibis.options.interactive = True
+    ibis.options.interactive = True  # type: ignore
 
     test_df = pd.DataFrame({"a": [1, 2, 1, 1, 2], "b": ["foo", "bar", "baz", "qux", None]})
     _, columns = test_df.shape
-    t = ibis.memtable(test_df, name="df")
+    t = ibis.memtable(test_df, name="df")  # type: ignore
     table_type = "ibis.Table"
 
     verify_inspector(
@@ -933,7 +933,7 @@ def test_inspect_ibis_exprs() -> None:
     [
         (lambda: np.array([[1, 2, 3], [4, 5, 6]], dtype="int64"), 48),
         pytest.param(
-            lambda: torch.Tensor([[1, 2, 3], [4, 5, 6]]),
+            lambda: torch.Tensor([[1, 2, 3], [4, 5, 6]]),  # type: ignore
             24,
             marks=pytest.mark.skipif(torch is None, reason="torch not available"),
         ),
@@ -998,7 +998,7 @@ class VeryLongClassNameThatShouldDefinitelyBeTruncatedBecauseItIsWayTooLong:
         ),
         pytest.param(lambda: np.ones((20, 20)), id="numpy_array"),
         pytest.param(
-            lambda: torch.ones((20, 20)),
+            lambda: torch.ones((20, 20)),  # type: ignore
             id="torch_tensor",
             marks=pytest.mark.skipif(torch is None, reason="torch not available"),
         ),

extensions/positron-python/python_files/posit/positron/tests/test_ui.py

@@ -105,7 +105,7 @@ def test_set_console_width(ui_comm: DummyComm) -> None:
     assert pd.get_option("display.width") is None
     assert pl.Config.state()["POLARS_TABLE_WIDTH"] == str(width)
     if torch is not None:
-        assert torch._tensor_str.PRINT_OPTS.linewidth == width  # noqa: SLF001
+        assert torch._tensor_str.PRINT_OPTS.linewidth == width  # noqa: SLF001  # type: ignore
 
 
 def test_open_editor(ui_service: UiService, ui_comm: DummyComm) -> None:

extensions/positron-python/python_files/posit/test-requirements.txt

@@ -24,7 +24,7 @@ pytest-mock
 syrupy
 torch; python_version < '3.14'
 scipy
-snowflake-connector-python
+snowflake-connector-python; python_version < '3.14'
 sqlalchemy
 
 # putting this last because holoviews is picky about dependency versions (including bokeh),

extensions/positron-python/src/test/common/platform/filesystem.functional.test.ts

@@ -13,7 +13,6 @@ import {
     DOES_NOT_EXIST,
     fixPath,
     FSFixture,
-    OSX,
     SUPPORTS_SOCKETS,
     SUPPORTS_SYMLINKS,
     WINDOWS,
@@ -469,12 +468,6 @@ suite('FileSystem - utils', () => {
             if (!SUPPORTS_SOCKETS) {
                 this.skip();
             }
-            // --- Start Positron ---
-            // for some reason this fails only on macOS, which upstream does not test on but we do
-            if (OSX) {
-                this.skip();
-            }
-            // --- End Positron ---
             const sockFile = await fix.createSocket('x/y/z/ipc.sock');
 
             const exists = utils.fileExistsSync(sockFile);
@@ -771,12 +764,6 @@ suite('FileSystem', () => {
                 if (!SUPPORTS_SOCKETS) {
                     this.skip();
                 }
-                // --- Start Positron ---
-                // for some reason this fails only on macOS, which upstream does not test on but we do
-                if (OSX) {
-                    this.skip();
-                }
-                // --- End Positron ---
                 const sockFile = await fix.createSocket('x/y/z/ipc.sock');
 
                 const exists = fileSystem.fileExistsSync(sockFile);

extensions/positron-python/src/test/common/platform/utils.ts

@@ -28,7 +28,11 @@ export const SUPPORTS_SYMLINKS = (() => {
     return true;
 })();
 export const SUPPORTS_SOCKETS = (() => {
-    if (WINDOWS) {
+    // --- Start Positron ---
+    // Upstream doesn't run on OSX but that seems to fail too.
+    // if (WINDOWS) {
+    if (WINDOWS || OSX) {
+        // --- End Positron ---
         // Windows requires named pipes to have a specific path under
         // the local domain ("\\.\pipe\*").  This makes them relatively
         // useless in our functional tests, where we want to use them