api_taster 0.4.1 → 0.4.2

data/README.md

@@ -81,7 +81,7 @@ Instead of manually finding out which route definitions you need, API Taster pro
 
 APIs evolve - especially during the development stage. To keep `ApiTaster.routes` in sync with your route definitions, API Taster provides a warning page that shows you the definitions that are obsolete/mismatched therefore you could correct or remove them.
 
-![](http://i.imgur.com/Fo7kQ.png)
+![](http://i.imgur.com/qK7g5.png)
 
 ### Use with an Engine
 

data/lib/api_taster.rb

@@ -8,11 +8,10 @@ module ApiTaster
   def self.routes(&block)
     Route.route_set            = Rails.application.routes
     Route.inputs               = {}
-    Route.missing_definitions  = []
     Route.obsolete_definitions = []
 
     Mapper.instance_eval(&block)
-
-    Route.calculate_missing_definitions
   end
+
+  class Exception < ::Exception; end
 end

data/lib/api_taster/route.rb

@@ -10,6 +10,10 @@ module ApiTaster
         _routes = []
         i = -1
 
+        unless route_set.respond_to?(:routes)
+          raise ApiTaster::Exception.new('Route definitions are missing, have you defined ApiTaster.routes?')
+        end
+
         route_set.routes.each do |route|
           next if route.app.is_a?(Sprockets::Environment)
           next if route.app == ApiTaster::Engine
@@ -51,12 +55,8 @@ module ApiTaster
         inputs[route[:id]].collect { |input| split_input(input, route) }
       end
 
-      def calculate_missing_definitions
-        routes.each do |route|
-          if undefined_route?(route)
-            self.missing_definitions << route
-          end
-        end
+      def missing_definitions
+        routes.select { |route| undefined_route?(route) }
       end
 
       private

data/lib/api_taster/version.rb

@@ -1,3 +1,3 @@
 module ApiTaster
-  VERSION = "0.4.1"
+  VERSION = "0.4.2"
 end

data/spec/route_spec.rb

@@ -33,6 +33,13 @@ module ApiTaster
       Route.routes.first.should == app_home_route
     end
 
+    context "undefined ApiTaster.routes" do
+      it "errors out" do
+        Route.route_set = nil
+        expect { Route.routes }.to raise_exception(ApiTaster::Exception)
+      end
+    end
+
     it "#grouped_routes" do
       Route.grouped_routes.has_key?('application').should == true
       Route.grouped_routes.has_key?('users').should == true

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: api_taster
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
   prerelease: 
 platform: ruby
 authors:
@@ -354,7 +354,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -548788455716746857
+      hash: -2192023684559152796
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -363,7 +363,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -548788455716746857
+      hash: -2192023684559152796
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24