Filter by subgroup name before determining if our group has a subgroup

See merge request gitlab-org/gitlab-ce!15969

Only keep screenshots from last run for QA specs

See merge request gitlab-org/gitlab-ce!15967

We assumed that the QA sandbox group would continually be wiped and
re-created, and thus never have enough subgroups that the one we just
created during a new run wouldn't appear on the first page.

But we know what they say about assumptions.

Add cop to make sure we don't use ivar in a module

See merge request gitlab-org/gitlab-ce!12800

* upstream/master: (671 commits)
  Make rubocop happy
  Use guard clause
  Improve language
  Prettify
  Use temp branch
  Pass info about who started the job and which job triggered it
  Docs: add indexes for monitoring and performance monitoring
  clearer-documentation-on-inline-diffs
  Add docs for commit diff discussion in merge requests
  sorting for tags api
  Clear BatchLoader after each spec to prevent holding onto records longer than necessary
  Include project in BatchLoader key to prevent returning blobs for the wrong project
  moved lfs_blob_ids method into ExtractsPath module
  Converted JS modules into exported modules
  spec fixes
  Bump gitlab-shell version to 5.10.3
  Clear caches before updating MR diffs
  Use new Ruby version 2.4 in GitLab QA images
  moved lfs blob fetch from extractspath file
  Update GitLab QA dependencies
  ...