Skip to content

Add tests to Consul dispatcher

Richard Clamp requested to merge rc-add-consul-test-with-fixup into master

What does this MR do?

Adds a missing test to the gitlab-ctl consul <foo> <bar> dispatch helper and refactors a little to avoid namespace conficts.

Checklist

See Definition of done.

  • Change added to CHANGELOG.md. Not applicable for Documentation changes and minor changes.
  • Documentation created/updated
  • Tests added
  • Integration tests added to GitLab QA, if applicable
  • MR targeting master branch
  • MR has a green pipeline on GitLab.com

Reviewer Checklist

In addition to above, reviewer must:

  • Pipeline is green on dev.gitlab.org if the change is not touching documentation or internal cookbooks

References

Splitting out from !2230 (closed), it's an indepdent refactor

Edited by Richard Clamp

Merge request reports