Merge pull request gazay#23 from moktin/master

Fixe specs

Author: gazay

lib/gon.rb

@@ -17,6 +17,10 @@ def all_variables
       @request_env[:gon]
     end
 
+    def all_variables=(values)
+      raise "You can't use Gon public methods for storing data"
+    end
+
     def clear
       @request_env[:gon] = {}
     end
@@ -37,7 +41,7 @@ def request
     end
 
     def request=(request_id)
-      @request_id = request_id      
+      @request_id = request_id
     end
 
     def method_missing(m, *args, &block)
@@ -61,10 +65,10 @@ def set_variable(name, value)
 
     def rabl(view_path, options = {})
       if !defined?(Gon::Rabl)
-        raise NoMethodError.new('You should define Rabl in your Gemfile') 
+        raise NoMethodError.new('You should define Rabl in your Gemfile')
       end
-      rabl_data = Gon::Rabl.parse_rabl(view_path, options[:controller] || 
-                                                  @request_env['action_controller.instance'] || 
+      rabl_data = Gon::Rabl.parse_rabl(view_path, options[:controller] ||
+                                                  @request_env['action_controller.instance'] ||
                                                   @request_env['action_controller.rescue.response'].
                                                     instance_variable_get('@template').
                                                     instance_variable_get('@controller'))
@@ -81,13 +85,13 @@ def rabl(view_path, options = {})
 
     def jbuilder(view_path, options = {})
       if RUBY_VERSION !~ /9/
-        raise NoMethodError.new('You can use Jbuilder support only in 1.9+') 
+        raise NoMethodError.new('You can use Jbuilder support only in 1.9+')
       elsif !defined?(Gon::Jbuilder)
-        raise NoMethodError.new('You should define Jbuilder in your Gemfile') 
+        raise NoMethodError.new('You should define Jbuilder in your Gemfile')
       end
 
-      jbuilder_data = Gon::Jbuilder.parse_jbuilder(view_path, options[:controller] || 
-                                                  @request_env['action_controller.instance'] || 
+      jbuilder_data = Gon::Jbuilder.parse_jbuilder(view_path, options[:controller] ||
+                                                  @request_env['action_controller.instance'] ||
                                                   @request_env['action_controller.rescue.response'].
                                                     instance_variable_get('@template').
                                                     instance_variable_get('@controller'))

spec/gon/gon_spec.rb

@@ -1,7 +1,7 @@
 # gon_spec_rb
 require 'gon'
 
-describe Gon, '#all_variables' do 
+describe Gon, '#all_variables' do
 
   before(:each) do
     Gon.request_env = {}
@@ -41,13 +41,26 @@
     lambda { Gon.all_variables = 123 }.should raise_error
   end
 
-  it 'render json from rabl template' do
-    Gon.clear
-    controller = ActionController::Base.new
-    objects = [1,2]
-    controller.instance_variable_set('@objects', objects)
-    Gon.rabl 'spec/test_data/sample.rabl', :controller => controller
-    Gon.objects.length.should == 2
+  context 'render json from rabl template' do
+    before :each do
+      Gon.clear
+      controller.instance_variable_set('@objects', objects)
+    end
+
+    let(:controller) { ActionController::Base.new}
+    let(:objects) { [1,2]}
+
+    it 'raises an error if rabl is not included' do
+      expect { Gon.rabl 'spec/test_data/sample.rabl', :controller => controller}.to raise_error
+    end
+
+    it 'works if rabl is included' do
+      require 'rabl'
+      require 'gon/rabl'
+      Gon.rabl 'spec/test_data/sample.rabl', :controller => controller
+      Gon.objects.length.should == 2
+    end
+
   end
 
   if RUBY_VERSION =~ /1.9/