GraphQL: CreateGroup mutation

[JeffreyThiessen]

What does this PR do and why?

Describe in detail what your merge request does and why.
Fixes #787

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other pull requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

1. sign in to a user and load GraphiQL
2. Get information on a group to use as a parent group

query get_groups{
  groups(first:5){
    nodes{
      puid
      id
      name
      fullName
      path
      fullPath
    }
  }
}

3. Make a group under a parent group using puid

mutation make_group_with_parent_group_puid{
  createGroup(input:{
    name:"my group in a parent group",
    groupPuid:"INXT_GRP_AYJ2V6XLN2"
  }){
    group{
      puid
      id
      name
      fullName
      path
      fullPath
    },
    errors{
      path,
      message
    }
  }
}

4. Make a group using a parent group id

mutation make_group_with_parent_group_id{
  createGroup(input:{
    name:"my group 2 in a parent group",
    groupId:"gid://irida/Group/31f2ed96-82e1-41fa-82cc-e4b83d4e24bb"
  }){
    group{
      puid
      id
      name
      fullName
      path
      fullPath
    },
    errors{
      path,
      message
    }
  }
}

5. Try making a group without a parent group, and with a custom path, make sure that the path gets slugified

mutation make_group{
  createGroup(input:{
    name:"my new group",
    description:"my group desc",
    path:"My custom path"
  }){
    group{
      puid
      id
      name
      fullName
      path
      fullPath
    },
    errors{
      path,
      message
    }
  }
}

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

• I have evaluated the PR acceptance checklist for this PR.

[ChrisHuynh333]

Looks great!

[ksierks]

I'm not sure if it matters, but I think we slugify with '-' by default instead of '_'.
Works great!

[ericenns]

This is looking good, just a few comments on processing path argument.

[github-actions]

Code Metrics Report

Coverage	Test Execution Time
93.4%	7m57s

Code coverage of files in pull request scope (100.0%)

Files	Coverage
app/graphql/mutations/create_group.rb	100.0%
app/graphql/types/mutation_type.rb	100.0%

---

Reported by octocov

[ericenns]

Looks good to me!