git_multicast 0.0.5 → 0.0.6

data/spec/git_multicast/adapters/bitbucket_spec.rb

@@ -0,0 +1,25 @@
+module GitMulticast
+  describe Adapters::Bitbucket do
+    subject(:adapter) { described_class.new(repo) }
+
+    describe '#adapt' do
+      subject(:adapt) { adapter.adapt }
+
+      let(:url) do
+        'https://bitbucket.org/api/2.0/repositories/rranelli/cronofaker'
+      end
+      let(:repo) { RepositoryFetcher::Bitbucket.get_repo(url) }
+
+      # I know this is ugly, but well...
+      it do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(adapt.url).to eq('https://bitbucket.org/api/2.0/repositories/rranelli/cronofaker')
+          expect(adapt.ssh_url).to eq('ssh://git@bitbucket.org/rranelli/cronofaker.git')
+          expect(adapt.name).to eq('CronoFaker')
+          expect(adapt.fork).to be_falsy
+          expect(adapt.parent).to be_nil
+        end
+      end
+    end
+  end
+end

data/spec/git_multicast/puller_spec.rb

@@ -16,6 +16,10 @@ describe GitMulticast::Puller do
     allow(puller).to receive(:waitall).and_return([[1, success], [2, success]])
   end
 
+  after do
+    $stdout = STDOUT
+  end
+
   describe '#pull' do
     subject(:pull) { puller.pull }
 

data/spec/git_multicast/repository_fetcher/bitbucket_spec.rb

@@ -0,0 +1,80 @@
+module GitMulticast
+  describe RepositoryFetcher::Bitbucket do
+    subject(:fetcher) { described_class }
+
+    describe '.get_repo' do
+      subject(:get_repo) { fetcher.get_repo(url) }
+
+      let(:url) do
+        'https://bitbucket.org/api/2.0/repositories/rranelli/cronofaker'
+      end
+      let(:json) do
+        { 'links' => { 'clone' => 'I be a body' } }
+      end
+
+      before do
+        allow(JSON).to receive(:parse).and_return(json)
+      end
+
+      it 'calls http get' do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(Net::HTTP).to receive(:get_response)
+            .with(URI(url)).and_call_original
+
+          get_repo
+        end
+      end
+
+      it 'parses the resulting json' do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(JSON).to receive(:parse)
+
+          get_repo
+        end
+      end
+
+      it 'makes a struct with the result body' do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(RecursiveOpenStruct).to receive(:new)
+            .with(json, recurse_over_arrays: true)
+
+          get_repo
+        end
+      end
+    end
+
+    describe '.get_all_repos_from_user' do
+      subject(:get_all_repos_from_user) do
+        fetcher.get_all_repos_from_user(user)
+      end
+
+      let(:user) { 'rranelli' }
+      let(:url) { URI('https://bitbucket.org/api/2.0/repositories/rranelli') }
+
+      it 'calls http get' do
+        VCR.use_cassette('bitbucket_all_user_repos') do
+          expect(Net::HTTP).to receive(:get_response)
+            .with(url).and_call_original
+
+          get_all_repos_from_user
+        end
+      end
+
+      it 'parses the resulting json' do
+        VCR.use_cassette('bitbucket_all_user_repos') do
+          expect(JSON).to receive(:parse).and_call_original
+
+          get_all_repos_from_user
+        end
+      end
+
+      it 'builds each repository as a RecursiveOpenStruct' do
+        VCR.use_cassette('bitbucket_all_user_repos') do
+          expect(RecursiveOpenStruct).to receive(:new).exactly(3).times
+
+          get_all_repos_from_user
+        end
+      end
+    end
+  end
+end

data/spec/git_multicast/repository_fetcher/github_spec.rb

@@ -0,0 +1,81 @@
+module GitMulticast
+  describe RepositoryFetcher::Github do
+    subject(:fetcher) { described_class }
+
+    let(:uri) { URI(url) }
+
+    describe '.get_repo' do
+      subject(:get_repo) { fetcher.get_repo(url) }
+
+      let(:url) { 'https://api.github.com/repos/rranelli/git_multicast' }
+      let(:json) do
+        { 'value' => 'I be a body' }
+      end
+
+      before do
+        allow(JSON).to receive(:parse).and_return(json)
+      end
+
+      it 'calls http get' do
+        VCR.use_cassette('github_repo') do
+          expect(Net::HTTP).to receive(:get_response)
+            .with(uri).and_call_original
+
+          get_repo
+        end
+      end
+
+      it 'parses the resulting json' do
+        VCR.use_cassette('github_repo') do
+          expect(JSON).to receive(:parse)
+
+          get_repo
+        end
+      end
+
+      it 'Makes a struct with the result body' do
+        VCR.use_cassette('github_repo') do
+          expect(RecursiveOpenStruct).to receive(:new).with(
+            json, recurse_over_arrays: true
+          )
+
+          get_repo
+        end
+      end
+    end
+
+    describe '.get_all_repos_from_user' do
+      subject(:get_all_repos_from_user) do
+        fetcher.get_all_repos_from_user(user)
+      end
+
+      let(:user) { 'rranelli' }
+      let(:url) { 'https://api.github.com/users/rranelli/repos' }
+
+      it 'calls http get' do
+        VCR.use_cassette('github_all_user_repos') do
+          expect(Net::HTTP).to receive(:get_response)
+            .with(uri).and_call_original
+
+          get_all_repos_from_user
+        end
+      end
+
+      it 'parses the resulting json' do
+        VCR.use_cassette('github_all_user_repos') do
+          expect(JSON).to receive(:parse).and_call_original
+
+          get_all_repos_from_user
+        end
+      end
+
+      it 'builds each repository as an OpenStruct' do
+        VCR.use_cassette('github_all_user_repos') do
+          expect(RecursiveOpenStruct).to receive(:new).exactly(29).times
+
+          get_all_repos_from_user
+        end
+      end
+    end
+  end
+end

data/spec/git_multicast/repository_fetcher_spec.rb

@@ -1,33 +1,116 @@
-describe GitMulticast::RepositoryFetcher do
-  subject(:fetcher) { described_class }
+module GitMulticast
+  describe RepositoryFetcher do
+    subject(:fetcher) { described_class }
 
-  let(:fetchers) { described_class::FETCHERS }
+    let(:fetchers) { described_class::FETCHERS }
+    let(:username) { 'rranelli' }
 
-  let(:username) { 'chuck norris' }
+    describe '.get_all_repos_from_user' do
+      subject(:get_all_repos_from_user) do
+        fetcher.get_all_repos_from_user(username)
+      end
+
+      it 'gets repositories from all fetchers' do
+        VCR.use_cassette('repos_from_all_services') do
+          fetchers.each do |fetcher|
+            expect(fetcher).to receive(:get_all_repos_from_user)
+               .with(username).and_call_original
+          end
+
+          get_all_repos_from_user
+        end
+      end
 
-  describe '.get_all_repos_from_user' do
-    subject(:get_all_repos_from_user) do
-      fetcher.get_all_repos_from_user(username)
+      it 'aggregates all repositories in a single list' do
+        VCR.use_cassette('repos_from_all_services') do
+          repo_names = get_all_repos_from_user.map(&:name)
+
+          expect(get_all_repos_from_user.size).to eq(32)
+          expect(repo_names).to include('git_multicast')
+          expect(repo_names).to include('CronoFaker')
+        end
+      end
     end
 
-    it 'gets repositories from all fetchers' do
-      fetchers.each do |e|
-        expect(e).to receive(:get_all_repos_from_user)
+    describe '.get_repo_parent' do
+      subject(:get_repo_parent) { fetcher.get_repo_parent(url) }
+
+      let(:url) do
+        'https://api.github.com/repos/rranelli/git_multicast'
       end
 
-      get_all_repos_from_user
+      it 'delegates to the right fetcher' do
+        VCR.use_cassette('github_repo') do
+          expect(RepositoryFetcher::Github).to receive(:get_repo)
+            .with(url).and_call_original
+
+          get_repo_parent
+        end
+      end
+
+      it 'adapts with the right adapter' do
+        VCR.use_cassette('github_repo') do
+          expect(Adapters::Github).to receive_message_chain(
+            :new, :adapt, :parent
+          )
+
+          get_repo_parent
+        end
+      end
+
+      it 'gets no parent when repo has no parent' do
+        VCR.use_cassette('github_repo') do
+          is_expected.to be_nil
+        end
+      end
+
+      context 'when repository has a parent' do
+        let(:url) do
+          'https://api.github.com/repos/rranelli/ruby-git-hooks'
+        end
+
+        it 'gets the parent' do
+          VCR.use_cassette('github_repo_parent') do
+            upstream_repo_owner = get_repo_parent.owner.login
+
+            expect(upstream_repo_owner).to eq('stupied4ever')
+          end
+        end
+      end
     end
-  end
 
-  describe 'self.get_parent_repo' do
-    subject(:get_parent_repo) { fetcher.get_parent_repo(url) }
+    describe '.get_repo' do
+      subject(:get_repo) { fetcher.get_repo(url) }
+
+      let(:url) do
+        'https://bitbucket.org/api/2.0/repositories/rranelli/cronofaker'
+      end
 
-    let(:url) { 'http://bitbucket.im.wrong.as.hell' }
+      let(:bb_adapter) { double(:bb_adapter, adapt: nil) }
 
-    it 'delegates to the right fetcher' do
-      expect(GitMulticast::BitbucketFetcher).to receive(:get_parent_repo).with(url)
+      it 'delegates to the right fetcher' do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(RepositoryFetcher::Bitbucket).to receive(:get_repo)
+            .with(url).and_call_original
 
-      get_parent_repo
+          get_repo
+        end
+      end
+
+      it 'adapts with the right adapter' do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(Adapters::Bitbucket).to receive(:new).and_return(bb_adapter)
+          expect(bb_adapter).to receive(:adapt)
+
+          get_repo
+        end
+      end
+
+      it 'gets the repository' do
+        VCR.use_cassette('bitbucket_repo') do
+          expect(get_repo.name).to eq('CronoFaker')
+        end
+      end
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,6 +1,8 @@
+require 'vcr'
+
 require_relative '../lib/git_multicast'
 
-def fixture_path(filename)
-  return '' if filename == ''
-  File.join(File.absolute_path(File.dirname(__FILE__)), 'fixtures', filename)
+VCR.configure do |c|
+  c.cassette_library_dir = 'spec/fixtures/vcr_cassettes'
+  c.hook_into :webmock
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: git_multicast
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Renan Ranelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-06 00:00:00.000000000 Z
+date: 2014-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: recursive-open-struct
@@ -80,6 +80,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: vcr
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +122,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-doc
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: method_source
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 
 email:
 - renanranelli@gmail.com
@@ -104,6 +160,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - Gemfile
 - Gemfile.lock
 - LICENSE
@@ -112,25 +169,34 @@ files:
 - bin/git_multicast
 - git_multicast.gemspec
 - lib/git_multicast.rb
-- lib/git_multicast/bitbucket_adapter.rb
-- lib/git_multicast/bitbucket_fetcher.rb
+- lib/git_multicast/.rubocop.yml
+- lib/git_multicast/adapters.rb
+- lib/git_multicast/adapters/bitbucket.rb
+- lib/git_multicast/adapters/github.rb
 - lib/git_multicast/cli.rb
 - lib/git_multicast/cloner.rb
-- lib/git_multicast/github_fetcher.rb
 - lib/git_multicast/output_formatter.rb
 - lib/git_multicast/puller.rb
 - lib/git_multicast/repository_fetcher.rb
+- lib/git_multicast/repository_fetcher/bitbucket.rb
+- lib/git_multicast/repository_fetcher/github.rb
 - lib/git_multicast/version.rb
-- spec/git_multicast/bitbucket_adapter_spec.rb
-- spec/git_multicast/bitbucket_fetcher_spec.rb
+- spec/fixtures/vcr_cassettes/bitbucket_all_user_repos.yml
+- spec/fixtures/vcr_cassettes/bitbucket_repo.yml
+- spec/fixtures/vcr_cassettes/github_all_user_repos.yml
+- spec/fixtures/vcr_cassettes/github_repo.yml
+- spec/fixtures/vcr_cassettes/github_repo_parent.yml
+- spec/fixtures/vcr_cassettes/repos_from_all_services.yml
+- spec/git_multicast/adapters/bitbucket_spec.rb
 - spec/git_multicast/cloner_spec.rb
-- spec/git_multicast/github_fetcher_spec.rb
 - spec/git_multicast/puller_spec.rb
+- spec/git_multicast/repository_fetcher/bitbucket_spec.rb
+- spec/git_multicast/repository_fetcher/github_spec.rb
 - spec/git_multicast/repository_fetcher_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/rranelli/git_multicast
 licenses:
-- MIT
+- DWTF
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -153,3 +219,4 @@ signing_key:
 specification_version: 4
 summary: Execute mass actions on git repositories concurrently
 test_files: []
+has_rdoc: 

data/lib/git_multicast/bitbucket_adapter.rb

@@ -1,33 +0,0 @@
-module GitMulticast
-  class BitbucketAdapter
-    def initialize(repo)
-      @repo = repo
-    end
-
-    def adapt
-      make_struct(repo_hash)
-    end
-
-    protected
-
-    attr_reader :repo
-
-    def repo_hash
-      @repo_hash ||= make_repo_hash
-    end
-
-    def make_repo_hash
-      {
-        fork: !repo.parent.nil?,
-        ssh_url: repo.links._clone.last.href,
-        url: repo.links.self.href,
-        parent: nil,
-        name: repo.name
-      }
-    end
-
-    def make_struct(hash)
-      RecursiveOpenStruct.new(hash, recurse_over_arrays: true)
-    end
-  end
-end

data/lib/git_multicast/bitbucket_fetcher.rb

@@ -1,36 +0,0 @@
-module GitMulticast
-  class BitbucketFetcher
-    REPOS_URI = 'https://bitbucket.org/api/2.0/repositories/%{username}'
-
-    def self.get_all_repos_from_user(username)
-      uri_str = REPOS_URI % { username: username }
-      uri = URI(uri_str)
-
-      response = Net::HTTP.get_response(uri)
-      response_json = JSON.parse(response.body)
-
-      # Damn...
-      response_json['values'].each do |node|
-        node['links']['_clone'] = node['links']['clone']
-      end
-
-      bb_repos = response_json['values'].map { |hash| make_struct(hash) }
-      bb_repos.map { |bb_repo| BitbucketAdapter.new(bb_repo).adapt }
-    end
-
-    def self.get_repo_parent(url)
-      bb_repo = get_repo(url).parent
-      BitbucketAdapter.new(bb_repo).adapt
-    end
-
-    def self.get_repo(url)
-      response = Net::HTTP.get_response(URI(url))
-      bb_repo = make_struct(JSON.parse(response.body))
-      BitbucketAdapter.new(bb_repo).adapt
-    end
-
-    def self.make_struct(hash)
-      RecursiveOpenStruct.new(hash, recurse_over_arrays: true)
-    end
-  end
-end

data/lib/git_multicast/github_fetcher.rb

@@ -1,31 +0,0 @@
-require 'recursive-open-struct'
-require 'json'
-
-module GitMulticast
-  class GithubFetcher
-    REPOS_URI = 'https://api.github.com/users/%{username}/repos'
-
-    def self.get_all_repos_from_user(username)
-      uri_str = REPOS_URI % { username: username }
-      uri = URI(uri_str)
-
-      response = Net::HTTP.get_response(uri)
-      repos = JSON.parse(response.body)
-
-      repos.map { |hash| make_struct(hash) }
-    end
-
-    def self.get_repo_parent(url)
-      get_repo(url).parent
-    end
-
-    def self.get_repo(url)
-      response = Net::HTTP.get_response(URI(url))
-      make_struct(JSON.parse(response.body))
-    end
-
-    def self.make_struct(hash)
-      RecursiveOpenStruct.new(hash, recurse_over_arrays: true)
-    end
-  end
-end

data/spec/git_multicast/bitbucket_adapter_spec.rb

@@ -1,25 +0,0 @@
-describe GitMulticast::BitbucketAdapter do
-  subject(:adapter) { described_class.new(repo) }
-
-  let(:repo) { double(:repo) }
-
-  before do
-    allow(repo).to receive_message_chain(:links, :_clone, :last, :href)
-      .and_return('git@bucketbit.org:foo/bar.git')
-    allow(repo).to receive_message_chain(:links, :self, :href)
-      .and_return('http://bucketbit.org/test-repo')
-
-    allow(repo).to receive(:name).and_return('test-repo')
-    allow(repo).to receive(:parent).and_return(nil)
-  end
-
-  describe '#adapt' do
-    subject(:adapt) { adapter.adapt }
-
-    it { expect(adapt.name).to eq('test-repo') }
-    it { expect(adapt.fork).to be_falsy }
-    it { expect(adapt.url).to eq('http://bucketbit.org/test-repo') }
-    it { expect(adapt.ssh_url).to eq('git@bucketbit.org:foo/bar.git') }
-    it { expect(adapt.parent).to be_nil }
-  end
-end