Skip to content

Conversation

hagen-danswer
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Oct 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
internal-search ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 6:49pm

@hagen-danswer hagen-danswer changed the title Implement source testing framework Implement source testing framework + Slack Oct 1, 2024

from tests.integration.connector_job_tests.slack.slack_api_utils import SlackManager


Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this makes sure that if the test fails, the slack state gets reset

) -> DATestUser:
if name is None:
name = f"test{str(uuid4())}"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added email so exact email can be specified

f"Waiting for CC pruning to complete. elapsed={elapsed:.2f} timeout={timeout}"
)
time.sleep(5)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this tracks sync progress

if not verify_deleted:
raise ValueError(f"CC pair {cc_pair.id} not found")

@staticmethod
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this lets us run the connector on demand


logger = setup_logger()


Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved this into the celery check


logger = setup_logger()


Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved this here to filter without creating a task

register_time=last_sync_task.register_time,
)


Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a way to start a sync as well as check on sync progress.

This is reusable on the frontend as well if necessary

"group_join",
"group_leave",
"group_archive",
"group_unarchive",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is necessary to make the tests work. i also dont think this will change the quality of the results but im not sure.

@hagen-danswer hagen-danswer added this pull request to the merge queue Oct 2, 2024
Merged via the queue into main with commit c208860 Oct 2, 2024
7 checks passed
@hagen-danswer hagen-danswer deleted the implement-source-testing-framework branch October 17, 2024 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants