-
Notifications
You must be signed in to change notification settings - Fork 1
[CAI-608] Chatbot test refactor #1727
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
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 76a2b2b The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
apps/chatbot/src/app/test_main.py
Outdated
|
||
|
||
@mock_aws | ||
def test_get_healthz(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
def test_get_healthz() -> None:
....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Branch is not up to date with base branch@batdevis it seems this Pull Request is not updated with base branch. |
* settings * changeset * use AWS_SESSION from settings * chatbot-evaluate settings
Jira Pull Request LinkThis Pull Request refers to the following Jira issue CAI-608 |
This PR exceeds the recommended size of 800 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
List of Changes
Motivation and Context
We want to test locally without use real AWS environment.
How Has This Been Tested?
from
apps/chatbot
runScreenshots (if appropriate):
Types of changes
Checklist: