-
Notifications
You must be signed in to change notification settings - Fork 27
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
Test Apis Workflow #480
base: master
Are you sure you want to change the base?
Test Apis Workflow #480
Conversation
pranavv0
commented
May 1, 2023
•
edited
Loading
edited
- Workflow for Test Automation
- Script of Pytest
- Dockerfile
Can one of the admins verify this patch? |
"username": "<rmq user>", | ||
"password": "<rmq password>", | ||
"host": "<rmq host>", | ||
"port": "<rmq amq port>", | ||
"hport": "<rmq https port>", | ||
"hport": "<rmq https port, required only if createVhost is true >", |
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.
use proper config fields, should be clear by just reading it
- use
httpsPort
instead ofhport
- use
amqpsPort
instead ofport
--> also its better to connect and test over amqps .
response = requests.put(url, auth=(username, password)) | ||
if createVhost: | ||
url = f'https://{host}:{hport}/api/vhosts/{vhost}' | ||
response = requests.put(url, auth=(username, password)) | ||
|
||
# rabbitmq amqp connection details | ||
connection = pika.BlockingConnection( | ||
pika.URLParameters(f'amqp://{username}:{password}@{host}:{port}/{vhost}')) |
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.
its better to connect and test over amqps
@@ -1,9 +1,10 @@ | |||
{ | |||
"createVhost" : "True or False, if vhost is required to create", |
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.
should n't the True or False be without the quotes
Also update readme of the test. There is only one test now for lip |