synvert-core 0.5.2 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b3826c2b5eaae969299d3b3d05329c69a6b3dc19
-  data.tar.gz: c3bb0dc79380ad6c76c449ffef5f9addbe997b8f
+  metadata.gz: fa1fe37f86bbf786414a8e6f5190599bf5ee1e54
+  data.tar.gz: 818ff996234d1fb5400afdd9740c777ee7a3a691
 SHA512:
-  metadata.gz: f7f4fc704f75f3bd1c91d201ae6b6f6b82ac9e47899fa17ffd709f56861712166a50de011e6935630762c7002b2d9b9faa560c29aa9032ddc7bceed6e887c5d4
-  data.tar.gz: b259c712216db4b01ee3702f04a997e8f444b083bcd9580a087d5f608582dffc3d9063a3e916fddc9010649bb33d4da817ef476442c87d0373c5fc1748eda02d
+  metadata.gz: 4f12b89003300b961506a631fadb4981877dd49e012866ca742dc5af0f86a71aa2db00c2835939e21da96c95af55b4f71d2dcda55eb5ebb5b038cb32c158bfdd
+  data.tar.gz: 4efb84e75b9fb0c313596004b757d78cf40356c8aaae3b8a1d470eabd7f9ef4c92f9e856352ec7e1535db3fad8f0322958a245c9580506a710e129bea5a0eaed

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 0.5.3
+
+* Raise RewriterNotFound if rewriter not found
+
 ## 0.5.2
 
 * Add body for :defs node

data/lib/synvert/core/rewriter.rb

@@ -46,6 +46,7 @@ module Synvert::Core
       # @param name [String] the unique rewriter name.
       # @param rewriter [Synvert::Core::Rewriter] the rewriter to register.
       def register(group, name, rewriter)
+        group, name = group.to_s, name.to_s
         @rewriters ||= {}
         @rewriters[group] ||= {}
         @rewriters[group][name] = rewriter
@@ -56,8 +57,14 @@ module Synvert::Core
       # @param group [String] rewrtier group.
       # @param name [String] rewrtier name.
       # @return [Synvert::Core::Rewriter] the matching rewriter.
+      # @raise [Synvert::Core::RewriterNotFound] if the registered rewriter is not found.
       def fetch(group, name)
-        @rewriters[group][name]
+        group, name = group.to_s, name.to_s
+        if exist? group, name
+          @rewriters[group][name]
+        else
+          raise RewriterNotFound.new "Rewriter #{group} #{name} not found"
+        end
       end
 
       # Get a registered rewriter by group and name, then process that rewriter.
@@ -67,11 +74,27 @@ module Synvert::Core
       # @return [Synvert::Core::Rewriter] the registered rewriter.
       # @raise [Synvert::Core::RewriterNotFound] if the registered rewriter is not found.
       def call(group, name)
-        if (rewriter = @rewriters[group][name])
+        group, name = group.to_s, name.to_s
+        if exist? group, name
+          rewriter = @rewriters[group][name]
           rewriter.process
           rewriter
         else
-          raise RewriterNotFound.new "Rewriter #{group} #{name} not found"
+          raise RewriterNotFound.new "Rewriter #{group}/#{name} not found"
+        end
+      end
+
+      # Check if one rewriter exist.
+      #
+      # @param group [String] the rewriter group.
+      # @param name [String] the rewriter name.
+      # @return [Boolean] true if the rewriter exist.
+      def exist?(group, name)
+        group, name = group.to_s, name.to_s
+        if @rewriters && @rewriters[group] && @rewriters[group][name]
+          true
+        else
+          false
         end
       end
 
@@ -84,7 +107,7 @@ module Synvert::Core
 
       # Clear all registered rewriters.
       def clear
-        @rewriters.clear
+        @rewriters.clear if @rewriters
       end
     end
 

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = "0.5.2"
+    VERSION = "0.5.3"
   end
 end

data/spec/synvert/core/rewriter_spec.rb

@@ -169,6 +169,21 @@ module Synvert::Core
         Rewriter.call 'group', 'rewriter'
       end
 
+      it 'raises RewriterNotFound if rewriter not found' do
+        expect { Rewriter.call 'group', 'rewriter' }.to raise_error(RewriterNotFound)
+      end
+
+      context "exist?" do
+        it 'returns true if rewriter exists' do
+          Rewriter.new 'group', 'rewriter'
+          expect(Rewriter.exist? 'group', 'rewriter').to be_truthy
+        end
+
+        it 'returns false if rewriter does not exist' do
+          expect(Rewriter.exist? 'group', 'rewriter').to be_falsey
+        end
+      end
+
       context "available" do
         it 'lists empty rewriters' do
           expect(Rewriter.availables).to eq({})

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.3
 platform: ruby
 authors:
 - Richard Huang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-31 00:00:00.000000000 Z
+date: 2014-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser