git_duplicator 0.0.1 → 1.0.0

data/spec/git_duplicator/helpers/authorization_header_spec.rb

@@ -0,0 +1,29 @@
+require_relative '../../helper'
+
+module GitDuplicator
+  module Helpers
+    describe AuthorizationHeader do
+      let(:credentials) { { username: 'user', password: 'password' } }
+      let(:request) { { method: :post, url: 'onetwo' } }
+      let(:header) { described_class.new(credentials, request) }
+
+      describe '#basic_authenticaion_header' do
+        it 'should have the right value' do
+          result = header.send :basic_authenticaion_header
+          encode64 = Base64.encode64("#{credentials[:username]}" \
+                                     ":#{credentials[:password]}")
+          expect(result).to eq("Basic #{encode64}")
+        end
+      end
+
+      describe '#basic_authenticaion_header' do
+        it 'should have the right value' do
+          result = header.send :base64_username_password
+          encode64 = Base64.encode64("#{credentials[:username]}" \
+                                     ":#{credentials[:password]}")
+          expect(result).to eq(encode64)
+        end
+      end
+    end
+  end
+end

data/spec/git_duplicator/repository/repository_spec.rb

@@ -1,47 +1,95 @@
 require_relative '../../helper'
 
-describe GitDuplicator::Repository do
-  let(:path) { 'somepath' }
-  let(:name) { 'something' }
-  let(:url) { 'someurl' }
-  let(:to_url) { 'someurl' }
-  let(:repo) { described_class.new(name, url) }
-
-  describe '#repo=' do
-    it 'assignes the attribute in case of Git::Base type' do
-      val = Git::Base.new
-      repo.repo = val
-      expect(repo.repo).to eq(val)
-    end
+module GitDuplicator
+  describe Repository do
+    let(:path) { 'somepath' }
+    let(:name) { 'something' }
+    let(:url) { 'someurl' }
+    let(:to_url) { 'someurl' }
+    let(:repo) { described_class.new(name, url) }
+
+    # describe '#working_directory=' do
+    #   it 'assignes the attribute' do
+    #     val = 'somepath'
+    #     repo.working_directory = val
+    #     expect(repo.working_directory).to eq(val)
+    #   end
+
+    #   it 'assignes the repo attribute' do
+    #     val = 'somepath'
+    #     repo.working_directory = val
+    #     expect(repo.working_directory).to eq(val)
+    #   end
 
-    it 'raise ArgumentError in case the attribute type is not Git::Base' do
-      expect { repo.repo = 1 }.to raise_error(TypeError)
+    # end
+
+    describe '#bare_clone' do
+      it 'clones a repo' do
+        expect(Git).to receive(:clone).with(url, name, path: path, bare: true)
+        repo.bare_clone(path)
+      end
+
+      it 'raises an exception in case of failure' do
+        expect { repo.bare_clone('') }
+        .to raise_error(GitDuplicator::RepositoryCloningError)
+      end
     end
-  end
 
-  describe '#bare_clone' do
-    before(:each) { allow(repo).to receive(:repo=) }
-    it 'clones a repo' do
-      expect(Git).to receive(:clone).with(url, name, path: path, bare: true)
-      repo.bare_clone(path)
+    describe '#mirror_clone' do
+      it 'clones a repo' do
+        allow(Git).to receive(:bare)
+        wd = File.join(path, repo.name)
+        expect(repo).to receive(:command)
+        .with('clone', '--mirror', repo.url, wd)
+        repo.mirror_clone(path)
+      end
+
+      it 'raises an exception in case of failure' do
+        expect { repo.mirror_clone('') }
+        .to raise_error(GitDuplicator::RepositoryCloningError)
+      end
     end
 
-    it 'raises an exception in case of failure' do
-      expect { repo.bare_clone('') }
-      .to raise_error(GitDuplicator::RepositoryCloningError)
+    describe '#mirror' do
+      it 'mirrors a repo' do
+        repo.send(:repo=, Git::Base.new)
+        expect(repo.send(:repo)).to receive(:push).with(to_url, '--mirror')
+        repo.mirror(to_url)
+      end
+
+      it 'raises an exception in case of failure' do
+        expect { repo.mirror('') }
+        .to raise_error(GitDuplicator::RepositoryMirorringError)
+      end
     end
-  end
 
-  describe '#mirror' do
-    it 'mirrors a repo' do
-      repo.repo = Git::Base.new
-      expect(repo.repo).to receive(:push).with(to_url, '--mirror')
-      repo.mirror(to_url)
+    describe '#update_mirrored' do
+
+      it 'fetchs and pushes data' do
+        repo.send(:repo=, Git::Base.new)
+        expect(repo).to receive(:command).with('fetch', '-p', 'origin')
+        expect(repo).to receive(:command).with('push', '--mirror')
+        repo.update_mirrored
+      end
+
+      it 'raises an exception in case of failure' do
+        expect { repo.update_mirrored }
+        .to raise_error(GitDuplicator::RepositoryMirorredUpdatingError)
+      end
     end
 
-    it 'raises an exception in case of failure' do
-      expect { repo.mirror('') }
-      .to raise_error(GitDuplicator::RepositoryMirorringError)
+    describe '#set_mirrored_remote' do
+      it 'sets mirrored as remote' do
+        repo.send(:repo=, Git::Base.new)
+        expect(repo).to receive(:command)
+        .with('remote', 'set-url', '--push', 'origin', to_url)
+        repo.set_mirrored_remote(to_url)
+      end
+
+      it 'raises an exception in case of failure' do
+        expect { repo.set_mirrored_remote('') }
+        .to raise_error(GitDuplicator::RepositorySettingRemoteError)
+      end
     end
   end
 end

data/spec/git_duplicator/repository/service_repository_spec.rb

@@ -1,25 +1,27 @@
 require_relative '../../helper'
 
-describe GitDuplicator::ServiceRepository do
-  let(:name) { 'something' }
-  let(:owner) { 'someone' }
-  let(:repo) { described_class.new(name, owner) }
+module GitDuplicator
+  describe ServiceRepository do
+    let(:name) { 'something' }
+    let(:owner) { 'someone' }
+    let(:repo) { described_class.new(name, owner) }
 
-  describe '#delete' do
-    it 'raises an exception in case of not defined' do
-      expect { repo.delete }.to raise_error(NotImplementedError)
+    describe '#delete' do
+      it 'raises an exception in case of not defined' do
+        expect { repo.delete }.to raise_error(NotImplementedError)
+      end
     end
-  end
 
-  describe '#create' do
-    it 'raises an exception in case of not defined' do
-      expect { repo.create }.to raise_error(NotImplementedError)
+    describe '#create' do
+      it 'raises an exception in case of not defined' do
+        expect { repo.create }.to raise_error(NotImplementedError)
+      end
     end
-  end
 
-  describe '#url' do
-    it 'raises an exception in case of not defined' do
-      expect { repo.url }.to raise_error(NotImplementedError)
+    describe '#url' do
+      it 'raises an exception in case of not defined' do
+        expect { repo.url }.to raise_error(NotImplementedError)
+      end
     end
   end
 end

data/spec/git_duplicator/services/bitbucket_spec.rb

@@ -1,51 +1,55 @@
 require_relative '../../helper'
+module GitDuplicator
+  module Services
+    describe BitbucketRepository do
+      let(:credentials) do
+        { consumer_key: ENV['BITBUCKET_CONSUMER_KEY'],
+          consumer_secret: ENV['BITBUCKET_CONSUMER_SECRET'],
+          token: ENV['BITBUCKET_TOKEN'],
+          token_secret: ENV['BITBUCKET_TOKEN_SECRET'] }
+      end
+      let(:name) { ENV['TESTING_REPO'] }
+      let(:owner) { ENV['BITBUCKET_USER'] }
+      let(:remote_options) do
+        { scm: 'git', is_private: true, fork_policy: 'no_public_forks' }
+      end
+      let(:options) { { credentials: credentials, remote_options: remote_options } }
+      let(:repo) { described_class.new(name, owner, options) }
 
-describe GitDuplicator::Services::BitbucketRepository do
-  let(:credentials) do
-    { consumer_key: ENV['BITBUCKET_CONSUMER_KEY'],
-      consumer_secret: ENV['BITBUCKET_CONSUMER_SECRET'],
-      token: ENV['BITBUCKET_TOKEN'],
-      token_secret: ENV['BITBUCKET_TOKEN_SECRET'] }
-  end
-  let(:name) { ENV['TESTING_REPO'] }
-  let(:owner) { ENV['BITBUCKET_USER'] }
-  let(:options) do
-    { scm: 'git', is_private: true, fork_policy: 'no_public_forks' }
-  end
-  let(:repo) { described_class.new(name, owner, credentials, options) }
+      describe '#delete' do
+        it 'deletes the repo in case it exists' do
+          stub_request(:delete, described_class::BASE_URI +
+                       "/repositories/#{owner}/#{name}")
+          .to_return(body: '', status: 204)
+          expect { repo.delete }.not_to raise_error
+        end
+        it 'raises an exception in case of unknow problem' do
+          stub_request(:delete, described_class::BASE_URI +
+                       "/repositories/#{owner}/#{name}")
+          .to_return(body: 'something wrong', status: 401)
+          expect { repo.delete }
+          .to raise_error(GitDuplicator::RepositoryDeletionError, /something wrong/)
+        end
+      end
 
-  describe '#delete' do
-    it 'deletes the repo in case it exists' do
-      stub_request(:delete, described_class::BASE_URI +
-                   "/repositories/#{owner}/#{name}")
-      .to_return(body: '', status: 204)
-      expect { repo.delete }.not_to raise_error
-    end
-    it 'raises an exception in case of unknow problem' do
-      stub_request(:delete, described_class::BASE_URI +
-                   "/repositories/#{owner}/#{name}")
-      .to_return(body: 'something wrong', status: 401)
-      expect { repo.delete }
-      .to raise_error(GitDuplicator::RepositoryDeletionError, /something wrong/)
-    end
-  end
-
-  describe '#create' do
-    it 'creates the repository in case of not defined' do
-      stub_request(:post, described_class::BASE_URI +
-                   "/repositories/#{owner}/#{name}")
-      .with(body: options.to_json)
-      .to_return(body: '', status: 200)
-      expect { repo.create }.not_to raise_error
-    end
+      describe '#create' do
+        it 'creates the repository in case of not defined' do
+          stub_request(:post, described_class::BASE_URI +
+                       "/repositories/#{owner}/#{name}")
+          .with(body: remote_options.to_json)
+          .to_return(body: '', status: 200)
+          expect { repo.create }.not_to raise_error
+        end
 
-    it 'raises an exception in case of errors' do
-      stub_request(:post, described_class::BASE_URI +
-                   "/repositories/#{owner}/#{name}")
-      .with(body: options.to_json)
-      .to_return(body: 'something wrong', status: 401)
-      expect { repo.create }
-      .to raise_error(GitDuplicator::RepositoryCreationError, /something wrong/)
+        it 'raises an exception in case of errors' do
+          stub_request(:post, described_class::BASE_URI +
+                       "/repositories/#{owner}/#{name}")
+          .with(body: remote_options.to_json)
+          .to_return(body: 'something wrong', status: 401)
+          expect { repo.create }
+          .to raise_error(GitDuplicator::RepositoryCreationError, /something wrong/)
+        end
+      end
     end
   end
 end

data/spec/git_duplicator/services/github_spec.rb

@@ -1,52 +1,58 @@
 require_relative '../../helper'
 
-describe GitDuplicator::Services::GithubRepository do
-  let(:credentials) do
-    { oauth2_token: ENV['GITHUB_ACCESS_TOKEN'] }
-  end
-  let(:name) { ENV['TESTING_REPO'] }
-  let(:owner) { ENV['GITHUB_USER'] }
-  let(:options) do
-    { has_issues: false, has_wiki: false }
-  end
-  let(:repo) { described_class.new(name, owner, credentials, options) }
+module GitDuplicator
+  module Services
+    describe GitDuplicator::Services::GithubRepository do
+      let(:credentials) do
+        { oauth2_token: ENV['GITHUB_ACCESS_TOKEN'] }
+      end
+      let(:name) { ENV['TESTING_REPO'] }
+      let(:owner) { ENV['GITHUB_USER'] }
+      let(:remote_options) do
+        { has_issues: false, has_wiki: false }
+      end
+      let(:options) { { credentials: credentials, remote_options: remote_options } }
+      let(:repo) { described_class.new(name, owner, options) }
 
-  describe '#delete' do
-    it 'deletes the repo in case it exists' do
-      stub_request(:delete, described_class::BASE_URI +
-                   "/repos/#{owner}/#{name}")
-      .with(
-        headers: { 'Authorization' => "token #{credentials[:oauth2_token]}" }
-      )
-      .to_return(body: '', status: 204)
-      expect { repo.delete }.not_to raise_error
-    end
-    it 'raises an exception in case of unknow problem' do
-      stub_request(:delete, described_class::BASE_URI +
-                   "/repos/#{owner}/#{name}")
-      .to_return(body: 'something wrong', status: 401)
-      expect { repo.delete }
-      .to raise_error(GitDuplicator::RepositoryDeletionError)
-    end
-  end
+      describe '#delete' do
+        it 'deletes the repo in case it exists' do
+          stub_request(:delete, described_class::BASE_URI +
+                       "/repos/#{owner}/#{name}")
+          .with(
+            headers: { 'Authorization' => "token #{credentials[:oauth2_token]}" }
+          )
+          .to_return(body: '', status: 204)
+          expect { repo.delete }.not_to raise_error
+        end
+        it 'raises an exception in case of unknow problem' do
+          stub_request(:delete, described_class::BASE_URI +
+                       "/repos/#{owner}/#{name}")
+          .to_return(body: 'something wrong', status: 401)
+          expect { repo.delete }
+          .to raise_error(GitDuplicator::RepositoryDeletionError)
+        end
+      end
 
-  describe '#create' do
-    it 'creates the repository in case of not defined' do
-      stub_request(:post, described_class::BASE_URI + '/user/repos')
-      .with(
-        body: options.merge(name: name).to_json,
-        headers: { 'Authorization' => "token #{credentials[:oauth2_token]}" }
-      )
-      .to_return(body: '', status: 201)
-      expect { repo.create }.not_to raise_error
-    end
+      describe '#create' do
+        it 'creates the repository in case of not defined' do
+          stub_request(:post, described_class::BASE_URI + '/user/repos')
+          .with(
+            body: remote_options.merge(name: name).to_json,
+            headers: { 'Authorization' =>
+                       "token #{credentials[:oauth2_token]}" }
+          )
+          .to_return(body: '', status: 201)
+          expect { repo.create }.not_to raise_error
+        end
 
-    it 'raises an exception in case of errors' do
-      stub_request(:post, described_class::BASE_URI + '/user/repos')
-      .with(body: options.merge(name: name).to_json)
-      .to_return(body: 'something wrong', status: 401)
-      expect { repo.create }
-      .to raise_error(GitDuplicator::RepositoryCreationError)
+        it 'raises an exception in case of errors' do
+          stub_request(:post, described_class::BASE_URI + '/user/repos')
+          .with(body: remote_options.merge(name: name).to_json)
+          .to_return(body: 'something wrong', status: 401)
+          expect { repo.create }
+          .to raise_error(GitDuplicator::RepositoryCreationError)
+        end
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: git_duplicator
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Khaled alHabache
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-29 00:00:00.000000000 Z
+date: 2014-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http
@@ -71,7 +71,10 @@ files:
 - Rakefile
 - git_duplicator.gemspec
 - lib/git_duplicator.rb
-- lib/git_duplicator/duplicator.rb
+- lib/git_duplicator/duplicator/duplicator.rb
+- lib/git_duplicator/duplicator/mirror_duplicator.rb
+- lib/git_duplicator/duplicator/update_duplicator.rb
+- lib/git_duplicator/duplicators.rb
 - lib/git_duplicator/errors.rb
 - lib/git_duplicator/helpers/authorization_header.rb
 - lib/git_duplicator/null_logger.rb
@@ -82,8 +85,12 @@ files:
 - lib/git_duplicator/services/bitbucket.rb
 - lib/git_duplicator/services/github.rb
 - lib/git_duplicator/version.rb
-- spec/git_duplicator/duplicator_spec.rb
+- spec/git_duplicator/duplicator/duplicator_spec.rb
+- spec/git_duplicator/duplicator/mirror_duplicator_spec.rb
+- spec/git_duplicator/duplicator/update_duplicator_spec.rb
+- spec/git_duplicator/git_duplicator_acceptance_spec.rb
 - spec/git_duplicator/git_duplicator_spec.rb
+- spec/git_duplicator/helpers/authorization_header_spec.rb
 - spec/git_duplicator/repository/repository_spec.rb
 - spec/git_duplicator/repository/service_repository_spec.rb
 - spec/git_duplicator/services/bitbucket_spec.rb

data/spec/git_duplicator/duplicator_spec.rb

@@ -1,82 +0,0 @@
-require_relative '../helper'
-
-describe GitDuplicator::Duplicator do
-  let(:path) { '/somepath' }
-  let(:from) do
-    double(name: 'somename', bare_clone: nil, mirror: nil, url: 'somewhere1')
-  end
-  let(:to) { double(delete: nil, create: nil, url: 'somewhere2') }
-  let(:duplicator) { GitDuplicator::Duplicator.new(from, to, options) }
-
-  describe '#perform' do
-    context 'force_create_destination is set' do
-      let(:options) { { clone_path: path, force_create_destination: true } }
-
-      it 'deletes existing destination repo' do
-        expect(to).to receive(:delete)
-        duplicator.perform
-      end
-
-      it 'creates the destination repo' do
-        expect(to).to receive(:create)
-        duplicator.perform
-      end
-
-      it 'bare clones the source repo' do
-        expect(from).to receive(:bare_clone).with(path)
-        duplicator.perform
-      end
-
-      it 'mirrors from bare clone to destination' do
-        expect(from).to receive(:mirror).with(to.url)
-        duplicator.perform
-      end
-
-      it 'cleans up' do
-        expect(FileUtils).to receive(:rm_rf)
-        .with("#{duplicator.clone_path}/#{from.name}")
-        duplicator.perform
-      end
-
-      it 'logs 5 times' do
-        expect(duplicator.logger).to receive(:info).exactly(5).times
-        duplicator.perform
-      end
-    end
-
-    context 'force_create_destination is false' do
-      let(:options) { { clone_path: path } }
-
-      it 'deletes existing destination repo' do
-        expect(to).not_to receive(:delete)
-        duplicator.perform
-      end
-
-      it 'creates the destination repo' do
-        expect(to).not_to receive(:create)
-        duplicator.perform
-      end
-
-      it 'bare clones the source repo' do
-        expect(from).to receive(:bare_clone).with(path)
-        duplicator.perform
-      end
-
-      it 'mirrors from bare clone to destination' do
-        expect(from).to receive(:mirror).with(to.url)
-        duplicator.perform
-      end
-
-      it 'cleans up' do
-        expect(FileUtils).to receive(:rm_rf)
-        .with("#{duplicator.clone_path}/#{from.name}")
-        duplicator.perform
-      end
-
-      it 'logs 3 times' do
-        expect(duplicator.logger).to receive(:info).exactly(3).times
-        duplicator.perform
-      end
-    end
-  end
-end