Load Test - Central Registry "Search and Admit" Workflow #328
Conversation
| def _(parser): | ||
| parser.add_argument("--domain", help="CommCare domain", required=True, env_var="COMMCARE_DOMAIN") | ||
| parser.add_argument("--app-id", help="CommCare app id", required=True, env_var="COMMCARE_APP_ID") | ||
| parser.add_argument("--app-config", help="Configuration of CommCare app", required=True) |
There was a problem hiding this comment.
I was wondering about this yesterday - if this config doesn't change then we might as well just have it as a constant in python
There was a problem hiding this comment.
I was about to agree, but Woody brought up a scenario today where the app config would change. He wants to run the same script on a slightly different variation of the same app where the case search title is different. This can reuse everything with a slightly different app-config file. He was referring to a different app, but the same potential scenario is applicable here
|
|
||
| @events.init_command_line_parser.add_listener | ||
| def _(parser): | ||
| parser.add_argument("--domain", help="CommCare domain", required=True, env_var="COMMCARE_DOMAIN") |
There was a problem hiding this comment.
I know I suggested making these command line args originally but having worked on the bed tracker one I see how specific these are to the app and domain and that having to provide all the config via the command line is a bit painful.
for the bed tracker I reverted to using a yaml config file (which was the original approach).
I'm not suggesting you change this now, just making a note.
| logging.error("Error loading cases to select: %s", e) | ||
| raise InterruptTaskSet from e | ||
|
|
||
| class LoginCommCareHQWithUniqueUsers(HttpUser): |
There was a problem hiding this comment.
Considering that the users are basically exactly the same for the different tests I wonder if we shouldn't create a base class.
There was a problem hiding this comment.
Good idea, a1ecf7c
There was a problem hiding this comment.
Was it intentional to add this file? it's got a strange name prefix (~$)
There was a problem hiding this comment.
oops, was accidental. It was a temporary file excel makes when i have it open e40ac8b
corrected formatting
Output in relation to this doc.
Jira Ticket
This PR creates a load test script that captures the below workflow on the Central Registry app: