txgh 2.0.1 → 2.1.0

data/spec/resource_committer_spec.rb

@@ -14,18 +14,44 @@ describe ResourceCommitter do
     ResourceCommitter.new(transifex_project, github_repo, logger)
   end
 
+  before(:each) do
+    allow(github_api).to receive(:get_ref).and_return(
+      object: { sha: 'abc123shashasha' }
+    )
+  end
+
   describe '#commit_resource' do
-    it 'commits translations to the git repo' do
-      expect(ResourceDownloader).to receive(:new).and_return(downloader)
-      expect(downloader).to receive(:first).and_return([file_name, :translations])
+    context 'with apis available' do
+      before(:each) do
+        expect(ResourceDownloader).to receive(:new).and_return(downloader)
+        expect(downloader).to receive(:first).and_return([file_name, :translations])
+
+        expect(github_api).to(
+          receive(:commit).with(
+            repo_name, branch, { file_name => :translations }
+          )
+        )
+      end
+
+      it 'commits translations to the git repo' do
+        committer.commit_resource(resource, branch, language)
+      end
 
-      expect(github_api).to(
-        receive(:commit).with(
-          repo_name, branch, { file_name => :translations }
+      it 'fires the github.resource.committed event' do
+        expect { committer.commit_resource(resource, branch, language) }.to(
+          change { Txgh.events.published.size }.by(1)
         )
-      )
 
-      committer.commit_resource(resource, branch, language)
+        event = Txgh.events.published.first
+        expect(event[:channel]).to eq('github.resource.committed')
+
+        options = event[:options]
+        expect(options[:project].name).to eq(project_name)
+        expect(options[:repo].name).to eq(repo_name)
+        expect(options[:sha]).to eq('abc123shashasha')
+        expect(options[:resource].original_resource_slug).to eq(resource_slug)
+        expect(options[:language]).to eq(language)
+      end
     end
 
     it "doesn't commit anything if the language is the source language" do

data/spec/resource_updater_spec.rb

@@ -65,6 +65,23 @@ describe ResourceUpdater do
     updater.update_resource(resource, commit_sha)
   end
 
+  it 'fires the transifex.resource.updated event' do
+    allow(transifex_api).to receive(:create_or_update)
+
+    expect { updater.update_resource(resource, commit_sha) }.to(
+      change { Txgh.events.published.size }.by(1)
+    )
+
+    event = Txgh.events.published.first
+    expect(event[:channel]).to eq('transifex.resource.updated')
+
+    options = event[:options]
+    expect(options[:project].name).to eq(project_name)
+    expect(options[:repo].name).to eq(repo_name)
+    expect(options[:sha]).to eq(commit_sha)
+    expect(options[:resource].original_resource_slug).to eq(resource_slug)
+  end
+
   context 'when asked to process all branches' do
     let(:branch) { 'all' }
     let(:ref) { 'heads/master' }

data/spec/spec_helper.rb

@@ -8,6 +8,8 @@ require 'vcr'
 require 'webmock'
 require 'yaml'
 
+require 'helpers/test_events'
+
 module SpecHelpers
   def outdent(str)
     # The special YAML pipe operator treats the text that follows as literal,
@@ -24,6 +26,10 @@ RSpec.configure do |config|
   config.run_all_when_everything_filtered = true
   config.filter_run_excluding(integration: true) unless ENV['FULL_SPEC']
   config.include(SpecHelpers)
+
+  config.before(:each) do
+    Txgh.instance_variable_set(:@events, TestEvents.new)
+  end
 end
 
 VCR.configure do |config|

data/spec/transifex_api_spec.rb

@@ -8,7 +8,7 @@ describe TransifexApi do
 
   let(:connection) { double(:connection) }
   let(:api) { TransifexApi.create_from_connection(connection) }
-  let(:resource) { tx_config.resources.first }
+  let(:resource) { tx_config.resource(resource_slug) }
   let(:response) { double(:response) }
 
   describe '#create_or_update' do
@@ -77,6 +77,7 @@ describe TransifexApi do
           end_with("project/#{project_name}/resources/")
         )
 
+        expect(payload[:name]).to eq('sample.yml')
         expect(payload[:content].io.string).to eq('new_content')
         expect(payload[:categories]).to eq('abc def')
         response
@@ -102,6 +103,20 @@ describe TransifexApi do
       allow(response).to receive(:body).and_return('{}')
       expect { api.create(resource, 'new_content') }.to raise_error(TransifexApiError)
     end
+
+    context 'with a branch-based resource' do
+      let(:resource) { tx_config.resource(resource_slug, ref) }
+
+      it "includes the branch in the resource's name" do
+        expect(connection).to receive(:post) do |url, payload|
+          expect(payload[:name]).to eq('sample.yml (heads/master)')
+          response
+        end
+
+        allow(response).to receive(:status).and_return(200)
+        api.create(resource, 'new_content')
+      end
+    end
   end
 
   describe '#delete' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: txgh
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 2.1.0
 platform: ruby
 authors:
 - Matthew Jackowski
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-28 00:00:00.000000000 Z
+date: 2016-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: abroad
@@ -186,9 +186,11 @@ files:
 - lib/txgh/diff_calculator.rb
 - lib/txgh/empty_resource_contents.rb
 - lib/txgh/errors.rb
+- lib/txgh/events.rb
 - lib/txgh/github_api.rb
 - lib/txgh/github_repo.rb
 - lib/txgh/github_request_auth.rb
+- lib/txgh/github_status.rb
 - lib/txgh/handlers.rb
 - lib/txgh/handlers/download_handler.rb
 - lib/txgh/handlers/github.rb
@@ -232,9 +234,11 @@ files:
 - spec/config/tx_config_spec.rb
 - spec/config/tx_manager_spec.rb
 - spec/diff_calculator_spec.rb
+- spec/events_spec.rb
 - spec/github_api_spec.rb
 - spec/github_repo_spec.rb
 - spec/github_request_auth_spec.rb
+- spec/github_status_spec.rb
 - spec/handlers/download_handler_spec.rb
 - spec/handlers/github/delete_handler_spec.rb
 - spec/handlers/github/ping_handler_spec.rb
@@ -246,6 +250,7 @@ files:
 - spec/helpers/integration_setup.rb
 - spec/helpers/nil_logger.rb
 - spec/helpers/standard_txgh_setup.rb
+- spec/helpers/test_events.rb
 - spec/helpers/test_provider.rb
 - spec/integration/cassettes/github_l10n_hook_endpoint.yml
 - spec/integration/cassettes/pull.yml