Revert "slack permission tests are enterprise only"

This reverts commit 056a83493f05414c882b875b10643f4ef04d0074.
This commit is contained in:
Richard Kuo (Onyx) 2025-04-06 14:24:42 -07:00
parent 056a83493f
commit 4dc382b571

View File

@ -3,8 +3,6 @@ from datetime import datetime
from datetime import timezone from datetime import timezone
from typing import Any from typing import Any
import pytest
from onyx.connectors.models import InputType from onyx.connectors.models import InputType
from onyx.db.enums import AccessType from onyx.db.enums import AccessType
from onyx.server.documents.models import DocumentSource from onyx.server.documents.models import DocumentSource
@ -25,12 +23,6 @@ from tests.integration.common_utils.vespa import vespa_fixture
from tests.integration.connector_job_tests.slack.slack_api_utils import SlackManager 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( def test_slack_permission_sync(
reset: None, reset: None,
vespa_client: vespa_fixture, vespa_client: vespa_fixture,
@ -226,12 +218,6 @@ def test_slack_permission_sync(
assert private_message not in onyx_doc_message_strings 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( def test_slack_group_permission_sync(
reset: None, reset: None,
vespa_client: vespa_fixture, vespa_client: vespa_fixture,