txgh 1.0.0

data/spec/integration/payloads/github_postbody_release.json

@@ -0,0 +1,136 @@
+{
+  "ref": "refs/tags/v3.0.0",
+  "before": "0000000000000000000000000000000000000000",
+  "after": "a779a46381d0d10c8e9dc390da0eaf567465cb39",
+  "created": true,
+  "deleted": false,
+  "forced": true,
+  "base_ref": "refs/heads/master",
+  "compare": "https://github.com/txgh-bot/txgh-test-resources/compare/v3.0.0",
+  "commits": [
+
+  ],
+  "head_commit": {
+    "id": "a779a46381d0d10c8e9dc390da0eaf567465cb39",
+    "distinct": true,
+    "message": "11",
+    "timestamp": "2015-11-18T16:18:35-08:00",
+    "url": "https://github.com/txgh-bot/txgh-test-resources/commit/a779a46381d0d10c8e9dc390da0eaf567465cb39",
+    "author": {
+      "name": "Txgh Bot",
+      "email": "txgh.bot@gmail.com",
+      "username": "txgh-bot"
+    },
+    "committer": {
+      "name": "Txgh Bot",
+      "email": "txgh.bot@gmail.com",
+      "username": "txgh-bot"
+    },
+    "added": [
+
+    ],
+    "removed": [
+
+    ],
+    "modified": [
+      "sample.po"
+    ]
+  },
+  "repository": {
+    "id": 41740726,
+    "name": "txgh-test-resources",
+    "full_name": "txgh-bot/txgh-test-resources",
+    "owner": {
+      "name": "txgh-bot",
+      "email": "txgh.bot@gmail.com"
+    },
+    "private": false,
+    "html_url": "https://github.com/txgh-bot/txgh-test-resources",
+    "description": "Translation test repo for txgh",
+    "fork": false,
+    "url": "https://github.com/txgh-bot/txgh-test-resources",
+    "forks_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/forks",
+    "keys_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/keys{/key_id}",
+    "collaborators_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/collaborators{/collaborator}",
+    "teams_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/teams",
+    "hooks_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/hooks",
+    "issue_events_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/issues/events{/number}",
+    "events_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/events",
+    "assignees_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/assignees{/user}",
+    "branches_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/branches{/branch}",
+    "tags_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/tags",
+    "blobs_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/git/blobs{/sha}",
+    "git_tags_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/git/tags{/sha}",
+    "git_refs_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/git/refs{/sha}",
+    "trees_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/git/trees{/sha}",
+    "statuses_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/statuses/{sha}",
+    "languages_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/languages",
+    "stargazers_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/stargazers",
+    "contributors_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/contributors",
+    "subscribers_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/subscribers",
+    "subscription_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/subscription",
+    "commits_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/commits{/sha}",
+    "git_commits_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/git/commits{/sha}",
+    "comments_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/comments{/number}",
+    "issue_comment_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/issues/comments{/number}",
+    "contents_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/contents/{+path}",
+    "compare_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/compare/{base}...{head}",
+    "merges_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/merges",
+    "archive_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/{archive_format}{/ref}",
+    "downloads_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/downloads",
+    "issues_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/issues{/number}",
+    "pulls_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/pulls{/number}",
+    "milestones_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/milestones{/number}",
+    "notifications_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/notifications{?since,all,participating}",
+    "labels_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/labels{/name}",
+    "releases_url": "https://api.github.com/repos/txgh-bot/txgh-test-resources/releases{/id}",
+    "created_at": 1441114620,
+    "updated_at": "2015-09-01T13:37:00Z",
+    "pushed_at": 1447892352,
+    "git_url": "git://github.com/txgh-bot/txgh-test-resources.git",
+    "ssh_url": "git@github.com:txgh-bot/txgh-test-resources.git",
+    "clone_url": "https://github.com/txgh-bot/txgh-test-resources.git",
+    "svn_url": "https://github.com/txgh-bot/txgh-test-resources",
+    "homepage": null,
+    "size": 11,
+    "stargazers_count": 0,
+    "watchers_count": 0,
+    "language": null,
+    "has_issues": true,
+    "has_downloads": true,
+    "has_wiki": true,
+    "has_pages": false,
+    "forks_count": 0,
+    "mirror_url": null,
+    "open_issues_count": 0,
+    "forks": 0,
+    "open_issues": 0,
+    "watchers": 0,
+    "default_branch": "master",
+    "stargazers": 0,
+    "master_branch": "master"
+  },
+  "pusher": {
+    "name": "txgh-bot",
+    "email": "txgh.bot@gmail.com"
+  },
+  "sender": {
+    "login": "txgh-bot",
+    "id": 974299,
+    "avatar_url": "https://avatars.githubusercontent.com/u/974299?v=3",
+    "gravatar_id": "",
+    "url": "https://api.github.com/users/txgh-bot",
+    "html_url": "https://github.com/txgh-bot",
+    "followers_url": "https://api.github.com/users/txgh-bot/followers",
+    "following_url": "https://api.github.com/users/txgh-bot/following{/other_user}",
+    "gists_url": "https://api.github.com/users/txgh-bot/gists{/gist_id}",
+    "starred_url": "https://api.github.com/users/txgh-bot/starred{/owner}{/repo}",
+    "subscriptions_url": "https://api.github.com/users/txgh-bot/subscriptions",
+    "organizations_url": "https://api.github.com/users/txgh-bot/orgs",
+    "repos_url": "https://api.github.com/users/txgh-bot/repos",
+    "events_url": "https://api.github.com/users/txgh-bot/events{/privacy}",
+    "received_events_url": "https://api.github.com/users/txgh-bot/received_events",
+    "type": "User",
+    "site_admin": false
+  }
+}

data/spec/integration/triggers_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+require 'rack/test'
+require 'helpers/integration_setup'
+
+describe 'trigger integration tests', integration: true do
+  include Rack::Test::Methods
+  include IntegrationSetup
+
+  def app
+    @app ||= Txgh::Triggers.new
+  end
+
+  around(:each) do |example|
+    Dir.chdir('./spec/integration') do
+      example.run
+    end
+  end
+
+  it 'verifies the pull endpoint works' do
+    VCR.use_cassette('pull') do
+      params = {
+        project_slug: 'test-project-88',
+        resource_slug: 'samplepo',
+        branch: 'master'
+      }
+
+      patch '/pull', params
+      expect(last_response).to be_ok
+    end
+  end
+
+  it 'verifies the push endpoint works' do
+    VCR.use_cassette('push') do
+      params = {
+        project_slug: 'test-project-88',
+        resource_slug: 'samplepo',
+        branch: 'master'
+      }
+
+      patch '/push', params
+      expect(last_response).to be_ok
+    end
+  end
+end

data/spec/merge_calculator_spec.rb

@@ -0,0 +1,112 @@
+require 'spec_helper'
+
+include Txgh
+
+describe MergeCalculator do
+  def phrase(key, string)
+    { 'key' => key, 'string' => string }
+  end
+
+  let(:resource) do
+    TxResource.new(
+      'project_name', 'resource_slug', 'YAML',
+      'en', 'en.yml', '', 'translation_file'
+    )
+  end
+
+  let(:head_contents) do
+    ResourceContents.from_phrase_list(resource, head_phrases)
+  end
+
+  let(:diff_point_contents) do
+    ResourceContents.from_phrase_list(resource, diff_point_phrases)
+  end
+
+  let(:diff_hash) do
+    head_contents.diff_hash(diff_point_contents)
+  end
+
+  let(:merge_result) do
+    MergeCalculator.merge(head_contents, diff_point_contents, diff_hash)
+  end
+
+  context 'with phrases added to HEAD' do
+    let(:diff_point_phrases) do
+      [phrase('planet.earth', 'Human')]
+    end
+
+    let(:head_phrases) do
+      diff_point_phrases + [
+        phrase('planet.bajor', 'Bajoran')
+      ]
+    end
+
+    it 'includes the added phrase' do
+      expect(merge_result.phrases).to eq(head_phrases)
+    end
+  end
+
+  context 'with phrases removed from HEAD' do
+    let(:diff_point_phrases) do
+      head_phrases + [
+        phrase('planet.bajor', 'Bajoran')
+      ]
+    end
+
+    let(:head_phrases) do
+      [phrase('planet.earth', 'Human')]
+    end
+
+    it 'does not include the removed phrase' do
+      expect(merge_result.phrases).to eq(head_phrases)
+    end
+  end
+
+  context 'with phrases modified in HEAD' do
+    let(:diff_point_phrases) do
+      [phrase('planet.bajor', 'Cardassian')]
+    end
+
+    let(:head_phrases) do
+      [phrase('planet.bajor', 'Bajoran')]
+    end
+
+    it 'includes the modified phrase' do
+      expect(merge_result.phrases).to eq(head_phrases)
+    end
+  end
+
+  context 'with no phrases modified, added, or removed' do
+    let(:diff_point_phrases) do
+      [phrase('planet.bajor', 'Bajoran')]
+    end
+
+    let(:head_phrases) do
+      diff_point_phrases
+    end
+
+    it 'returns an unmodified set of phrases' do
+      expect(merge_result.phrases).to eq(diff_point_phrases)
+    end
+  end
+
+  context 'with an separately defined diff hash' do
+    let(:diff_point_phrases) do
+      [phrase('planet.earth', 'Human')]
+    end
+
+    let(:head_phrases) do
+      diff_point_phrases + [
+        phrase('planet.bajor', 'Bajoran')
+      ]
+    end
+
+    let(:diff_hash) do
+      { added: [], removed: [], modified: [] }
+    end
+
+    it 'respects the diff hash by returning an unmodified set of phrases' do
+      expect(merge_result.phrases).to eq(diff_point_phrases)
+    end
+  end
+end

data/spec/parse_config_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+require 'tempfile'
+
+include Txgh
+
+describe Txgh::ParseConfig do
+  let(:contents) do
+    """
+    [header]
+    key = val
+
+    [header2]
+    key2 = val2
+    """
+  end
+
+  shared_examples 'a correct config loader' do
+    it 'has correctly parsed the given config' do
+      expect(config).to be_a(::ParseConfig)
+      expect(config.groups).to eq(%w(header header2))
+      expect(config.params).to eq(
+        'header' => { 'key' => 'val' },
+        'header2' => { 'key2' => 'val2' }
+      )
+    end
+  end
+
+  describe '.load' do
+    let(:config) do
+      Txgh::ParseConfig.load(contents)
+    end
+
+    it_behaves_like 'a correct config loader'
+  end
+
+  describe '.load_file' do
+    around(:each) do |example|
+      Tempfile.open('parseconfig-test') do |f|
+        f.write(contents)
+        f.flush
+        @file = f
+        example.run
+      end
+    end
+
+    let(:config) do
+      Txgh::ParseConfig.load_file(@file.path)
+    end
+
+    it_behaves_like 'a correct config loader'
+  end
+end

data/spec/resource_committer_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+require 'helpers/standard_txgh_setup'
+
+include Txgh
+
+describe ResourceCommitter do
+  include StandardTxghSetup
+
+  let(:language) { 'es' }
+  let(:resource) { tx_config.resource(resource_slug, branch) }
+  let(:downloader) { instance_double(ResourceDownloader) }
+  let(:file_name) { "translations/#{language}/sample.yml" }
+  let(:committer) do
+    ResourceCommitter.new(transifex_project, github_repo, logger)
+  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])
+
+      expect(github_api).to(
+        receive(:commit).with(
+          repo_name, branch, { file_name => :translations }
+        )
+      )
+
+      committer.commit_resource(resource, branch, language)
+    end
+
+    it "doesn't commit anything if the language is the source language" do
+      expect(github_api).to_not receive(:commit)
+      committer.commit_resource(resource, branch, resource.source_lang)
+    end
+
+    it "doesn't commit anything if the branch is protected" do
+      transifex_project.protected_branches << 'foobranch'
+      expect(github_api).to_not receive(:commit)
+      committer.commit_resource(resource, 'foobranch', resource.source_lang)
+    end
+  end
+end

data/spec/resource_contents_spec.rb

@@ -0,0 +1,212 @@
+require 'spec_helper'
+require 'stringio'
+
+include Txgh
+
+describe ResourceContents do
+  let(:tx_resource) do
+    TxResource.new(
+      'project_slug', 'resource_slug', 'YAML',
+      'en', 'source_file', '', 'translation_file'
+    )
+  end
+
+  let(:raw_contents) do
+    outdent(%Q(
+      en:
+        welcome:
+          message: Hello!
+        goodbye:
+          message: Goodbye!
+    ))
+  end
+
+  let(:contents) do
+    ResourceContents.from_string(tx_resource, raw_contents)
+  end
+
+  describe '#phrases' do
+    it 'extracts phrases from the resource contents' do
+      expect(contents.phrases).to eq([
+        { 'key' => 'welcome.message', 'string' => 'Hello!' },
+        { 'key' => 'goodbye.message', 'string' => 'Goodbye!' }
+      ])
+    end
+  end
+
+  describe '#add' do
+    it 'adds a new phrase' do
+      contents.add('foo.bar', 'baz')
+      expect(contents.phrases).to include(
+        { 'key' => 'foo.bar', 'string' => 'baz' }
+      )
+    end
+  end
+
+  describe '#write_to' do
+    it 'serializes the phrases to the given stream' do
+      stream = StringIO.new
+      contents.write_to(stream)
+      expect(stream.string).to eq(outdent(%Q(
+        en:
+          welcome:
+            message: ! "Hello!"
+          goodbye:
+            message: ! "Goodbye!"
+      )))
+    end
+
+    it 'includes phrases that were added after the fact' do
+      contents.add('foo.bar.baz', 'boo')
+
+      stream = StringIO.new
+      contents.write_to(stream)
+
+      expect(stream.string).to eq(outdent(%Q(
+        en:
+          welcome:
+            message: ! "Hello!"
+          goodbye:
+            message: ! "Goodbye!"
+          foo:
+            bar:
+              baz: ! "boo"
+      )))
+    end
+  end
+
+  describe '#to_s' do
+    it 'serializes to a string' do
+      expect(contents.to_s).to eq(outdent(%Q(
+        en:
+          welcome:
+            message: ! "Hello!"
+          goodbye:
+            message: ! "Goodbye!"
+      )))
+    end
+  end
+
+  describe '#empty?' do
+    it 'returns true if the resource contains no phrases' do
+      contents = ResourceContents.from_phrase_list(tx_resource, [])
+      expect(contents).to be_empty
+    end
+
+    it 'returns false if the resource contains phrases' do
+      expect(contents).to_not be_empty
+    end
+  end
+
+  describe '#diff' do
+    let(:head) do
+      ResourceContents.from_string(tx_resource, head_contents)
+    end
+
+    let(:diff_point) do
+      ResourceContents.from_string(tx_resource, diff_point_contents)
+    end
+
+    let(:diff) do
+      head.diff(diff_point)
+    end
+
+    context 'with phrases added to HEAD' do
+      let(:head_contents) do
+        outdent(%Q(
+          en:
+            welcome:
+              message: Hello!
+            goodbye:
+              message: Goodbye!
+        ))
+      end
+
+      let(:diff_point_contents) do
+        outdent(%Q(
+          en:
+            welcome:
+              message: Hello!
+        ))
+      end
+
+      it 'includes the added phrase' do
+        expect(diff.phrases).to eq([
+          { 'key' => 'goodbye.message', 'string' => 'Goodbye!' }
+        ])
+      end
+
+      it 'returns an instance of ResourceContents' do
+        expect(diff).to be_a(ResourceContents)
+      end
+    end
+
+    context 'with phrases removed from HEAD' do
+      let(:head_contents) do
+        outdent(%Q(
+          en:
+            welcome: Hello
+        ))
+      end
+
+      let(:diff_point_contents) do
+        outdent(%Q(
+          en:
+            welcome: Hello
+            goodbye: Goodbye
+        ))
+      end
+
+      it 'does not include any phrases' do
+        expect(diff.phrases).to eq([])
+      end
+    end
+
+    context 'with phrases modified in HEAD' do
+      let(:head_contents) do
+        outdent(%Q(
+          en:
+            welcome: Hello world
+            goodbye: Goodbye
+        ))
+      end
+
+      let(:diff_point_contents) do
+        outdent(%Q(
+          en:
+            welcome: Hello
+            goodbye: Goodbye
+        ))
+      end
+
+      it 'includes the modified phrase' do
+        expect(diff.phrases).to eq([
+          { 'key' => 'welcome', 'string' => 'Hello world' }
+        ])
+      end
+    end
+
+    context 'with no modifications or additions' do
+      let(:head_contents) do
+        outdent(%Q(
+          en:
+            welcome: Hello
+            goodbye: Goodbye
+        ))
+      end
+
+      let(:diff_point_contents) do
+        outdent(%Q(
+          en:
+            welcome: Hello
+            goodbye: Goodbye
+        ))
+      end
+
+      it 'hands back an empty diff' do
+        expect(diff.phrases).to eq([])
+      end
+    end
+
+  end
+end