modulesync 1.2.0 → 2.1.0

data/README.md

@@ -169,7 +169,7 @@ msync hook activate
 ```
 
 If you have activated the hook but want to make changes to the configuration
-directory (such as changes to managed_modules.yml or modulesync.yml) without
+directory (such as changes to `managed_modules.yml` or `modulesync.yml`) without
 touching the modules, you can deactivate the hook.
 
 ```
@@ -185,17 +185,38 @@ GitLab automatically with the `--pr` CLI option.
 msync update --pr
 ```
 
-You must set the `GITHUB_TOKEN` or `GITLAB_TOKEN` environment variable
-for GitHub PRs or GitLab MRs to work. Additional options:
+In order for GitHub PRs or GitLab MRs to work, you must either provide
+the `GITHUB_TOKEN` or `GITLAB_TOKEN` environment variables,
+or set them per repository in `managed_modules.yml`, using the `github` or
+`gitlab` keys respectively.
+
+For GitHub Enterprise and self-hosted GitLab instances you also need to set the
+`GITHUB_BASE_URL` or `GITLAB_BASE_URL` environment variables, or specify the
+`base_url` parameter in `modulesync.yml`:
+
+```yaml
+---
+repo1:
+  github:
+    token: 'EXAMPLE_TOKEN'
+    base_url: 'https://api.github.com/'
+
+repo2:
+  gitlab:
+    token: 'EXAMPLE_TOKEN'
+    base_url: 'https://git.example.com/api/v4'
+```
+
+Then:
 
 * Set the PR/MR title with `--pr-title` or in `modulesync.yml` with the
   `pr_title` attribute.
 * Assign labels to the PR/MR with `--pr-labels` or in `modulesync.yml` with
   the `pr_labels` attribute. **NOTE:** `pr_labels` should be a list. When
   using the `--pr-labels` CLI option, you should use a comma separated list.
+* Set the target branch with `--pr-target-branch` or in `modulesync.yml` with
+  the `pr_target_branch` attribute.
 
-For GitHub Enterprise and self-hosted GitLab instances you need to set the
-`GITHUB_BASE_URL` or `GITLAB_BASE_URL` environment variable in addition.
 More details for GitHub:
 
 * Octokit [`api_endpoint`](https://github.com/octokit/octokit.rb#interacting-with-the-githubcom-apis-in-github-enterprise)
@@ -255,7 +276,9 @@ msync update -m "Commit message"
 Available parameters for modulesync.yml
 
 * `git_base` : The default URL to git clone from (Default: 'git@github.com:')
-* `namespace` : Namespace of the projects to manage (Default: 'puppetlabs')
+* `namespace` : Namespace of the projects to manage (Default: 'puppetlabs').
+  This value can be overridden in the module name (e.g. 'namespace/mod') or by
+  using the `namespace` key for the module in `managed_modules.yml`.
 * `branch` : Branch to push to (Default: 'master')
 * `remote_branch` : Remote branch to push to (Default: Same value as branch)
 * `message` : Commit message to apply to updated modules.

data/Rakefile

@@ -20,3 +20,27 @@ end
 
 task :test => %i[clean spec cucumber rubocop]
 task :default => %i[test]
+
+begin
+  require 'github_changelog_generator/task'
+  GitHubChangelogGenerator::RakeTask.new :changelog do |config|
+    config.header = "# Changelog\n\nAll notable changes to this project will be documented in this file."
+    config.exclude_labels = %w[duplicate question invalid wontfix wont-fix modulesync skip-changelog]
+    config.user = 'voxpupuli'
+    config.project = 'modulesync'
+    config.future_release = Gem::Specification.load("#{config.project}.gemspec").version
+  end
+
+  # Workaround for https://github.com/github-changelog-generator/github-changelog-generator/issues/715
+  require 'rbconfig'
+  if RbConfig::CONFIG['host_os'] =~ /linux/
+    task :changelog do
+      puts 'Fixing line endings...'
+      changelog_file = File.join(__dir__, 'CHANGELOG.md')
+      changelog_txt = File.read(changelog_file)
+      new_contents = changelog_txt.gsub(/\r\n/, "\n")
+      File.open(changelog_file, 'w') { |file| file.puts new_contents }
+    end
+  end
+rescue LoadError
+end

data/features/cli.feature

@@ -4,21 +4,26 @@ Feature: CLI
   Scenario: When passing no arguments to the msync command
     When I run `msync`
     And the output should match /Commands:/
+    Then the exit status should be 1
 
   Scenario: When passing invalid arguments to the msync update command
     When I run `msync update`
     And the output should match /No value provided for required option/
+    Then the exit status should be 1
 
   Scenario: When passing invalid arguments to the msync hook command
     When I run `msync hook`
     And the output should match /Commands:/
+    Then the exit status should be 1
 
-  Scenario: When running the help subcommand
+  Scenario: When running the help command
     When I run `msync help`
     And the output should match /Commands:/
+    Then the exit status should be 0
 
   Scenario: When overriding a setting from the config file on the command line
-    Given a file named "managed_modules.yml" with:
+    Given a puppet module "puppet-test" from "fakenamespace"
+    And a file named "managed_modules.yml" with:
       """
       ---
         - puppet-test
@@ -26,10 +31,13 @@ Feature: CLI
     And a file named "modulesync.yml" with:
       """
       ---
-        namespace: maestrodev
-        git_base: 'git@github.com:'
+      namespace: default
       """
+    And a git_base option appended to "modulesync.yml" for local tests
     And a directory named "moduleroot"
-    When I run `msync update --noop --git-base https://github.com/`
+    When I run `msync update --noop --namespace fakenamespace --branch command-line-branch`
     Then the exit status should be 0
-    And the output should not match /git@github.com:/
+    And the output should contain:
+      """
+      Creating new branch command-line-branch
+      """

data/features/hook.feature

@@ -5,8 +5,7 @@ Feature: hook
     Given a directory named ".git/hooks"
     When I run `msync hook activate`
     Then the exit status should be 0
-    Given I run `cat .git/hooks/pre-push`
-    Then the output should contain "bash"
+    And the file named ".git/hooks/pre-push" should contain "bash"
 
   Scenario: Deactivating a hook
     Given a file named ".git/hooks/pre-push" with:
@@ -21,8 +20,7 @@ Feature: hook
     Given a directory named ".git/hooks"
     When I run `msync hook activate -a '--foo bar --baz quux' -b master`
     Then the exit status should be 0
-    Given I run `cat .git/hooks/pre-push`
-    Then the output should match:
+    And the file named ".git/hooks/pre-push" should contain:
       """
-      "\$message" -n puppetlabs -b master --foo bar --baz quux
+      "$message" -n puppetlabs -b master --foo bar --baz quux
       """

data/features/step_definitions/git_steps.rb

@@ -1,47 +1,85 @@
-Given 'a mocked git configuration' do
-  steps %(
-    Given a mocked home directory
-    And I run `git config --global user.name Test`
-    And I run `git config --global user.email test@example.com`
-  )
-end
+require_relative '../../spec/helpers/faker/puppet_module_remote_repo'
 
-Given 'a remote module repository' do
+Given 'a basic setup with a puppet module {string} from {string}' do |name, namespace|
   steps %(
-    Given a directory named "sources"
-    And I run `git clone https://github.com/maestrodev/puppet-test sources/puppet-test`
+    Given a mocked git configuration
+    And a puppet module "#{name}" from "#{namespace}"
     And a file named "managed_modules.yml" with:
       """
       ---
-        - puppet-test
+        - #{name}
       """
+    And a file named "modulesync.yml" with:
+      """
+      ---
+      namespace: #{namespace}
+      """
+    And a git_base option appended to "modulesync.yml" for local tests
   )
-  write_file('modulesync.yml', <<-CONFIG)
----
-  namespace: sources
-  git_base: file://#{expand_path('.')}/
-  CONFIG
 end
 
-Given Regexp.new(/a remote module repository with "(.+?)" as the default branch/) do |branch|
+Given 'a mocked git configuration' do
   steps %(
-    Given a directory named "sources"
-    And I run `git clone --mirror https://github.com/maestrodev/puppet-test sources/puppet-test`
-    And a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
+    Given a mocked home directory
+    And I run `git config --global user.name Aruba`
+    And I run `git config --global user.email aruba@example.com`
   )
-  write_file('modulesync.yml', <<-CONFIG)
----
-  namespace: sources
-  git_base: file://#{expand_path('.')}/
-  CONFIG
-  cd('sources/puppet-test') do
-    steps %(
-      And I run `git branch -M master #{branch}`
-      And I run `git symbolic-ref HEAD refs/heads/#{branch}`
-    )
-  end
+end
+
+Given 'a puppet module {string} from {string}' do |name, namespace|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  pmrr.populate
+end
+
+Given 'a git_base option appended to "modulesync.yml" for local tests' do
+  File.write "#{Aruba.config.working_directory}/modulesync.yml", "\ngit_base: #{ModuleSync::Faker::PuppetModuleRemoteRepo.git_base}", mode: 'a'
+end
+
+Given 'the puppet module {string} from {string} is read-only' do |name, namespace|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  pmrr.read_only = true
+end
+
+Then 'the puppet module {string} from {string} should have no commits between {string} and {string}' do |name, namespace, commit1, commit2|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  expect(pmrr.commit_count_between(commit1, commit2)).to eq 0
+end
+
+Then 'the puppet module {string} from {string} should have( only) {int} commit(s) made by {string}' do |name, namespace, commit_count, author|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  expect(pmrr.commit_count_by(author)).to eq commit_count
+end
+
+Then 'the puppet module {string} from {string} should have( only) {int} commit(s) made by {string} in branch {string}' do |name, namespace, commit_count, author, branch|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  expect(pmrr.commit_count_by(author, branch)).to eq commit_count
+end
+
+Then 'the puppet module {string} from {string} should have no commits made by {string}' do |name, namespace, author|
+  step "the puppet module \"#{name}\" from \"#{namespace}\" should have 0 commits made by \"#{author}\""
+end
+
+Given 'the puppet module {string} from {string} has a file named {string} with:' do |name, namespace, filename, content|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  pmrr.add_file(filename, content)
+end
+
+Then 'the puppet module {string} from {string} should have a branch {string} with a file named {string} which contains:' do |name, namespace, branch, filename, content|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  expect(pmrr.read_file(filename, branch)).to include(content)
+end
+
+Given 'the puppet module {string} from {string} has the default branch named {string}' do |name, namespace, default_branch|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  pmrr.default_branch = default_branch
+end
+
+Then('the puppet module {string} from {string} should have a tag named {string}') do |name, namespace, tag|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  expect(pmrr.tags).to include(tag)
+end
+
+Then('the puppet module {string} from {string} should not have a tag named {string}') do |name, namespace, tag|
+  pmrr = ModuleSync::Faker::PuppetModuleRemoteRepo.new(name, namespace)
+  expect(pmrr.tags).not_to include(tag)
 end

data/features/support/env.rb

@@ -1,5 +1,9 @@
 require 'aruba/cucumber'
 
+require_relative '../../spec/helpers/faker'
+
+ModuleSync::Faker.working_directory = File.expand_path('faker', Aruba.config.working_directory)
+
 Before do
   @aruba_timeout_seconds = 5
 end

data/features/update.feature

@@ -2,17 +2,7 @@ Feature: update
   ModuleSync needs to update module boilerplate
 
   Scenario: Adding a new file
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -31,21 +21,11 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
 
   Scenario: Using skip_broken option and adding a new file to repo without write access
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: 'git@github.com:'
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And the puppet module "puppet-test" from "fakenamespace" is read-only
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -59,19 +39,11 @@ Feature: update
       """
     When I run `msync update -s -m "Add test"`
     Then the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Adding a new file to repo without write access
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: 'git@github.com:'
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And the puppet module "puppet-test" from "fakenamespace" is read-only
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -85,19 +57,10 @@ Feature: update
       """
     When I run `msync update -m "Add test" -r`
     Then the exit status should be 1
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Adding a new file, without the .erb suffix
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -120,21 +83,11 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Adding a new file using global values
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -153,21 +106,11 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Adding a new file overriding global values
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -189,21 +132,11 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Adding a new file ignoring global values
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -225,21 +158,11 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Adding a file that ERB can't parse
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -255,19 +178,10 @@ Feature: update
       """
     When I run `msync update --noop`
     Then the exit status should be 1
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Using skip_broken option with invalid files
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -283,19 +197,10 @@ Feature: update
       """
     When I run `msync update --noop -s`
     Then the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Using skip_broken and fail_on_warnings options with invalid files
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -311,18 +216,13 @@ Feature: update
       """
     When I run `msync update --noop --skip_broken --fail_on_warnings`
     Then the exit status should be 1
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Modifying an existing file
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And the puppet module "puppet-test" from "fakenamespace" has a file named "Gemfile" with:
       """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
+      source 'https://example.com'
       """
     And a file named "config_defaults.yml" with:
       """
@@ -342,15 +242,18 @@ Feature: update
       Files changed:
       +diff --git a/Gemfile b/Gemfile
       """
-    Given I run `cat modules/maestrodev/puppet-test/Gemfile`
-    Then the output should contain:
+    And the file named "modules/fakenamespace/puppet-test/Gemfile" should contain:
       """
       source 'https://somehost.com'
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Modifying an existing file and committing the change
-    Given a mocked git configuration
-    And a remote module repository
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And the puppet module "puppet-test" from "fakenamespace" has a file named "Gemfile" with:
+      """
+      source 'https://example.com'
+      """
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -364,30 +267,25 @@ Feature: update
       """
     When I run `msync update -m "Update Gemfile" -r test`
     Then the exit status should be 0
-    Given I cd to "sources/puppet-test"
-    And I run `git checkout test`
-    Then the file "Gemfile" should contain:
+    And the puppet module "puppet-test" from "fakenamespace" should have only 1 commit made by "Aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have 1 commit made by "Aruba" in branch "test"
+    And the puppet module "puppet-test" from "fakenamespace" should have a branch "test" with a file named "Gemfile" which contains:
       """
       source 'https://somehost.com'
       """
 
   Scenario: Setting an existing file to unmanaged
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And the puppet module "puppet-test" from "fakenamespace" has a file named "Gemfile" with:
       """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
+      source 'https://rubygems.org'
       """
     And a file named "config_defaults.yml" with:
       """
       ---
       Gemfile:
         unmanaged: true
+        gem_source: https://somehost.com
       """
     And a directory named "moduleroot"
     And a file named "moduleroot/Gemfile.erb" with:
@@ -402,27 +300,17 @@ Feature: update
       """
     And the output should match:
       """
-      Not managing Gemfile in puppet-test
+      Not managing 'Gemfile' in 'puppet-test'
       """
     And the exit status should be 0
-    Given I run `cat modules/maestrodev/puppet-test/Gemfile`
-    Then the output should contain:
+    And the file named "modules/fakenamespace/puppet-test/Gemfile" should contain:
       """
       source 'https://rubygems.org'
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Setting an existing file to deleted
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -434,6 +322,10 @@ Feature: update
       """
       source '<%= @configs['gem_source'] %>'
       """
+    And the puppet module "puppet-test" from "fakenamespace" has a file named "Gemfile" with:
+      """
+      source 'https://rubygems.org'
+      """
     When I run `msync update --noop`
     Then the output should match:
       """
@@ -442,19 +334,10 @@ Feature: update
       deleted file mode 100644
       """
     And the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Setting a non-existent file to deleted
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -464,19 +347,10 @@ Feature: update
     And a directory named "moduleroot"
     When I run `msync update -m 'deletes a file that doesnt exist!' -f puppet-test`
     And the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Setting a directory to unmanaged
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppetlabs-apache
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: puppetlabs
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-apache" from "puppetlabs"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -488,36 +362,26 @@ Feature: update
       """
       some spec_helper fud
       """
-    And a directory named "modules/puppetlabs/puppetlabs-apache/spec"
-    And a file named "modules/puppetlabs/puppetlabs-apache/spec/spec_helper.rb" with:
+    And a directory named "modules/puppetlabs/puppet-apache/spec"
+    And a file named "modules/puppetlabs/puppet-apache/spec/spec_helper.rb" with:
       """
       This is a fake spec_helper!
       """
     When I run `msync update --offline`
     Then the output should contain:
       """
-      Not managing spec/spec_helper.rb in puppetlabs-apache
+      Not managing 'spec/spec_helper.rb' in 'puppet-apache'
       """
     And the exit status should be 0
-    Given I run `cat modules/puppetlabs/puppetlabs-apache/spec/spec_helper.rb`
-    Then the output should contain:
+    And the file named "modules/puppetlabs/puppet-apache/spec/spec_helper.rb" should contain:
       """
       This is a fake spec_helper!
       """
     And the exit status should be 0
+    And the puppet module "puppet-apache" from "puppetlabs" should have no commits made by "Aruba"
 
   Scenario: Adding a new file in a new subdirectory
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -538,24 +402,14 @@ Feature: update
       Files added:
       spec/spec_helper.rb
       """
-    Given I run `cat modules/maestrodev/puppet-test/spec/spec_helper.rb`
-    Then the output should contain:
+    And the file named "modules/fakenamespace/puppet-test/spec/spec_helper.rb" should contain:
       """
       require 'puppetlabs_spec_helper/module_helper'
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Updating offline
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -572,19 +426,15 @@ Feature: update
     When I run `msync update --offline`
     Then the exit status should be 0
     And the output should not match /Files (changed|added|deleted):/
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Pulling a module that already exists in the modules directory
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - maestrodev/puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        git_base: https://github.com/
-      """
-    And a file named "config_defaults.yml" with:
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And a directory named "moduleroot"
+    When I run `msync update --message "First update run"`
+    Then the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
+    Given a file named "config_defaults.yml" with:
       """
       ---
       spec/spec_helper.rb:
@@ -597,20 +447,6 @@ Feature: update
         require '<%= required %>'
       <% end %>
       """
-    Given I run `git init modules/maestrodev/puppet-test`
-    Given a file named "modules/maestrodev/puppet-test/.git/config" with:
-      """
-      [core]
-          repositoryformatversion = 0
-          filemode = true
-          bare = false
-          logallrefupdates = true
-          ignorecase = true
-          precomposeunicode = true
-      [remote "origin"]
-          url = https://github.com/maestrodev/puppet-test.git
-          fetch = +refs/heads/*:refs/remotes/origin/*
-      """
     When I run `msync update --noop`
     Then the exit status should be 0
     And the output should match:
@@ -618,37 +454,24 @@ Feature: update
       Files added:
       spec/spec_helper.rb
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
-  Scenario: When running update with no changes
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+  Scenario: When running update without changes
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a directory named "moduleroot"
-    When I run `msync update --noop`
+    When I run `msync update --message "Running without changes"`
     Then the exit status should be 0
-    And the output should not match /diff/
+    And the stdout should contain "There were no changes in 'modules/fakenamespace/puppet-test'. Not committing."
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: When specifying configurations in managed_modules.yml
-    Given a file named "managed_modules.yml" with:
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And a file named "managed_modules.yml" with:
       """
       ---
         puppet-test:
           module_name: test
       """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -667,11 +490,14 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: When specifying configurations in managed_modules.yml and using a filter
-    Given a file named "managed_modules.yml" with:
+    Given a mocked git configuration
+    And a puppet module "puppet-test" from "fakenamespace"
+    And a puppet module "puppet-blacksmith" from "fakenamespace"
+    And a file named "managed_modules.yml" with:
       """
       ---
         puppet-blacksmith:
@@ -681,9 +507,9 @@ Feature: update
     And a file named "modulesync.yml" with:
       """
       ---
-        namespace: maestrodev
-        git_base: https://github.com/
+      namespace: fakenamespace
       """
+    And a git_base option appended to "modulesync.yml" for local tests
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -702,12 +528,15 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
-    And a directory named "modules/maestrodev/puppet-blacksmith" should not exist
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And a directory named "modules/fakenamespace/puppet-blacksmith" should not exist
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: When specifying configurations in managed_modules.yml and using a negative filter
-    Given a file named "managed_modules.yml" with:
+    Given a mocked git configuration
+    And a puppet module "puppet-test" from "fakenamespace"
+    And a puppet module "puppet-blacksmith" from "fakenamespace"
+    And a file named "managed_modules.yml" with:
       """
       ---
         puppet-blacksmith:
@@ -717,9 +546,9 @@ Feature: update
     And a file named "modulesync.yml" with:
       """
       ---
-        namespace: maestrodev
-        git_base: https://github.com/
+      namespace: fakenamespace
       """
+    And a git_base option appended to "modulesync.yml" for local tests
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -738,21 +567,12 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain "aruba"
-    And a directory named "modules/maestrodev/puppet-blacksmith" should not exist
+    And the file named "modules/fakenamespace/puppet-test/test" should contain "aruba"
+    And a directory named "modules/fakenamespace/puppet-blacksmith" should not exist
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Updating a module with a .sync.yml file
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - maestrodev/puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -775,21 +595,7 @@ Feature: update
       <%= @configs['global-to-overwrite'] %>
       <%= @configs['module-default'] %>
       """
-    Given I run `git init modules/maestrodev/puppet-test`
-    Given a file named "modules/maestrodev/puppet-test/.git/config" with:
-      """
-      [core]
-          repositoryformatversion = 0
-          filemode = true
-          bare = false
-          logallrefupdates = true
-          ignorecase = true
-          precomposeunicode = true
-      [remote "origin"]
-          url = https://github.com/maestrodev/puppet-test.git
-          fetch = +refs/heads/*:refs/remotes/origin/*
-      """
-    Given a file named "modules/maestrodev/puppet-test/.sync.yml" with:
+    And the puppet module "puppet-test" from "fakenamespace" has a file named ".sync.yml" with:
       """
       ---
       :global:
@@ -802,18 +608,21 @@ Feature: update
     Then the exit status should be 0
     And the output should match:
       """
-      Not managing spec/spec_helper.rb in puppet-test
+      Not managing 'spec/spec_helper.rb' in 'puppet-test'
       """
-    Given I run `cat modules/maestrodev/puppet-test/global-test.md`
-    Then the output should match:
+    And the file named "modules/fakenamespace/puppet-test/global-test.md" should contain:
       """
       some-default
       it-is-overwritten
       some-value
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Module with custom namespace
-    Given a file named "managed_modules.yml" with:
+    Given a mocked git configuration
+    And a puppet module "puppet-test" from "fakenamespace"
+    And a puppet module "puppet-lib-file_concat" from "electrical"
+    And a file named "managed_modules.yml" with:
       """
       ---
         - puppet-test
@@ -822,9 +631,9 @@ Feature: update
     And a file named "modulesync.yml" with:
       """
       ---
-        namespace: maestrodev
-        git_base: https://github.com/
+      namespace: fakenamespace
       """
+    And a git_base option appended to "modulesync.yml" for local tests
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -843,23 +652,13 @@ Feature: update
       Files added:
       test
       """
-    Given I run `cat modules/maestrodev/puppet-test/.git/config`
-    Then the output should contain "url = https://github.com/maestrodev/puppet-test.git"
-    Given I run `cat modules/electrical/puppet-lib-file_concat/.git/config`
-    Then the output should contain "url = https://github.com/electrical/puppet-lib-file_concat.git"
+    And the file named "modules/fakenamespace/puppet-test/.git/config" should match /^\s+url = .*fakenamespace.puppet-test$/
+    And the file named "modules/electrical/puppet-lib-file_concat/.git/config" should match /^\s+url = .*electrical.puppet-lib-file_concat$/
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
+    And the puppet module "puppet-lib-file_concat" from "electrical" should have no commits made by "Aruba"
 
   Scenario: Modifying an existing file with values exposed by the module
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -868,7 +667,12 @@ Feature: update
     And a directory named "moduleroot"
     And a file named "moduleroot/README.md.erb" with:
       """
-      echo '<%= @configs[:git_base] + @configs[:namespace] %>'
+      module: <%= @configs[:puppet_module] %>
+      namespace: <%= @configs[:namespace] %>
+      """
+    And the puppet module "puppet-test" from "fakenamespace" has a file named "README.md" with:
+      """
+      Hello world!
       """
     When I run `msync update --noop`
     Then the exit status should be 0
@@ -877,15 +681,15 @@ Feature: update
       Files changed:
       +diff --git a/README.md b/README.md
       """
-    Given I run `cat modules/maestrodev/puppet-test/README.md`
-    Then the output should contain:
+    And the file named "modules/fakenamespace/puppet-test/README.md" should contain:
       """
-      echo 'https://github.com/maestrodev'
+      module: puppet-test
+      namespace: fakenamespace
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Running the same update twice and pushing to a remote branch
-    Given a mocked git configuration
-    And a remote module repository
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -899,15 +703,41 @@ Feature: update
       """
     When I run `msync update -m "Update Gemfile" -r test`
     Then the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have only 1 commit made by "Aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have 1 commit made by "Aruba" in branch "test"
     Given I remove the directory "modules"
     When I run `msync update -m "Update Gemfile" -r test`
     Then the exit status should be 0
     Then the output should not contain "error"
     Then the output should not contain "rejected"
+    And the puppet module "puppet-test" from "fakenamespace" should have only 1 commit made by "Aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have 1 commit made by "Aruba" in branch "test"
+
+  Scenario: Creating a GitHub PR with an update
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And a directory named "moduleroot"
+    And I set the environment variables to:
+      | variable     | value  |
+      | GITHUB_TOKEN | foobar |
+    When I run `msync update --noop --branch managed_update --pr`
+    Then the output should contain "Would submit PR "
+    And the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
+
+  Scenario: Creating a GitLab MR with an update
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And a directory named "moduleroot"
+    And I set the environment variables to:
+      | variable     | value  |
+      | GITLAB_TOKEN | foobar |
+    When I run `msync update --noop --branch managed_update --pr`
+    Then the output should contain "Would submit MR "
+    And the exit status should be 0
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"
 
   Scenario: Repository with a default branch other than master
-    Given a mocked git configuration
-    And a remote module repository with "develop" as the default branch
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
+    And the puppet module "puppet-test" from "fakenamespace" has the default branch named "develop"
     And a file named "config_defaults.yml" with:
       """
       ---
@@ -921,24 +751,16 @@ Feature: update
       """
     When I run `msync update -m "Update Gemfile"`
     Then the exit status should be 0
-    Then the output should contain "Using repository's default branch: develop"
+    And the output should contain "Using repository's default branch: develop"
+    And the puppet module "puppet-test" from "fakenamespace" should have only 1 commit made by "Aruba"
+    And the puppet module "puppet-test" from "fakenamespace" should have 1 commit made by "Aruba" in branch "develop"
 
-  Scenario: Adding a new file from a template using meta data
+  Scenario: Adding a new file from a template using metadata
+    Given a basic setup with a puppet module "puppet-test" from "fakenamespace"
     And a file named "config_defaults.yml" with:
       """
       ---
       """
-    Given a file named "managed_modules.yml" with:
-      """
-      ---
-        - puppet-test
-      """
-    And a file named "modulesync.yml" with:
-      """
-      ---
-        namespace: maestrodev
-        git_base: https://github.com/
-      """
     And a directory named "moduleroot"
     And a file named "moduleroot/test.erb" with:
       """
@@ -948,10 +770,10 @@ Feature: update
       """
     When I run `msync update --noop`
     Then the exit status should be 0
-    Given I run `cat modules/maestrodev/puppet-test/test`
-    Then the output should contain:
+    And the file named "modules/fakenamespace/puppet-test/test" should contain:
       """
       module: puppet-test
-      target: modules/maestrodev/puppet-test/test
-      workdir: modules/maestrodev/puppet-test
+      target: modules/fakenamespace/puppet-test/test
+      workdir: modules/fakenamespace/puppet-test
       """
+    And the puppet module "puppet-test" from "fakenamespace" should have no commits made by "Aruba"