chore: Add operation names to dashboard graphql queries for better lo…

…gging (#2776) * chore: Add operation names to dashboard graphql queries for better logging * test: Fix graphql test endpoint * test: update golden files
infracost · Dec 5, 2023 · 77a3dbc · 77a3dbc
1 parent ca4bf75
commit 77a3dbc
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/cmd/infracost/comment_github_test.go b/cmd/infracost/comment_github_test.go
@@ -501,7 +501,7 @@ func governanceTestEndpoint(garr governanceAddRunResponse) *httptest.Server {
  bodyBytes, _ := io.ReadAll(r.Body)
  graphqlQuery := string(bodyBytes)
 
- if strings.Contains(graphqlQuery, "mutation($run: RunInput!, $commentFormat: CommentFormat!)") {
+ if strings.Contains(graphqlQuery, "mutation AddRun($run: RunInput!, $commentFormat: CommentFormat!)") {
  guardrailJson, _ := json.Marshal(garr)
 
  fmt.Fprintf(w, `[{"data": {"addRun":

diff --git a/...breakdown_with_policy_data_upload_hcl/breakdown_with_policy_data_upload_hcl-upload.golden b/...breakdown_with_policy_data_upload_hcl/breakdown_with_policy_data_upload_hcl-upload.golden
@@ -1,7 +1,7 @@
 policyResourceAllowList:
 [
  {
- "query": "\n\t\tquery {\n\t\t\tpolicyResourceAllowList {\n\t\t\t\tresourceType\n allowed\n\t\t\t}\n\t\t}\n\t",
+ "query": "\n\t\tquery GetPolicyResourceAllowList{\n\t\t\tpolicyResourceAllowList {\n\t\t\t\tresourceType\n allowed\n\t\t\t}\n\t\t}\n\t",
  "variables": {}
  }
 ]

diff --git a/...th_policy_data_upload_plan_json/breakdown_with_policy_data_upload_plan_json-upload.golden b/...th_policy_data_upload_plan_json/breakdown_with_policy_data_upload_plan_json-upload.golden
@@ -1,7 +1,7 @@
 policyResourceAllowList:
 [
  {
- "query": "\n\t\tquery {\n\t\t\tpolicyResourceAllowList {\n\t\t\t\tresourceType\n allowed\n\t\t\t}\n\t\t}\n\t",
+ "query": "\n\t\tquery GetPolicyResourceAllowList{\n\t\t\tpolicyResourceAllowList {\n\t\t\t\tresourceType\n allowed\n\t\t\t}\n\t\t}\n\t",
  "variables": {}
  }
 ]

diff --git a/..._policy_data_upload_terragrunt/breakdown_with_policy_data_upload_terragrunt-upload.golden b/..._policy_data_upload_terragrunt/breakdown_with_policy_data_upload_terragrunt-upload.golden
@@ -1,7 +1,7 @@
 policyResourceAllowList:
 [
  {
- "query": "\n\t\tquery {\n\t\t\tpolicyResourceAllowList {\n\t\t\t\tresourceType\n allowed\n\t\t\t}\n\t\t}\n\t",
+ "query": "\n\t\tquery GetPolicyResourceAllowList{\n\t\t\tpolicyResourceAllowList {\n\t\t\t\tresourceType\n allowed\n\t\t\t}\n\t\t}\n\t",
  "variables": {}
  }
 ]

diff --git a/internal/apiclient/dashboard.go b/internal/apiclient/dashboard.go
@@ -139,7 +139,7 @@ func (c *DashboardAPIClient) AddRun(ctx *config.RunContext, out output.Root, com
  }
 
  q := `
- mutation($run: RunInput!, $commentFormat: CommentFormat!) {
+ mutation AddRun($run: RunInput!, $commentFormat: CommentFormat!) {
  addRun(run: $run) {
  id
  shareUrl

diff --git a/internal/apiclient/policy.go b/internal/apiclient/policy.go
@@ -55,7 +55,7 @@ func (c *PolicyAPIClient) CheckPolicies(ctx *config.RunContext, out output.Root,
  }
 
  q := `
- query($run: RunInput!, $onlyDiff: Boolean) {
+ query EvaluatePolicies($run: RunInput!, $onlyDiff: Boolean) {
  evaluatePolicies(run: $run, onlyDiff: $onlyDiff) {
  tagPolicyResults {
  name
@@ -395,7 +395,7 @@ func (c *PolicyAPIClient) fetchAllowList() error {
 
 func (c *PolicyAPIClient) getPolicyResourceAllowList() (map[string]allowList, error) {
  q := `
- query {
+ query GetPolicyResourceAllowList{
  policyResourceAllowList {
  resourceType
  allowed