highperfocused/danswer
1
0
Fork 0
mirror of https://github.com/danswer-ai/danswer.git synced 2025-06-28 17:01:10 +02:00
Code Issues Packages Projects Releases Wiki Activity

slack permission tests are enterprise only (#4463)

Browse Source 
* slack permission tests are enterprise only

* xfail highspot connector

* test is broken

---------

Co-authored-by: Richard Kuo (Onyx) <rkuo@onyx.app>
This commit is contained in:
rkuo-danswer 2025-04-06 15:23:17 -07:00 committed by GitHub
parent 4dc382b571
commit bb73bb224a
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
backend/tests/daily/connectors/highspot/test_highspot_connector.py
View File

@ -106,6 +106,9 @@ def test_highspot_connector_slim(
assert len(all_slim_doc_ids) > 0 assert len(all_slim_doc_ids) > 0
@pytest.mark.xfail(
reason="failing, needs fix",
)
@patch( @patch(
"onyx.file_processing.extract_file_text.get_unstructured_api_key", "onyx.file_processing.extract_file_text.get_unstructured_api_key",
return_value=None, return_value=None,

14
backend/tests/integration/connector_job_tests/slack/test_permission_sync.py
View File

@ -3,6 +3,8 @@ 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
@ -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 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,
@ -218,6 +226,12 @@ 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,