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

Adjust test_prompt_generator and add test report generation #1609

Merged
merged 1 commit into from
Apr 15, 2023

Conversation

younessZMZ
Copy link
Contributor

Background

Add coverage report generation.
Adjust test_prompt_generator file.

Changes

  • Remove unused imports.
  • Add docstrings.
  • Correct checking the presence of key-value pair of the command args dictionary in the generated prompt_string.
  • Use setUpClass because we need one time execution of the setup function instead of setUp which is executed each time before each test function.
  • Use covearge to load tests and adjust test location.
  • Generate test report with missing tests to help in test integration.

Documentation

Test Plan

Tests impacted are passed before opening the PR.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.

@richbeales richbeales merged commit af46c04 into Significant-Gravitas:master Apr 15, 2023
@nponeccop nponeccop mentioned this pull request Apr 15, 2023
1 task
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
Adjust test_prompt_generator and add test report generation
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.

3 participants