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(builder): scaffold playwright #8109

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Sep 19, 2024

Background

We need to do more robust testing of our services and platforms. Its clear due to some major breakages that have been occuring

Changes 🏗️

Scaffolds out playwright and adds it to the CI

Testing 🔍

Note

Only for the new autogpt platform, currently in rnd/

  • Create from scratch and execute an agent with at least 3 blocks
  • Import an agent from file upload, and confirm it executes correctly
  • Upload agent to marketplace
  • Import an agent from marketplace and confirm it executes correctly
  • Edit an agent from monitor, and confirm it executes correctly

@ntindle ntindle requested review from a team as code owners September 19, 2024 20:29
@ntindle ntindle requested review from Swiftyos and Pwuts and removed request for a team September 19, 2024 20:29
@github-actions github-actions bot added the Builder AutoGPT Builder label Sep 19, 2024
Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 PR contains tests
🔒 No security concerns identified
⚡ Key issues to review

Removed Error Reporting
The Sentry error reporting dialog has been commented out, which may impact the ability to collect user feedback on errors.

Incomplete Configuration
The web server configuration is commented out, which might cause issues when running tests that require a local server.

Copy link

netlify bot commented Sep 19, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit feeaf90
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/66eca6c8bf2e7200086dd4fb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🆕 Needs initial review
Development

Successfully merging this pull request may close these issues.

1 participant