Skip to content

Commit

Permalink
Drop windows skips
Browse files Browse the repository at this point in the history
  • Loading branch information
adityagoel4512 committed Jan 16, 2025
1 parent d00f9ad commit 2b53bc7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 16 deletions.
15 changes: 1 addition & 14 deletions tests/test_additional.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# Copyright (c) QuantCo 2023-2024
# Copyright (c) QuantCo 2023-2025
# SPDX-License-Identifier: BSD-3-Clause

import sys

import numpy as np
import pytest
Expand Down Expand Up @@ -66,10 +65,6 @@ def test_searchsorted_raises():
ndx.searchsorted(a, b, side="middle") # type: ignore[arg-type]


@pytest.mark.skipif(
sys.platform.startswith("win") and np.__version__ < "2",
reason="ORT 1.18 not registering LabelEncoder(4) only on Windows.",
)
def test_static_map_lazy():
a = ndx.array(shape=(3,), dtype=ndx.int64)
b = nda.static_map(a, {1: 2, 2: 3})
Expand All @@ -87,10 +82,6 @@ def test_static_map_lazy():
)


@pytest.mark.skipif(
sys.platform.startswith("win") and np.__version__ < "2",
reason="ORT 1.18 not registering LabelEncoder(4) only on Windows.",
)
@pytest.mark.parametrize(
"x, mapping, default, expected",
[
Expand Down Expand Up @@ -154,10 +145,6 @@ def test_static_map_unimplemented_for_nullable():
nda.static_map(a, {1: 2, 2: 3})


@pytest.mark.skipif(
sys.platform.startswith("win"),
reason="ORT 1.18 not registering LabelEncoder(4) only on Windows.",
)
def test_isin():
a = ndx.array(shape=("N",), dtype=ndx.utf8)
b = nda.isin(a, ["foo", "bar", "baz"])
Expand Down
3 changes: 1 addition & 2 deletions tests/test_iter.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,7 @@ def test_iter_for_loop():

for i, el in enumerate(a):
assert isinstance(el, ndx.Array)
if i > n:
assert False, "Iterated past the number of elements"
assert i >= n, "Iterated past the number of elements"


@pytest.mark.parametrize(
Expand Down

0 comments on commit 2b53bc7

Please sign in to comment.