Update graphql specs in preparation of gem upgrade
What does this MR do and why?
This updates some graphql specs to pass a Query::Context
object rather than a hash to conform to changes made in a recent update to the graphql
gem.
It is the first in what will be a series of MRs to address #473155 (closed)
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #473155 (closed)
Merge request reports
Activity
changed milestone to %17.3
assigned to @carlad-gl
added workflowin review label and removed workflowready for development label
added pipelinetier-1 label
mentioned in issue Taucher2003/reporting#630 (closed)
- Resolved by James Nutt
requested review from @SamWord
added pipeline:run-all-e2e label
Reviewer roulette
Category Reviewer Maintainer backend @ivaneG
(UTC+1, 1 hour behind author)
@jwoodwardgl
(UTC+1, 1 hour behind author)
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by Ghost UserE2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for aea2fe5bexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Verify | 43 | 0 | 2 | 0 | 45 | ✅ | | Package | 19 | 0 | 12 | 0 | 31 | ✅ | | Create | 127 | 0 | 12 | 0 | 139 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Data Stores | 30 | 0 | 1 | 0 | 31 | ✅ | | Plan | 67 | 0 | 2 | 0 | 69 | ✅ | | Govern | 69 | 0 | 0 | 0 | 69 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Secure | 1 | 0 | 0 | 0 | 1 | ✅ | | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | | Fulfillment | 1 | 0 | 0 | 0 | 1 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 372 | 0 | 30 | 0 | 402 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for aea2fe5bexpand test summary
+---------------------------------------------------------------------+ | suites summary | +----------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +----------------+--------+--------+---------+-------+-------+--------+ | Plan | 80 | 0 | 11 | 0 | 91 | ✅ | | Systems | 7 | 0 | 0 | 0 | 7 | ✅ | | Package | 31 | 0 | 14 | 6 | 45 | ✅ | | Data Stores | 46 | 0 | 11 | 0 | 57 | ✅ | | Govern | 105 | 0 | 8 | 2 | 113 | ✅ | | GitLab Metrics | 2 | 0 | 1 | 0 | 3 | ✅ | | Fulfillment | 4 | 0 | 7 | 0 | 11 | ✅ | | Manage | 30 | 0 | 15 | 2 | 45 | ✅ | | Analytics | 3 | 0 | 0 | 0 | 3 | ✅ | | Create | 417 | 0 | 51 | 27 | 468 | ✅ | | Verify | 51 | 0 | 15 | 2 | 66 | ✅ | | Configure | 1 | 0 | 3 | 0 | 4 | ✅ | | Monitor | 12 | 0 | 13 | 0 | 25 | ✅ | | Ai-powered | 1 | 0 | 2 | 0 | 3 | ✅ | | Secure | 4 | 0 | 2 | 0 | 6 | ✅ | | Release | 5 | 0 | 1 | 0 | 6 | ✅ | | Growth | 0 | 0 | 2 | 0 | 2 | ➖ | | ModelOps | 0 | 0 | 1 | 0 | 1 | ➖ | +----------------+--------+--------+---------+-------+-------+--------+ | Total | 799 | 0 | 157 | 39 | 956 | ✅ | +----------------+--------+--------+---------+-------+-------+--------+
Edited by Ghost Userrequested review from @ivaneG
- Resolved by Sam Word
- Resolved by Carla Drago
- Resolved by Carla Drago
- Resolved by Carla Drago
- Resolved by Carla Drago