migration_investigation 0.0.2 → 0.0.3

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 91513fab4ee07b15ace6022637aac1c4a6b0d63c
+  data.tar.gz: 678ac5207077c480991cbf8434c782dad0723d51
+SHA512:
+  metadata.gz: 8e932ec123891349b920a8bd9b6d0f7c54eea89e0fd5bc39900635bb7beda5bac25f0bede1f84524e00bed31e51c38f446b60c740ad77f0c706c3f2b4822cc2f
+  data.tar.gz: ab1cc4ed15336aae4d5ef5a5409f5f873690db37e5e9b6467272867d691c8da11889ff8786099eae9ca639779c9280dc159445faacb4e5624caa40553d932d5f

data/Gemfile

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

data/README.md

@@ -1,12 +1,14 @@
 # MigrationInvestigation
 
-TODO: Write a gem description
+This gem will check to see if you need to run migrations without loading your environment.
+It assumes that you are using [auto_tagger](https://github.com/zilkey/auto_tagger) to tag the SHA
+that has been deployed.
 
 ## Installation
 
 Add this line to your application's Gemfile:
 
-    gem 'migration_investigation'
+    gem "migration_investigation"
 
 And then execute:
 
@@ -18,12 +20,22 @@ Or install it yourself as:
 
 ## Usage
 
-TODO: Write usage instructions here
+The gem is very simple:
+
+  $ MigrationInvestigation.pending_migrations? "staging", "abc123"
+
+Where `staging` is the name of the tag, and `abc123` is the SHA you're about to deploy.
+You can also pass nil for the SHA you're about to deploy and it will assume you're deloying `HEAD`.
+
+If you'd also like to check additional files or directories, say seed data, you can pass in a third
+parameter:
+
+  $ MigrationInvestigation.pending_migrations? "staging", "abc123", additional_paths: ["db/seed.rb", "app/foo.txt"]
 
 ## Contributing
 
 1. Fork it
 2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Add some feature'`)
+3. Commit your changes (`git commit -am "Add some feature"`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request

data/lib/migration_investigation.rb

@@ -2,20 +2,23 @@ require "migration_investigation/version"
 
 module MigrationInvestigation
   class << self
-    def pending_migrations? stage, new_sha
+    def pending_migrations?(stage, new_sha, options={})
+      interesting_paths = options.fetch(:additional_paths, [])
+      interesting_paths = [interesting_paths] unless interesting_paths.is_a?(Array)
+      interesting_paths << "db/migrate/"
       old_sha = get_latest_sha stage
-      change_count = get_change_count old_sha, new_sha
+      change_count = get_change_count old_sha, new_sha, interesting_paths
       change_count != 0
     end
 
     private
 
-    def get_latest_sha stage
+    def get_latest_sha(stage)
       AutoTagger::Base.new({}).refs_for_stage(stage).last.sha
     end
 
-    def get_change_count old_sha, new_sha
-      changes = `git diff --name-only #{old_sha} #{new_sha} db/migrate/ | wc -l`
+    def get_change_count(old_sha, new_sha, interesting_paths)
+      changes = `git diff --name-only #{old_sha} #{new_sha} #{interesting_paths.join(" ")} | wc -l`
       changes.to_i
     end
   end

data/lib/migration_investigation/version.rb

@@ -1,3 +1,3 @@
 module MigrationInvestigation
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/migration_investigation.gemspec

@@ -1,15 +1,15 @@
 # -*- encoding: utf-8 -*-
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'migration_investigation/version'
+require "migration_investigation/version"
 
 Gem::Specification.new do |gem|
   gem.name          = "migration_investigation"
   gem.version       = MigrationInvestigation::VERSION
   gem.authors       = ["Brent Wheeldon"]
   gem.email         = ["brent.wheeldon@gmail.com"]
-  gem.description   = %q{Checks whether migrations need to be run, without loading your environment}
-  gem.summary       = %q{Checks whether migrations need to be run, without loading your environment}
+  gem.description   = %q{Checks whether migrations, or other operations, need to be run, without loading your environment}
+  gem.summary       = %q{Checks whether migrations, or other parts of your repo, have any changes.}
   gem.homepage      = "https://github.com/BrentWheeldon/migration_investigation"
   gem.license       = "MIT"
 
@@ -18,6 +18,6 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.add_dependency 'auto_tagger'
-  gem.add_development_dependency 'rspec'
+  gem.add_dependency "auto_tagger", "~> 0"
+  gem.add_development_dependency "rspec", "~> 3"
 end

data/spec/migration_investigation_spec.rb

@@ -1,34 +1,59 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe MigrationInvestigation do
   describe ".pending_migrations?" do
-    subject { MigrationInvestigation.pending_migrations? stage, new_sha }
-
     let(:stage) { "stage" }
     let(:new_sha) { "abc123" }
 
     let(:refs) { [ref1, ref2] }
-    let(:ref1) { stub(:ref1) }
-    let(:ref2) { stub(:ref2, sha: old_sha) }
+    let(:ref1) { double(:ref1) }
+    let(:ref2) { double(:ref2, sha: old_sha) }
     let(:old_sha) { "cba321" }
 
     before do
-      auto_tagger = stub(:auto_tagger)
-      AutoTagger::Base.stub(:new).with({}) { auto_tagger }
-      auto_tagger.stub(:refs_for_stage).with(stage) { refs }
-      MigrationInvestigation.stub(:`).with("git diff --name-only #{old_sha} #{new_sha} db/migrate/ | wc -l") { git_diff }
+      auto_tagger = double(:auto_tagger)
+      allow(AutoTagger::Base).to receive(:new).with({}).and_return(auto_tagger)
+      allow(auto_tagger).to receive(:refs_for_stage).with(stage).and_return(refs)
     end
 
-    context "when there are migrations changed between the last tag for the given change, and the new SHA" do
-      let(:git_diff) { "       3\n" }
+    context "when no additional paths are passed in" do
+      subject { MigrationInvestigation.pending_migrations? stage, new_sha }
+
+      before do
+        allow(MigrationInvestigation).to receive(:`).with("git diff --name-only #{old_sha} #{new_sha} db/migrate/ | wc -l").and_return(git_diff)
+      end
+
+      context "when there are migrations changed between the last tag for the given change, and the new SHA" do
+        let(:git_diff) { "       3\n" }
+
+        it { is_expected.to be_truthy }
+      end
+
+      context "when there are no migrations changes between the last tag for the given change, and the new SHA" do
+        let(:git_diff) { "       0\n" }
 
-      it { should be_true }
+        it { is_expected.to be_falsy }
+      end
     end
 
-    context "when there are no migrations changes between the last tag for the given change, and the new SHA" do
-      let(:git_diff) { "       0\n" }
+    context "when additional paths are passed in" do
+      subject { MigrationInvestigation.pending_migrations? stage, new_sha, additional_paths: ["app/foo.rb"]  }
+
+      before do
+        allow(MigrationInvestigation).to receive(:`).with("git diff --name-only #{old_sha} #{new_sha} app/foo.rb db/migrate/ | wc -l").and_return(git_diff)
+      end
+
+      context "when there are changes between the last tag for the given change, and the new SHA" do
+        let(:git_diff) { "       3\n" }
+
+        it { is_expected.to be_truthy }
+      end
+
+      context "when there are no changes between the last tag for the given change, and the new SHA" do
+        let(:git_diff) { "       0\n" }
 
-      it { should be_false }
+        it { is_expected.to be_falsy }
+      end
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,9 +1,6 @@
-require 'migration_investigation'
-require 'auto_tagger'
+require "migration_investigation"
+require "auto_tagger"
 
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
-  config.run_all_when_everything_filtered = true
-  config.filter_run :focus
-  config.order = 'random'
+  config.order = "random"
 end

metadata

@@ -1,57 +1,53 @@
 --- !ruby/object:Gem::Specification
 name: migration_investigation
 version: !ruby/object:Gem::Version
-  version: 0.0.2
-  prerelease: 
+  version: 0.0.3
 platform: ruby
 authors:
 - Brent Wheeldon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-29 00:00:00.000000000 Z
+date: 2015-04-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: auto_tagger
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-description: Checks whether migrations need to be run, without loading your environment
+        version: '3'
+description: Checks whether migrations, or other operations, need to be run, without
+  loading your environment
 email:
 - brent.wheeldon@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
+- ".gitignore"
+- ".rspec"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -64,28 +60,27 @@ files:
 homepage: https://github.com/BrentWheeldon/migration_investigation
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 2.4.6
 signing_key: 
-specification_version: 3
-summary: Checks whether migrations need to be run, without loading your environment
+specification_version: 4
+summary: Checks whether migrations, or other parts of your repo, have any changes.
 test_files:
 - spec/migration_investigation_spec.rb
 - spec/spec_helper.rb