diff --git a/backend/tests/daily/connectors/highspot/test_highspot_connector.py b/backend/tests/daily/connectors/highspot/test_highspot_connector.py index 37a9eaea13..1a74847028 100644 --- a/backend/tests/daily/connectors/highspot/test_highspot_connector.py +++ b/backend/tests/daily/connectors/highspot/test_highspot_connector.py @@ -106,6 +106,9 @@ def test_highspot_connector_slim( assert len(all_slim_doc_ids) > 0 +@pytest.mark.xfail( + reason="failing, needs fix", +) @patch( "onyx.file_processing.extract_file_text.get_unstructured_api_key", return_value=None, diff --git a/backend/tests/integration/connector_job_tests/slack/test_permission_sync.py b/backend/tests/integration/connector_job_tests/slack/test_permission_sync.py index c8b28635a5..ba3cccbf60 100644 --- a/backend/tests/integration/connector_job_tests/slack/test_permission_sync.py +++ b/backend/tests/integration/connector_job_tests/slack/test_permission_sync.py @@ -3,6 +3,8 @@ from datetime import datetime from datetime import timezone from typing import Any +import pytest + from onyx.connectors.models import InputType from onyx.db.enums import AccessType from onyx.server.documents.models import DocumentSource @@ -23,6 +25,12 @@ from tests.integration.common_utils.vespa import vespa_fixture from tests.integration.connector_job_tests.slack.slack_api_utils import SlackManager +# NOTE(rkuo): it isn't yet clear if the reason these were previously xfail'd +# still exists. May need to xfail again if flaky (DAN-789) +@pytest.mark.skipif( + os.environ.get("ENABLE_PAID_ENTERPRISE_EDITION_FEATURES", "").lower() != "true", + reason="Permission tests are enterprise only", +) def test_slack_permission_sync( reset: None, vespa_client: vespa_fixture, @@ -218,6 +226,12 @@ def test_slack_permission_sync( assert private_message not in onyx_doc_message_strings +# NOTE(rkuo): it isn't yet clear if the reason these were previously xfail'd +# still exists. May need to xfail again if flaky (DAN-789) +@pytest.mark.skipif( + os.environ.get("ENABLE_PAID_ENTERPRISE_EDITION_FEATURES", "").lower() != "true", + reason="Permission tests are enterprise only", +) def test_slack_group_permission_sync( reset: None, vespa_client: vespa_fixture,