Skip to content

Commit

Permalink
more agg tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Colin Ho authored and Colin Ho committed Sep 25, 2024
1 parent 2026e4b commit aa61bcf
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 18 deletions.
6 changes: 0 additions & 6 deletions tests/cookbook/test_aggregations.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,11 @@
import pandas as pd
import pytest

from daft import context
from daft.datatype import DataType
from daft.expressions import col
from daft.udf import udf
from tests.conftest import assert_df_equals

pytestmark = pytest.mark.skipif(
context.get_context().daft_execution_config.enable_native_executor is True,
reason="Native executor fails for these tests",
)


def test_sum(daft_df, service_requests_csv_pd_df, repartition_nparts):
"""Sums across an entire column for the entire table"""
Expand Down
7 changes: 1 addition & 6 deletions tests/dataframe/test_aggregations.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,18 +7,13 @@
import pytest

import daft
from daft import col, context
from daft import col
from daft.context import get_context
from daft.datatype import DataType
from daft.errors import ExpressionTypeError
from daft.utils import freeze
from tests.utils import sort_arrow_table

pytestmark = pytest.mark.skipif(
context.get_context().daft_execution_config.enable_native_executor is True,
reason="Native executor fails for these tests",
)


@pytest.mark.parametrize("repartition_nparts", [1, 2, 4])
def test_agg_global(make_df, repartition_nparts):
Expand Down
6 changes: 0 additions & 6 deletions tests/dataframe/test_distinct.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,9 @@
import pyarrow as pa
import pytest

from daft import context
from daft.datatype import DataType
from tests.utils import sort_arrow_table

pytestmark = pytest.mark.skipif(
context.get_context().daft_execution_config.enable_native_executor is True,
reason="Native executor fails for these tests",
)


@pytest.mark.parametrize("repartition_nparts", [1, 2, 5])
def test_distinct_with_nulls(make_df, repartition_nparts):
Expand Down

0 comments on commit aa61bcf

Please sign in to comment.