Skip to content

Create iteration cadence mutation [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Alexandru Croitor requested to merge create-iteration-cadence-mutation into master

What does this MR do?

This MR adds GraphQL support to create iterations cadence within a given group. This is one MR out of a series of MRs to add support for iteration cadences.

More context on iterations and iteration cadences:

To test it locally:

  • With enabled feature flag:
 Feature.enable(:iteration_cadences)
  • Then run the mutation in graphiql: /-/graphql-explorer
    • with success:
      mutation {
        createIterationsCadence(input: {groupPath: "top-group/sub-group", title: "Second cadence", startDate: "01-02-2021", durationInWeeks: 2, iterationsInAdvance: 2, automatic: false, automatic: true}) {
          iterationsCadence {
            id      
          }
          errors
        }
      }
    • with errors
      mutation {
        createIterationsCadence(input: {groupPath: "top-group/sub-group", title: "Second cadence", startDate: "01-02-2021", automatic: false, automatic: true}) {
          iterationsCadence {
            id      
          }
          errors
        }
      }
  • With disabled feature flag:
     Feature.disable(:iterations_cadences)
    mutation {
       createIterationsCadence(input: {groupPath: "top-group/sub-group", title: "Second cadence", startDate: "01-02-2021", durationInWeeks: 2, iterationsInAdvance: 2, automatic:false, active:false}) {
        iterationsCadence {
          id      
        }
        errors
      }
    }
    • response
     {
      "data": {
        "createIterationsCadence": {
          "iterationsCadence": null,
          "errors": [
            "Operation not allowed"
          ]
        }
      }
    }

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

re #322741 (closed)

Edited by Alexandru Croitor

Merge request reports