Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/selenium e2e tests #65

Merged
merged 135 commits into from
Jul 11, 2024
Merged

Feat/selenium e2e tests #65

merged 135 commits into from
Jul 11, 2024

Conversation

dividor
Copy link
Contributor

@dividor dividor commented Jul 9, 2024

In this PR I retired the e2e tests patching chainlit as they ended up too complex and relied on too many patches.

The PR instead implements testing with Selenium, exactly replicating user interaction with the user interface running as part of the environment.

It includes:

  1. e2e tests github actions, using docker-compose to spin up DBs, chat environment, etc
  2. Test data for on-the-fly analysis
  3. Set of e2e tests

NOTE: This PR reverts docker-compose-github.yml for simplicity. It builds the main docker-compose.yml in prep for testing the Autogen team.

Future work:

  • The build takes a very long time, might consider a nightly build and puch, so tests just pull images
  • Promptflow container is built with docker build rather than docker-compose. Might want to add to main docker-compose and use profiles

@dividor dividor had a problem deploying to GitHub Actions 1 July 10, 2024 23:03 — with GitHub Actions Failure
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:03 — with GitHub Actions Inactive
@dividor dividor had a problem deploying to GitHub Actions 1 July 10, 2024 23:06 — with GitHub Actions Failure
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:06 — with GitHub Actions Inactive
@dividor dividor had a problem deploying to GitHub Actions 1 July 10, 2024 23:18 — with GitHub Actions Failure
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:18 — with GitHub Actions Inactive
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:20 — with GitHub Actions Inactive
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:34 — with GitHub Actions Inactive
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:42 — with GitHub Actions Inactive
@dividor dividor temporarily deployed to GitHub Actions 1 July 10, 2024 23:50 — with GitHub Actions Inactive
@dividor dividor had a problem deploying to GitHub Actions 1 July 10, 2024 23:50 — with GitHub Actions Failure
@dividor dividor temporarily deployed to GitHub Actions 1 July 11, 2024 00:03 — with GitHub Actions Inactive
@dividor dividor had a problem deploying to GitHub Actions 1 July 11, 2024 00:03 — with GitHub Actions Failure
@dividor dividor temporarily deployed to GitHub Actions 1 July 11, 2024 00:05 — with GitHub Actions Inactive
@dividor dividor had a problem deploying to GitHub Actions 1 July 11, 2024 00:05 — with GitHub Actions Failure
@dividor dividor merged commit 6c35800 into main Jul 11, 2024
3 of 4 checks passed
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