okuribito_rails 0.0.2 → 0.1.0

data/spec/factories.rb

@@ -1,14 +1,14 @@
 FactoryGirl.define do
   factory :method_call_situation, class: OkuribitoRails::MethodCallSituation do
-    class_name "User"
-    method_symbol "."
-    method_name "feed"
+    class_name "TestTarget"
+    method_symbol "#"
+    method_name "deprecated_method"
     called_num 0
   end
 
   factory :method_call_log, class: OkuribitoRails::MethodCallLog do
-    class_name "User"
-    method_symbol "."
-    method_name "feed"
+    class_name "TestTarget"
+    method_symbol "#"
+    method_name "deprecated_method"
   end
 end

data/spec/lib/okuribito_rails/observe_method_spec.rb

@@ -0,0 +1,33 @@
+require "spec_helper"
+require "support/test_target"
+
+describe OkuribitoRails::ObserveMethod do
+  let(:setting_path) { "spec/support/test_config.yml" }
+
+  describe "create MethodCallLog" do
+    before do
+      allow_any_instance_of(OkuribitoRails::ObserveMethod).to receive(:config_once_detect).and_return(true)
+      FactoryGirl.create(:method_call_situation)
+      okuribito = OkuribitoRails::ObserveMethod.new.patch_okuribito
+      okuribito.apply(setting_path)
+    end
+
+    subject { OkuribitoRails::MethodCallLog.count }
+
+    context "call patched method" do
+      before do
+        TestTarget.new.deprecated_method
+      end
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "call unpatched method" do
+      before do
+        TestTarget.deprecated_self_method
+      end
+
+      it { is_expected.to eq 0 }
+    end
+  end
+end

data/spec/lib/okuribito_rails/regist_method_spec.rb

@@ -0,0 +1,40 @@
+require "spec_helper"
+require "support/test_target"
+
+shared_examples "Evaluate the condition of MethodCallSituation" do
+  it { expect(OkuribitoRails::MethodCallSituation.find_by(class_name: "TestTarget", method_symbol: ".", method_name: "deprecated_self_method")).to be_present }
+  it { expect(OkuribitoRails::MethodCallSituation.find_by(class_name: "TestTarget", method_symbol: "#", method_name: "deprecated_method")).to be_present }
+end
+
+describe OkuribitoRails::RegistMethod do
+  let(:setting_path) { "spec/support/test_config.yml" }
+
+  describe "create MethodCallSituation" do
+    context "registered none" do
+      before do
+        OkuribitoRails::RegistMethod.new.update_observe_methods(setting_path)
+      end
+
+      it_behaves_like "Evaluate the condition of MethodCallSituation"
+    end
+
+    context "registered one (Already registered)" do
+      before do
+        FactoryGirl.create(:method_call_situation, class_name: "TestTarget", method_symbol: "#", method_name: "deprecated_method")
+        OkuribitoRails::RegistMethod.new.update_observe_methods(setting_path)
+      end
+
+      it_behaves_like "Evaluate the condition of MethodCallSituation"
+    end
+
+    context "registered one (Delete target)" do
+      before do
+        FactoryGirl.create(:method_call_situation, class_name: "TestTarget", method_symbol: "#", method_name: "delete_target_method")
+        OkuribitoRails::RegistMethod.new.update_observe_methods(setting_path)
+      end
+
+      it_behaves_like "Evaluate the condition of MethodCallSituation"
+      it { expect(OkuribitoRails::MethodCallSituation.find_by(class_name: "TestTarget", method_symbol: "#", method_name: "delete_target_method")).not_to be_present }
+    end
+  end
+end

data/spec/lib/okuribito_rails/start_observer_spec.rb

@@ -0,0 +1,36 @@
+require "spec_helper"
+
+describe OkuribitoRails::StartObserver do
+  describe "#start" do
+    let(:prohibit_env) { false }
+    let(:before_migrate) { false }
+    let(:setting_path) { "spec/support/test_config.yml" }
+    let(:start_observer) { OkuribitoRails::StartObserver.new }
+
+    before do
+      allow(start_observer).to receive(:prohibit_env?).and_return(prohibit_env)
+      allow(start_observer).to receive(:before_migrate?).and_return(before_migrate)
+      allow(start_observer).to receive(:setting_path).and_return(setting_path)
+      allow(start_observer).to receive(:regist_method)
+      allow(start_observer).to receive(:start_observer)
+      start_observer.start
+    end
+
+    context "allowed env / after_migrate" do
+      it { expect(start_observer).to have_received(:regist_method) }
+      it { expect(start_observer).to have_received(:start_observer) }
+    end
+
+    context "prohibited env" do
+      let(:prohibit_env) { true }
+      it { expect(start_observer).not_to have_received(:regist_method) }
+      it { expect(start_observer).not_to have_received(:start_observer) }
+    end
+
+    context "before_migrate" do
+      let(:before_migrate) { true }
+      it { expect(start_observer).not_to have_received(:regist_method) }
+      it { expect(start_observer).not_to have_received(:start_observer) }
+    end
+  end
+end

data/spec/models/method_call_log_spec.rb

@@ -9,9 +9,9 @@ describe OkuribitoRails::MethodCallLog do
                       method_symbol: method_symbol,
                       method_name: method_name)
   end
-  let(:class_name) { "User" }
-  let(:method_symbol) { "." }
-  let(:method_name) { "feed" }
+  let(:class_name) { "TestTarget" }
+  let(:method_symbol) { "#" }
+  let(:method_name) { "deprecated_method" }
 
   subject { method_call_log }
 
@@ -24,7 +24,7 @@ describe OkuribitoRails::MethodCallLog do
     it { is_expected.to be_valid }
 
     context "with valid long name" do
-      let(:class_name) { "a" * 255 }
+      let(:class_name) { "A" << "a" * 254 }
       it { is_expected.to be_valid }
     end
   end
@@ -41,7 +41,7 @@ describe OkuribitoRails::MethodCallLog do
     end
 
     context "with too long name" do
-      let(:class_name) { "a" * 256 }
+      let(:class_name) { "A" << "a" * 255 }
       it { is_expected.not_to be_valid }
     end
   end

data/spec/models/method_call_situation_spec.rb

@@ -8,9 +8,9 @@ describe OkuribitoRails::MethodCallSituation do
                       method_name: method_name,
                       called_num: called_num)
   end
-  let(:class_name) { "User" }
-  let(:method_symbol) { "." }
-  let(:method_name) { "feed" }
+  let(:class_name) { "TestTarget" }
+  let(:method_symbol) { "#" }
+  let(:method_name) { "deprecated_method" }
   let(:called_num) { 0 }
 
   subject { method_call_situation }
@@ -19,7 +19,7 @@ describe OkuribitoRails::MethodCallSituation do
     it { is_expected.to be_valid }
 
     context "with valid long name" do
-      let(:class_name) { "a" * 255 }
+      let(:class_name) { "A" << "a" * 254 }
       it { is_expected.to be_valid }
     end
   end
@@ -36,7 +36,7 @@ describe OkuribitoRails::MethodCallSituation do
     end
 
     context "with too long name" do
-      let(:class_name) { "a" * 256 }
+      let(:class_name) { "A" << "a" * 255 }
       it { is_expected.not_to be_valid }
     end
   end

data/spec/spec_helper.rb

@@ -12,3 +12,19 @@ require File.expand_path("../dummy/config/environment.rb", __FILE__)
 require "rspec/rails"
 require "factory_girl"
 require "factories"
+
+require 'database_cleaner'
+
+RSpec.configure do |config|
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :truncation
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/test_config.yml

@@ -0,0 +1,3 @@
+TestTarget:
+  - ".deprecated_self_method"
+  - "#deprecated_method"

data/spec/support/test_target.rb

@@ -0,0 +1,5 @@
+class TestTarget
+  def self.deprecated_self_method; end
+
+  def deprecated_method; end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: okuribito_rails
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.1.0
 platform: ruby
 authors:
 - Yasuhiro Matsumura
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-07 00:00:00.000000000 Z
+date: 2016-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: okuribito
@@ -114,6 +114,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: database_cleaner
+  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: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -156,8 +170,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.0.0
-description: OkuribitoRails is an engine for Rails that aims to be the managing method
-  call status.
+description: OkuribitoRails is an engine for Rails that aims to manage method call
+  status.
 email:
 - ym.contributor@gmail.com
 executables: []
@@ -167,8 +181,10 @@ files:
 - MIT-LICENSE
 - README.md
 - Rakefile
+- app/assets/images/okuribito_rails/logo.png
 - app/assets/javascripts/okuribito_rails/application.js
 - app/assets/stylesheets/okuribito_rails/application.css
+- app/assets/stylesheets/okuribito_rails/header.css
 - app/assets/stylesheets/okuribito_rails/method_call_info.css
 - app/controllers/okuribito_rails/application_controller.rb
 - app/controllers/okuribito_rails/method_call_logs_controller.rb
@@ -176,6 +192,7 @@ files:
 - app/helpers/okuribito_rails/application_helper.rb
 - app/models/okuribito_rails/method_call_log.rb
 - app/models/okuribito_rails/method_call_situation.rb
+- app/views/layouts/okuribito_rails/_header.html.erb
 - app/views/layouts/okuribito_rails/application.html.erb
 - app/views/okuribito_rails/method_call_logs/index.html.erb
 - app/views/okuribito_rails/method_call_situations/index.html.erb
@@ -190,6 +207,7 @@ files:
 - lib/okuribito_rails/observe_method.rb
 - lib/okuribito_rails/railtie.rb
 - lib/okuribito_rails/regist_method.rb
+- lib/okuribito_rails/start_observer.rb
 - lib/okuribito_rails/version.rb
 - lib/tasks/okuribito_rails_tasks.rake
 - spec/dummy/Rakefile
@@ -248,9 +266,14 @@ files:
 - spec/dummy/public/apple-touch-icon.png
 - spec/dummy/public/favicon.ico
 - spec/factories.rb
+- spec/lib/okuribito_rails/observe_method_spec.rb
+- spec/lib/okuribito_rails/regist_method_spec.rb
+- spec/lib/okuribito_rails/start_observer_spec.rb
 - spec/models/method_call_log_spec.rb
 - spec/models/method_call_situation_spec.rb
 - spec/spec_helper.rb
+- spec/support/test_config.yml
+- spec/support/test_target.rb
 homepage: https://github.com/muramurasan/okuribito_rails
 licenses: []
 metadata: {}
@@ -273,8 +296,7 @@ rubyforge_project:
 rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
-summary: OkuribitoRails is an engine for Rails that aims to be the managing method
-  call status.
+summary: OkuribitoRails is an engine for Rails that aims to manage method call status.
 test_files:
 - spec/dummy/app/assets/config/manifest.js
 - spec/dummy/app/assets/javascripts/application.js
@@ -332,6 +354,11 @@ test_files:
 - spec/dummy/public/favicon.ico
 - spec/dummy/Rakefile
 - spec/factories.rb
+- spec/lib/okuribito_rails/observe_method_spec.rb
+- spec/lib/okuribito_rails/regist_method_spec.rb
+- spec/lib/okuribito_rails/start_observer_spec.rb
 - spec/models/method_call_log_spec.rb
 - spec/models/method_call_situation_spec.rb
 - spec/spec_helper.rb
+- spec/support/test_config.yml
+- spec/support/test_target.rb