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(rnd): Add Agent Block sample/test input output for auto-generated unit testing for block #7310

Conversation

majdyz
Copy link
Contributor

@majdyz majdyz commented Jul 3, 2024

Background

Sample test input and output on the block can serve as documentation and auto-generated unit-testing code for the agent block.

What's within the scope of this change:
Adding the fields for block test (input, output, mocks), and its execution.

What's still outside the scope:
Handling of mock and stub for a block using sensitive credentials or network calls or 3rd-party connections.

Changes πŸ—οΈ

  • Add test_input and test_output for Block class
  • Add a unit test that executes all Block by using the provided test_input and test_output.

PR Quality Scorecard ✨

  • Have you used the PR description template?   +2 pts
  • Is your pull request atomic, focusing on a single change?   +5 pts
  • Have you linked the GitHub issue(s) that this PR addresses?   +5 pts
  • Have you documented your changes clearly and comprehensively?   +5 pts
  • Have you changed or added a feature?   -4 pts
    • Have you added/updated corresponding documentation?   +4 pts
    • Have you added/updated corresponding integration tests?   +5 pts
  • Have you changed the behavior of AutoGPT?   -5 pts
    • Have you also run agbenchmark to verify that these changes do not regress performance?   +10 pts

@github-actions github-actions bot added the size/l label Jul 3, 2024
Base automatically changed from zamilmajdy/agpt-758-populated-agent-blocks-for-reddit-marketing-analyzing-demo to master July 4, 2024 10:37
Copy link
Contributor

github-actions bot commented Jul 4, 2024

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot added conflicts Automatically applied to PRs with merge conflicts size/xl and removed size/l labels Jul 4, 2024
Copy link

netlify bot commented Jul 5, 2024

βœ… Deploy Preview for auto-gpt-docs canceled.

Name Link
πŸ”¨ Latest commit 2dbb68e
πŸ” Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/668cf7af658900000740240d

@majdyz majdyz force-pushed the zamilmajdy/agpt-759-add-blocks-test-input-ouput-field-for-auto-generated-unit branch from ecc667c to af642fe Compare July 5, 2024 07:00
@github-actions github-actions bot added size/l and removed size/xl conflicts Automatically applied to PRs with merge conflicts labels Jul 5, 2024
Copy link
Contributor

github-actions bot commented Jul 5, 2024

Conflicts have been resolved! πŸŽ‰ A maintainer will review the pull request shortly.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jul 5, 2024
Copy link
Contributor

github-actions bot commented Jul 5, 2024

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@majdyz majdyz requested a review from aarushik93 July 8, 2024 04:20
@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jul 8, 2024
Copy link
Contributor

github-actions bot commented Jul 8, 2024

Conflicts have been resolved! πŸŽ‰ A maintainer will review the pull request shortly.

Copy link
Contributor

github-actions bot commented Jul 8, 2024

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jul 8, 2024
@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jul 9, 2024
Copy link
Contributor

github-actions bot commented Jul 9, 2024

Conflicts have been resolved! πŸŽ‰ A maintainer will review the pull request shortly.

@majdyz majdyz enabled auto-merge (squash) July 9, 2024 09:13
@majdyz majdyz requested a review from Swiftyos July 9, 2024 10:05
@majdyz majdyz merged commit d9c9b22 into master Jul 9, 2024
18 of 19 checks passed
@majdyz majdyz deleted the zamilmajdy/agpt-759-add-blocks-test-input-ouput-field-for-auto-generated-unit branch July 9, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: βœ… Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants