danger-synx 0.1.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 17c29d9dae89afa6205d49c9f5125a4b3ba18153
+  data.tar.gz: e02f2dfcc0064d33226c27154e38a41bee247523
+SHA512:
+  metadata.gz: 6b222bb5bfe6b8367a8b540e123eb293e3b43b4b60540e1e38b5d8a6c649e9030ab15da4b8ac1b4cb60f71fc4628228150953fe4afea1c602a606b95af949556
+  data.tar.gz: c54aa133d280e38f433f3488effbabe6b2346abc87dc9b50f05fc93161d093bd28a13f5138ca76a7039dbe88af5c586ec1c2e11a2dc75e298ebe485fdfe58924

data/.gitignore

@@ -0,0 +1,54 @@
+*.gem
+*.rbc
+/.config
+/coverage/
+/InstalledFiles
+/pkg/
+/spec/reports/
+/spec/examples.txt
+/test/tmp/
+/test/version_tmp/
+/tmp/
+
+# Used by dotenv library to load environment variables.
+# .env
+
+## Specific to RubyMotion:
+.dat*
+.repl_history
+build/
+*.bridgesupport
+build-iPhoneOS/
+build-iPhoneSimulator/
+
+## Specific to RubyMotion (use of CocoaPods):
+#
+# We recommend against adding the Pods directory to your .gitignore. However
+# you should judge for yourself, the pros and cons are mentioned at:
+# https://guides.cocoapods.org/using/using-cocoapods.html#should-i-check-the-pods-directory-into-source-control
+#
+# vendor/Pods/
+
+## Documentation cache and generated files:
+/.yardoc/
+/_yardoc/
+/doc/
+/rdoc/
+
+## Environment normalization:
+/.bundle/
+/vendor/bundle
+/lib/bundler/man/
+
+# for a library or gem, you might want to ignore these files since the code is
+# intended to run in multiple environments; otherwise, check them in:
+# Gemfile.lock
+# .ruby-version
+# .ruby-gemset
+
+# unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
+.rvmrc
+=======
+.DS_Store
+pkg
+.idea/

data/.travis.yml

@@ -0,0 +1,12 @@
+language: ruby
+cache:
+  directories:
+    - bundle
+
+rvm:
+  - 2.0
+  - 2.1.3
+  - 2.3.1
+
+script:
+    - bundle exec rake spec

data/Gemfile

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+
+# Specify your gem's dependencies in danger-synx.gemspec
+gemspec

data/Gemfile.lock

@@ -0,0 +1,159 @@
+PATH
+  remote: .
+  specs:
+    danger-synx (0.1.0)
+      danger-plugin-api (~> 1.0)
+      synx
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    CFPropertyList (2.3.5)
+    activesupport (5.0.1)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (~> 0.7)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+    addressable (2.5.0)
+      public_suffix (~> 2.0, >= 2.0.2)
+    ast (2.3.0)
+    claide (1.0.1)
+    claide-plugins (0.9.2)
+      cork
+      nap
+      open4 (~> 1.3)
+    clamp (0.6.5)
+    coderay (1.1.1)
+    colored (1.2)
+    colored2 (3.1.1)
+    colorize (0.8.1)
+    concurrent-ruby (1.0.4)
+    cork (0.2.0)
+      colored (~> 1.2)
+    danger (4.2.0)
+      claide (~> 1.0)
+      claide-plugins (>= 0.9.2)
+      colored2 (~> 3.1)
+      cork (~> 0.1)
+      faraday (~> 0.9)
+      faraday-http-cache (~> 1.0)
+      git (~> 1)
+      kramdown (~> 1.5)
+      octokit (~> 4.2)
+      terminal-table (~> 1)
+    danger-plugin-api (1.0.0)
+      danger (> 2.0)
+    diff-lcs (1.3)
+    faraday (0.11.0)
+      multipart-post (>= 1.2, < 3)
+    faraday-http-cache (1.3.1)
+      faraday (~> 0.8)
+    ffi (1.9.17)
+    formatador (0.2.5)
+    git (1.3.0)
+    guard (2.14.1)
+      formatador (>= 0.2.4)
+      listen (>= 2.7, < 4.0)
+      lumberjack (~> 1.0)
+      nenv (~> 0.1)
+      notiffany (~> 0.0)
+      pry (>= 0.9.12)
+      shellany (~> 0.0)
+      thor (>= 0.18.1)
+    guard-compat (1.2.1)
+    guard-rspec (4.7.3)
+      guard (~> 2.1)
+      guard-compat (~> 1.1)
+      rspec (>= 2.99.0, < 4.0)
+    i18n (0.8.0)
+    kramdown (1.13.2)
+    listen (3.0.7)
+      rb-fsevent (>= 0.9.3)
+      rb-inotify (>= 0.9.7)
+    lumberjack (1.0.11)
+    method_source (0.8.2)
+    minitest (5.10.1)
+    multipart-post (2.0.0)
+    nanaimo (0.2.3)
+    nap (1.1.0)
+    nenv (0.3.0)
+    notiffany (0.1.1)
+      nenv (~> 0.1)
+      shellany (~> 0.0)
+    octokit (4.6.2)
+      sawyer (~> 0.8.0, >= 0.5.3)
+    open4 (1.3.4)
+    parser (2.3.3.1)
+      ast (~> 2.2)
+    powerpack (0.1.1)
+    pry (0.10.4)
+      coderay (~> 1.1.0)
+      method_source (~> 0.8.1)
+      slop (~> 3.4)
+    public_suffix (2.0.5)
+    rainbow (2.2.1)
+    rake (10.5.0)
+    rb-fsevent (0.9.8)
+    rb-inotify (0.9.8)
+      ffi (>= 0.5.0)
+    rspec (3.5.0)
+      rspec-core (~> 3.5.0)
+      rspec-expectations (~> 3.5.0)
+      rspec-mocks (~> 3.5.0)
+    rspec-core (3.5.4)
+      rspec-support (~> 3.5.0)
+    rspec-expectations (3.5.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.5.0)
+    rspec-mocks (3.5.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.5.0)
+    rspec-support (3.5.0)
+    rubocop (0.47.1)
+      parser (>= 2.3.3.1, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 1.99.1, < 3.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.8.1)
+    sawyer (0.8.1)
+      addressable (>= 2.3.5, < 2.6)
+      faraday (~> 0.8, < 1.0)
+    shellany (0.0.1)
+    slop (3.6.0)
+    synx (0.2.1)
+      clamp (~> 0.6)
+      colorize (~> 0.7)
+      xcodeproj (~> 1.0)
+    terminal-table (1.7.3)
+      unicode-display_width (~> 1.1.1)
+    thor (0.19.4)
+    thread_safe (0.3.5)
+    tzinfo (1.2.2)
+      thread_safe (~> 0.1)
+    unicode-display_width (1.1.3)
+    xcodeproj (1.4.2)
+      CFPropertyList (~> 2.3.3)
+      activesupport (>= 3)
+      claide (>= 1.0.1, < 2.0)
+      colored (~> 1.2)
+      nanaimo (~> 0.2.3)
+    yard (0.9.8)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler (~> 1.3)
+  danger-synx!
+  guard (~> 2.14)
+  guard-rspec (~> 4.7)
+  listen (= 3.0.7)
+  pry
+  rake (~> 10.0)
+  rspec (~> 3.4)
+  rubocop (~> 0.41)
+  yard (~> 0.8)
+
+BUNDLED WITH
+   1.13.6

data/Guardfile

@@ -0,0 +1,19 @@
+# A guardfile for making Danger Plugins
+# For more info see https://github.com/guard/guard#readme
+
+# To run, use `bundle exec guard`.
+
+guard :rspec, cmd: 'bundle exec rspec' do
+  require 'guard/rspec/dsl'
+  dsl = Guard::RSpec::Dsl.new(self)
+
+  # RSpec files
+  rspec = dsl.rspec
+  watch(rspec.spec_helper) { rspec.spec_dir }
+  watch(rspec.spec_support) { rspec.spec_dir }
+  watch(rspec.spec_files)
+
+  # Ruby files
+  ruby = dsl.ruby
+  dsl.watch_spec_files_for(ruby.lib_files)
+end

data/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2017 Jakub Turek
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

data/LICENSE.txt

@@ -0,0 +1,22 @@
+Copyright (c) 2017 Jakub Turek <jkbturek@gmail.com>
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -0,0 +1,43 @@
+# Danger Synx
+
+A Danger plugin for [Synx](https://github.com/venmo/synx/). 
+
+## Installation
+
+Currently only Bundler installations are supported. Add following contents to your `Gemfile`:
+
+    gem 'danger-synx'
+    gem 'synx', :github => 'turekj/synx', :branch => 'v0.3'
+    
+And run `bundle install`. 
+
+The plugin relies on a Synx fork because [PR #125](https://github.com/venmo/synx/pull/125) is not yet merged with the core app.
+
+## Usage
+
+Add following line to the `Dangerfile`:
+
+    synx.ensure_clean_structure
+
+`ensure_clean_structure` task runs Synx on every `.xcodeproj` file that is added/modified by a pull request. Issues are gathered and reported in a following format:
+
+| Project file | Issue |
+| --- | --- |
+| MessyProject.xcodeproj | File reference RootController.swift is not synchronized with file system. |
+| MessyProject.xcodeproj | Group /MessyGroup is not sorted alphabetically. |
+
+## Roadmap
+
+- [x] Reporting synx issues for added/modified `.xcodeproj` files.
+- [ ] Indicate unused files (`--prune`).
+- [ ] Allow custom exclusions for directories (`--exclusion`).
+- [ ] Support for toggleable unsorted group reports (`--no-sort-by-name`).
+- [ ] Support for `--no-sort-by-name` option.
+
+## Development
+
+1. Clone this repo
+2. Run `bundle install` to setup dependencies.
+3. Run `bundle exec rspec` to run the tests.
+4. Use `bundle exec guard` to automatically have tests run as you make changes.
+5. Make your changes.

data/Rakefile

@@ -0,0 +1,23 @@
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
+
+RSpec::Core::RakeTask.new(:specs)
+
+task default: :specs
+
+task :spec do
+  Rake::Task['specs'].invoke
+  Rake::Task['rubocop'].invoke
+  Rake::Task['spec_docs'].invoke
+end
+
+desc 'Run RuboCop on the lib/specs directory'
+RuboCop::RakeTask.new(:rubocop) do |task|
+  task.patterns = ['lib/**/*.rb', 'spec/**/*.rb']
+end
+
+desc 'Ensure that the plugin passes `danger plugins lint`'
+task :spec_docs do
+  sh 'bundle exec danger plugins lint'
+end

data/danger-synx.gemspec

@@ -0,0 +1,50 @@
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'synx/gem_version.rb'
+
+Gem::Specification.new do |spec|
+  spec.name          = 'danger-synx'
+  spec.version       = Synx::VERSION
+  spec.authors       = ['Jakub Turek']
+  spec.email         = ['jkbturek@gmail.com']
+  spec.description   = %q{A Danger plugin for running commited .xcodeproj through Synx.}
+  spec.summary       = %q{A Danger plugin for running commited .xcodeproj through Synx.}
+  spec.homepage      = 'https://github.com/elpassion/danger-synx'
+  spec.license       = 'MIT'
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ['lib']
+
+  spec.add_runtime_dependency 'danger-plugin-api', '~> 1.0'
+  spec.add_dependency 'synx'
+
+  # General ruby development
+  spec.add_development_dependency 'bundler', '~> 1.3'
+  spec.add_development_dependency 'rake', '~> 10.0'
+
+  # Testing support
+  spec.add_development_dependency 'rspec', '~> 3.4'
+
+  # Linting code and docs
+  spec.add_development_dependency "rubocop", "~> 0.41"
+  spec.add_development_dependency "yard", "~> 0.8"
+
+  # Makes testing easy via `bundle exec guard`
+  spec.add_development_dependency 'guard', '~> 2.14'
+  spec.add_development_dependency 'guard-rspec', '~> 4.7'
+
+  # If you want to work on older builds of ruby
+  spec.add_development_dependency 'listen', '3.0.7'
+
+  # This gives you the chance to run a REPL inside your tests
+  # via:
+  #
+  #    require 'pry'
+  #    binding.pry
+  #
+  # This will stop test execution and let you inspect the results
+  spec.add_development_dependency 'pry'
+end

data/lib/danger_plugin.rb

@@ -0,0 +1 @@
+require 'synx/plugin'

data/lib/danger_synx.rb

@@ -0,0 +1 @@
+require 'synx/gem_version'

data/lib/synx/gem_version.rb

@@ -0,0 +1,3 @@
+module Synx
+  VERSION = '0.1.0'.freeze
+end

data/lib/synx/plugin.rb

@@ -0,0 +1,122 @@
+module Danger
+  # Enforces that .xcodeproj structure is tidy.
+  # It wraps around [Synx](https://github.com/venmo/synx)
+  # tool to perform the check.
+  #
+  # @example Ensure that all added / modified project files are synchronized
+  #
+  #          danger_synx.ensure_clean_structure
+  #
+  # @see  turekj/danger-synx
+  # @tags synx, xcodeproj
+  #
+  class DangerSynx < Plugin
+    # Ensures clean project structure. Runs Synx on all .xcodeproj
+    # files that where either added or modified.
+    #
+    # @return   [void]
+    #
+    def ensure_clean_structure
+      unless synx_installed?
+        raise 'synx > 0.2.1 not found in PATH and failed to install'
+      end
+
+      generate_output synx_issues
+    end
+
+    # Checks whether Synx in a correct version is installed in the system.
+    # If not, tries to recover by installing it.
+    # Returns true if Synx is present or installation was successful.
+    #
+    # @return [bool]
+    #
+    def precheck_synx_installation?
+      `gem install synx` unless synx_installed?
+      synx_installed?
+    end
+
+    # Returns a command to run for synx
+    #
+    # @return [String]
+    def synx
+      "#{'bundle exec ' if File.exist?('Gemfile')}synx"
+    end
+
+    # Tests whether Synx is already installed and meets minimal
+    # version requirements.
+    #
+    # @return [bool]
+    #
+    def synx_installed?
+      match = `#{synx} --version`.match(/Synx (\d+)\.(\d+)\.(\d+)/i)
+      if match
+        major, minor, patch = match.captures.map { |c| Integer(c) }
+        major > 0 || minor > 2 || (minor == 2 && patch > 1)
+      else
+        false
+      end
+    end
+
+    # Triggers Synx on all projects that were modified or added
+    # to the project. Returns accumulated list of issues
+    # for those projects.
+    #
+    # @return [Array<(String, String)>]
+    #
+    def synx_issues
+      (git.modified_files + git.added_files)
+        .select { |f| f.include? '.xcodeproj' }
+        .reduce([]) { |i, f| i + synx_project(f) }
+    end
+
+    # Triggers Synx in a dry-run mode on a project file.
+    # Parses output and returns a list of issues.
+    #
+    # @param  [String] modified_file_path
+    #         Path of file contained in .xcodeproj to Synx
+    #
+    # @return [(String, String)]
+    #
+    def synx_project(modified_file_path)
+      path = project_path modified_file_path
+      name = project_name path
+      output = `#{synx} -w warning "#{path}" 2>&1`.lines
+      output
+        .map(&:strip)
+        .select { |o| o.start_with? 'warning: ' }
+        .map { |o| [name, strip_prefix(o)] }
+    end
+
+    private
+
+    def project_path(modified_file_path)
+      match = modified_file_path.match('(.+\.xcodeproj)*+')
+      match[0] if match
+    end
+
+    def project_name(project_path)
+      project_path.split('/').last
+    end
+
+    def strip_prefix(output_line)
+      output_line.slice(9, output_line.size - 9)
+    end
+
+    def generate_output(issues)
+      return if issues.empty?
+
+      warn("Synx detected #{issues.size} structural issue(s)")
+
+      message = "### Synx structural issues\n\n"
+      message << "| Project file | Issue |\n"
+      message << "| --- | --- |\n"
+
+      issues.each do |(project, issue)|
+        message << "| #{project} | #{issue} |\n"
+      end
+
+      markdown message
+    end
+
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,59 @@
+require 'pathname'
+ROOT = Pathname.new(File.expand_path('../../', __FILE__))
+$:.unshift((ROOT + 'lib').to_s)
+$:.unshift((ROOT + 'spec').to_s)
+
+require 'bundler/setup'
+require 'pry'
+
+require 'rspec'
+require 'danger'
+
+# Use coloured output, it's the best.
+RSpec.configure do |config|
+  config.filter_gems_from_backtrace "bundler"
+  config.color = true
+  config.tty = true
+end
+
+require 'danger_plugin'
+
+# These functions are a subset of https://github.com/danger/danger/blob/master/spec/spec_helper.rb
+# If you are expanding these files, see if it's already been done ^.
+
+# A silent version of the user interface,
+# it comes with an extra function `.string` which will
+# strip all ANSI colours from the string.
+
+# rubocop:disable Lint/NestedMethodDefinition
+def testing_ui
+  @output = StringIO.new
+  def @output.winsize
+    [20, 9999]
+  end
+
+  cork = Cork::Board.new(out: @output)
+  def cork.string
+    out.string.gsub(/\e\[([;\d]+)?m/, "")
+  end
+  cork
+end
+# rubocop:enable Lint/NestedMethodDefinition
+
+# Example environment (ENV) that would come from
+# running a PR on TravisCI
+def testing_env
+  {
+    'HAS_JOSH_K_SEAL_OF_APPROVAL' => 'true',
+    'TRAVIS_PULL_REQUEST' => '800',
+    'TRAVIS_REPO_SLUG' => 'artsy/eigen',
+    'TRAVIS_COMMIT_RANGE' => '759adcbd0d8f...13c4dc8bb61d',
+    'DANGER_GITHUB_API_TOKEN' => '123sbdq54erfsd3422gdfio'
+  }
+end
+
+# A stubbed out Dangerfile for use in tests
+def testing_dangerfile
+  env = Danger::EnvironmentManager.new(testing_env)
+  Danger::Dangerfile.new(env, testing_ui)
+end

data/spec/synx_spec.rb

@@ -0,0 +1,153 @@
+require File.expand_path('../spec_helper', __FILE__)
+
+module Danger
+  describe Danger::DangerSynx do
+    it 'should be a plugin' do
+      expect(Danger::DangerSynx.new(nil)).to be_a Danger::Plugin
+    end
+
+    describe 'with Dangerfile' do
+      before do
+        @dangerfile = testing_dangerfile
+        @synx = @dangerfile.synx
+      end
+
+      describe :synx do
+        it 'should return synx when Gemfile is not present' do
+          allow(File).to receive(:exist?).with('Gemfile').and_return(false)
+          expect(@synx.synx).to eq('synx')
+        end
+
+        it 'should return bundle exec synx when Gemfile is present' do
+          allow(File).to receive(:exist?).with('Gemfile').and_return(true)
+          expect(@synx.synx).to eq('bundle exec synx')
+        end
+      end
+
+      describe :synx_installed? do
+        before do
+          allow(File).to receive(:exist?).with('Gemfile').and_return(true)
+        end
+
+        it 'reports that synx is not installed if executable is not found' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('sh: synx not found')
+          expect(@synx.synx_installed?).to be_falsy
+        end
+
+        it 'reports that synx is not installed if version is 0.1.5' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.1.5')
+          expect(@synx.synx_installed?).to be_falsy
+        end
+
+        it 'reports that synx is not installed if version is 0.2.1' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.2.1')
+          expect(@synx.synx_installed?).to be_falsy
+        end
+
+        it 'reports that synx is installed if version is 0.2.2' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.2.2')
+          expect(@synx.synx_installed?).to be_truthy
+        end
+
+        it 'reports that synx is installed if version is 0.3.0' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.3.0')
+          expect(@synx.synx_installed?).to be_truthy
+        end
+
+        it 'reports that synx is installed if version is 1.0.0' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 1.0.0')
+          expect(@synx.synx_installed?).to be_truthy
+        end
+      end
+
+      describe :precheck_synx_installation do
+        before do
+          allow(File).to receive(:exist?).with('Gemfile').and_return(true)
+        end
+
+        it 'should install synx if needed' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('sh: synx not found')
+          expect(@synx).to receive(:`).with('gem install synx')
+          @synx.precheck_synx_installation?
+        end
+
+        it 'should report that synx is installed correctly' do
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.3.0')
+          expect(@synx.precheck_synx_installation?).to be_truthy
+        end
+      end
+
+      describe :synx_issues do
+        before do
+          allow(File).to receive(:exist?).with('Gemfile').and_return(true)
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.3.0')
+        end
+
+        it 'should return a list of issues found in all projects' do
+          allow(@synx.git).to receive(:modified_files).and_return(['A.xcodeproj/project.pbxproj', 'B.xcodeproj/project.xcworkspace'])
+          allow(@synx.git).to receive(:added_files).and_return([])
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "A.xcodeproj" 2>&1').and_return("warning: Warning.\nwarning: Another warning.\n")
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "B.xcodeproj" 2>&1').and_return("warning: Issue.\n")
+          expect(@synx.synx_issues).to match_array([['A.xcodeproj', 'Warning.'], ['A.xcodeproj', 'Another warning.'], ['B.xcodeproj', 'Issue.']])
+        end
+      end
+
+      describe :ensure_clean_structure do
+        before do
+          allow(File).to receive(:exist?).with('Gemfile').and_return(true)
+          allow(@synx).to receive(:`).with('bundle exec synx --version').and_return('Synx 0.3.0')
+        end
+
+        it 'should trigger synx for modified project files' do
+          allow(@synx.git).to receive(:modified_files).and_return(['Project/Sources/AppDelegate.swift', 'Project/Project.xcodeproj/project.pbxproj'])
+          allow(@synx.git).to receive(:added_files).and_return(['Other Project/Other Project.xcodeproj/project.pbxproj', 'Other Project/Resources/image.png/project.xcworkspace'])
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "Project/Project.xcodeproj" 2>&1').and_return('')
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "Other Project/Other Project.xcodeproj" 2>&1').and_return('')
+          @synx.ensure_clean_structure
+        end
+
+        it 'should output a warning with number of issues' do
+          allow(@synx.git).to receive(:modified_files).and_return(['A.xcodeproj/project.pbxproj', 'B.xcodeproj/project.xcworkspace'])
+          allow(@synx.git).to receive(:added_files).and_return([])
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "A.xcodeproj" 2>&1').and_return("warning: Warning.\nwarning: Another warning.\n")
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "B.xcodeproj" 2>&1').and_return("warning: Issue.\n")
+          expect(@synx).to receive(:warn).with('Synx detected 3 structural issue(s)')
+          @synx.ensure_clean_structure
+        end
+
+        it 'should not output warning if there are no issues' do
+          allow(@synx.git).to receive(:modified_files).and_return(['A.xcodeproj/project.pbxproj', 'B.xcodeproj/project.xcworkspace'])
+          allow(@synx.git).to receive(:added_files).and_return([])
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "A.xcodeproj" 2>&1').and_return('')
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "B.xcodeproj" 2>&1').and_return('')
+          expect(@synx).to_not receive(:warn)
+          @synx.ensure_clean_structure
+        end
+
+        it 'should not output markdown when there are no issues' do
+          allow(@synx.git).to receive(:modified_files).and_return(['A.xcodeproj/project.pbxproj', 'B.xcodeproj/project.xcworkspace'])
+          allow(@synx.git).to receive(:added_files).and_return([])
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "A.xcodeproj" 2>&1').and_return('')
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "B.xcodeproj" 2>&1').and_return('')
+          expect(@synx).to_not receive(:markdown)
+          @synx.ensure_clean_structure
+        end
+
+        it 'should output table with issues as markdown' do
+          allow(@synx.git).to receive(:modified_files).and_return(['A.xcodeproj/project.pbxproj', 'B.xcodeproj/project.xcworkspace'])
+          allow(@synx.git).to receive(:added_files).and_return([])
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "A.xcodeproj" 2>&1').and_return("warning: Warning.\nwarning: Another warning.\n")
+          expect(@synx).to receive(:`).with('bundle exec synx -w warning "B.xcodeproj" 2>&1').and_return("warning: Issue.\n")
+
+          @synx.ensure_clean_structure
+          output = @synx.status_report[:markdowns].first.to_s
+
+          expect(output).to include('Synx structural issues')
+          expect(output).to include('A.xcodeproj | Warning.')
+          expect(output).to include('A.xcodeproj | Another warning.')
+          expect(output).to include('B.xcodeproj | Issue.')
+        end
+      end
+    end
+  end
+end

metadata

@@ -0,0 +1,216 @@
+--- !ruby/object:Gem::Specification
+name: danger-synx
+version: !ruby/object:Gem::Version
+  version: 0.1.0
+platform: ruby
+authors:
+- Jakub Turek
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2017-02-06 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: danger-plugin-api
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: synx
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.41'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.41'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.8'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.8'
+- !ruby/object:Gem::Dependency
+  name: guard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.14'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+- !ruby/object:Gem::Dependency
+  name: listen
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.7
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.7
+- !ruby/object:Gem::Dependency
+  name: pry
+  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: A Danger plugin for running commited .xcodeproj through Synx.
+email:
+- jkbturek@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".travis.yml"
+- Gemfile
+- Gemfile.lock
+- Guardfile
+- LICENSE
+- LICENSE.txt
+- README.md
+- Rakefile
+- danger-synx.gemspec
+- lib/danger_plugin.rb
+- lib/danger_synx.rb
+- lib/synx/gem_version.rb
+- lib/synx/plugin.rb
+- spec/spec_helper.rb
+- spec/synx_spec.rb
+homepage: https://github.com/elpassion/danger-synx
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.6.10
+signing_key: 
+specification_version: 4
+summary: A Danger plugin for running commited .xcodeproj through Synx.
+test_files:
+- spec/spec_helper.rb
+- spec/synx_spec.rb