-
Notifications
You must be signed in to change notification settings - Fork 31
🎨 Align map load test with ui workflow #8417
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: master
Are you sure you want to change the base?
🎨 Align map load test with ui workflow #8417
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8417 +/- ##
==========================================
+ Coverage 87.89% 87.91% +0.01%
==========================================
Files 1953 1953
Lines 76045 76045
Branches 1341 1341
==========================================
+ Hits 66843 66855 +12
+ Misses 8801 8789 -12
Partials 401 401
*This pull request uses carry forward flags. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
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.
thx
items = response.json().get("items", []) | ||
statuses = [item.get("status", {}) for item in items] | ||
all_job_statuses.extend( | ||
[status.get("status", None) for status in statuses if status] |
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.
MINOR: i really find strange when we embed the same keys twice ... for instance
{
"status":
{
"status": ...
}
I wonder why we do this? :-)
🧪 CI InsightsHere's what we observed from your CI run for b86d506. 🟢 All jobs passed!But CI Insights is watching 👀 |
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.
Thanks
What do these changes do?
GET /v0/function_jobs
instead ofGET /v0/function_job_collections/{function_job_collection_id}/status
.Related issue/s
How to test
Dev-ops