rhoconnect 4.0.4 → 5.1.1

data/spec/jobs/ping_job_spec.rb

@@ -1,190 +1,189 @@
 require File.join(File.dirname(__FILE__),'..','spec_helper')
 
 describe "PingJob" do
-  it_behaves_like "SharedRhoconnectHelper", :rhoconnect_data => true do
-    
-    before(:each) do
-      @u1_fields = {:login => 'testuser1'}
-      @u1 = User.create(@u1_fields) 
-      @u1.password = 'testpass1'
-      @c1_fields = {
-        :device_type => 'Apple',
-        :device_pin => 'abcde',
-        :device_port => '3333',
-        :user_id => @u1.id,
-        :app_id => @a.id 
-      }
-      @c1 = Client.create(@c1_fields,{:source_name => @s_fields[:name]})
-      @a.users << @u1.id
-    end
-    
-    it "should perform apple ping with integer parameters" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => 5, "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
+  include_examples "SharedRhoconnectHelper", :rhoconnect_data => true
 
-      scrubbed_params = params.dup
-      scrubbed_params['vibrate'] = '5'
+  before(:each) do
+    @u1_fields = {:login => 'testuser1'}
+    @u1 = User.create(@u1_fields)
+    @u1.password = 'testpass1'
+    @c1_fields = {
+      :device_type => 'Apple',
+      :device_pin => 'abcde',
+      :device_port => '3333',
+      :user_id => @u1.id,
+      :app_id => @a.id
+    }
+    @c1 = Client.create(@c1_fields,{:source_name => @s_fields[:name]})
+    @a.users << @u1.id
+  end
 
-      Apple.should_receive(:ping).once.with({'device_pin' => @c.device_pin,
-        'device_port' => @c.device_port}.merge!(scrubbed_params))
-      PingJob.perform(params)
-    end
+  it "should perform apple ping with integer parameters" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => 5, "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
 
-    it "should perform apple ping" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
-      Apple.should_receive(:ping).once.with({'device_pin' => @c.device_pin,
-        'device_port' => @c.device_port}.merge!(params))
-      PingJob.perform(params)
-    end
+    scrubbed_params = params.dup
+    scrubbed_params['vibrate'] = '5'
 
-    it "should skip ping for the unknown platform" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
-      @c.device_type = 'unknown_device_type'
-      PingJob.should_receive(:log).once.with("Dropping ping request for unsupported platform '#{@c.device_type}'")
-      PingJob.perform(params)
-    end
+    Apple.should_receive(:ping).once.with({'device_pin' => @c.device_pin,
+      'device_port' => @c.device_port}.merge!(scrubbed_params))
+    PingJob.perform(params)
+  end
 
-    it "should skip ping for empty device_type" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
-      @c.device_type = nil
-      PingJob.should_receive(:log).once.with("Skipping ping for non-registered client_id '#{@c.id}'...")
-      lambda { PingJob.perform(params) }.should_not raise_error
-    end
+  it "should perform apple ping" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
+    Apple.should_receive(:ping).once.with({'device_pin' => @c.device_pin,
+      'device_port' => @c.device_port}.merge!(params))
+    PingJob.perform(params)
+  end
 
-    it "should skip ping for empty device_pin" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
-      @c.device_type = 'Android'
-      @c.device_pin = nil
-      PingJob.should_receive(:log).once.with("Skipping ping for non-registered client_id '#{@c.id}'...")
-      lambda { PingJob.perform(params) }.should_not raise_error
-    end
-    
-    it "should drop ping if it's already in user's device pin list" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
-      # another client with the same device pin ...
-      @c_fields.delete(:id)
-      @c1 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
-      # and yet another one ...
-      @c_fields.delete(:id)
-      @c2 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
-
-      Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'device_port' => @c.device_port}.merge!(params))
-      PingJob.should_receive(:log).twice.with(/Dropping ping request for client/)
-      lambda { PingJob.perform(params) }.should_not raise_error
-    end
-    
-    it "should drop ping if it's already in user's phone id list and device pin is different" do
-      params = {"user_id" => @u.id, "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
-      @c.phone_id = '3'
-      @c_fields.merge!(:phone_id => '3')
-      # another client with the same phone id..
-      @c_fields.delete(:id)
-      @c1 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
-      #  yet another...
-      @c_fields.delete(:id)
-      @c2 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
-
-      Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port}.merge!(params))
-      PingJob.should_receive(:log).twice.with(/Dropping ping request for client/)
-      lambda { PingJob.perform(params) }.should_not raise_error
-    end
-    
-    it "should ping two different users from two different devices - Apple and GCM" do
-      params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3', 'phone_id' => nil }
-      @c.phone_id = '3'
-      
-      scrubbed_params = params.dup
-      scrubbed_params['vibrate'] = '5'
-      @c1.device_push_type = 'Gcm'
-      
-      Apple.should_receive(:ping).with(params.merge!({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port}))
-      Gcm.should_receive(:ping).with({'device_pin' => @c1.device_pin, 'device_port' => @c1.device_port}.merge!(scrubbed_params))
-      PingJob.perform(params)
-    end
-    
-    it "should drop ping with two different users from the same device" do
-      params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
-      @c.phone_id = '3'
-      @c1.phone_id = '3'
-      
-      Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port}.merge!(params))
-      PingJob.should_receive(:log).once.with(/Dropping ping request for client/)
-      lambda { PingJob.perform(params) }.should_not raise_error
-    end
-    
-    it "should drop ping with two different users with the same pin" do
-      params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', "phone_id" => nil,
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
-      @c1.device_pin = @c.device_pin
-
-      Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'device_port' => @c.device_port}.merge!(params))
-      PingJob.should_receive(:log).once.with(/Dropping ping request for client/)
-      lambda { PingJob.perform(params) }.should_not raise_error
-    end
-    
-    it "should process all pings even if some of them are failing" do
-      params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3', 'phone_id' => nil }
-      @c.phone_id = '3'
-      
-      scrubbed_params = params.dup
-      scrubbed_params['vibrate'] = '5'
-      @c1.device_push_type = 'Gcm'
-      
-      Apple.should_receive(:ping).with(params.merge!({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port})).and_return { raise SocketError.new("Socket failure") }
-      Gcm.should_receive(:ping).with({'device_pin' => @c1.device_pin, 'device_port' => @c1.device_port}.merge!(scrubbed_params))
-      exception_raised = false
-      begin
-        PingJob.perform(params)
-      rescue Exception => e
-        exception_raised = true
-      end
-      exception_raised.should == true
-    end
-    
-    it "should skip ping for unknown user or user with no clients" do
-      params = {"user_id" => [ 'fake_user' ], "api_token" => @api_token,
-        "sources" => [@s.name], "message" => 'hello world', 
-        "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3', 'phone_id' => nil }
-      PingJob.should_receive(:log).once.with(/Skipping ping for unknown user 'fake_user' or 'fake_user' has no registered clients.../)
-      PingJob.perform(params)
-    end
+  it "should skip ping for the unknown platform" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
+    @c.device_type = 'unknown_device_type'
+    PingJob.should_receive(:log).once.with("Dropping ping request for unsupported platform '#{@c.device_type}'")
+    PingJob.perform(params)
+  end
 
-    it "should process ping for device_push_type if available" do
-      @c.device_push_type = 'rhoconnect_push'
-      @c.device_port = nil
-      params = {
-        'user_id'  => @u.id, 
-        'message'  => 'hello world',
-        'device_port' => '',
-        'vibrate'  => '',
-        'phone_id' => nil
-      }
-      scrubbed_params = params.dup
-
-      RhoconnectPush.should_receive(:ping).once.with(
-        {'device_pin' => @c.device_pin}.merge!(scrubbed_params)
-      )
+  it "should skip ping for empty device_type" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
+    @c.device_type = nil
+    PingJob.should_receive(:log).once.with("Skipping ping for non-registered client_id '#{@c.id}'...")
+    lambda { PingJob.perform(params) }.should_not raise_error
+  end
+
+  it "should skip ping for empty device_pin" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
+    @c.device_type = 'Android'
+    @c.device_pin = nil
+    PingJob.should_receive(:log).once.with("Skipping ping for non-registered client_id '#{@c.id}'...")
+    lambda { PingJob.perform(params) }.should_not raise_error
+  end
+
+  it "should drop ping if it's already in user's device pin list" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3',"phone_id"=>nil}
+    # another client with the same device pin ...
+    @c_fields.delete(:id)
+    @c1 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
+    # and yet another one ...
+    @c_fields.delete(:id)
+    @c2 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
+
+    Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'device_port' => @c.device_port}.merge!(params))
+    PingJob.should_receive(:log).twice.with(/Dropping ping request for client/)
+    lambda { PingJob.perform(params) }.should_not raise_error
+  end
+
+  it "should drop ping if it's already in user's phone id list and device pin is different" do
+    params = {"user_id" => @u.id, "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
+    @c.phone_id = '3'
+    @c_fields.merge!(:phone_id => '3')
+    # another client with the same phone id..
+    @c_fields.delete(:id)
+    @c1 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
+    #  yet another...
+    @c_fields.delete(:id)
+    @c2 = Client.create(@c_fields,{:source_name => @s_fields[:name]})
+
+    Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port}.merge!(params))
+    PingJob.should_receive(:log).twice.with(/Dropping ping request for client/)
+    lambda { PingJob.perform(params) }.should_not raise_error
+  end
+
+  it "should ping two different users from two different devices - Apple and GCM" do
+    params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3', 'phone_id' => nil }
+    @c.phone_id = '3'
+
+    scrubbed_params = params.dup
+    scrubbed_params['vibrate'] = '5'
+    @c1.device_push_type = 'Gcm'
+
+    Apple.should_receive(:ping).with(params.merge!({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port}))
+    Gcm.should_receive(:ping).with({'device_pin' => @c1.device_pin, 'device_port' => @c1.device_port}.merge!(scrubbed_params))
+    PingJob.perform(params)
+  end
+
+  it "should drop ping with two different users from the same device" do
+    params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
+    @c.phone_id = '3'
+    @c1.phone_id = '3'
+
+    Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port}.merge!(params))
+    PingJob.should_receive(:log).once.with(/Dropping ping request for client/)
+    lambda { PingJob.perform(params) }.should_not raise_error
+  end
+
+  it "should drop ping with two different users with the same pin" do
+    params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world', "phone_id" => nil,
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3'}
+    @c1.device_pin = @c.device_pin
+
+    Apple.should_receive(:ping).with({'device_pin' => @c.device_pin, 'device_port' => @c.device_port}.merge!(params))
+    PingJob.should_receive(:log).once.with(/Dropping ping request for client/)
+    lambda { PingJob.perform(params) }.should_not raise_error
+  end
+
+  it "should process all pings even if some of them are failing" do
+    params = {"user_id" => [ @u.id, @u1.id], "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3', 'phone_id' => nil }
+    @c.phone_id = '3'
+
+    scrubbed_params = params.dup
+    scrubbed_params['vibrate'] = '5'
+    @c1.device_push_type = 'Gcm'
+
+    Apple.should_receive(:ping).with(params.merge!({'device_pin' => @c.device_pin, 'phone_id' => @c.phone_id, 'device_port' => @c.device_port})).and_return { raise SocketError.new("Socket failure") }
+    Gcm.should_receive(:ping).with({'device_pin' => @c1.device_pin, 'device_port' => @c1.device_port}.merge!(scrubbed_params))
+    exception_raised = false
+    begin
       PingJob.perform(params)
+    rescue Exception => e
+      exception_raised = true
     end
-  end  
+    exception_raised.should == true
+  end
+
+  it "should skip ping for unknown user or user with no clients" do
+    params = {"user_id" => [ 'fake_user' ], "api_token" => @api_token,
+      "sources" => [@s.name], "message" => 'hello world',
+      "vibrate" => '5', "badge" => '5', "sound" => 'hello.mp3', 'phone_id' => nil }
+    PingJob.should_receive(:log).once.with(/Skipping ping for unknown user 'fake_user' or 'fake_user' has no registered clients.../)
+    PingJob.perform(params)
+  end
+
+  it "should process ping for device_push_type if available" do
+    @c.device_push_type = 'rhoconnect_push'
+    @c.device_port = nil
+    params = {
+      'user_id'  => @u.id,
+      'message'  => 'hello world',
+      'device_port' => '',
+      'vibrate'  => '',
+      'phone_id' => nil
+    }
+    scrubbed_params = params.dup
+
+    RhoconnectPush.should_receive(:ping).once.with(
+      {'device_pin' => @c.device_pin}.merge!(scrubbed_params)
+    )
+    PingJob.perform(params)
+  end
 end

data/spec/jobs/source_job_spec.rb

@@ -1,30 +1,29 @@
 require File.join(File.dirname(__FILE__),'..','spec_helper')
 
 describe "SourceJob" do
-  it_behaves_like "SharedRhoconnectHelper", :rhoconnect_data => true do
-    before(:each) do
-      @s = Source.load(@s_fields[:name],@s_params)
-      @s.query_queue = 'query'
-      @s.cud_queue = 'cud'
-    end
-    
-    it "should perform process_query" do
-      set_state('test_db_storage' => @data)  
-      SourceJob.perform('query',@s.id,@s.app_id,@s.user_id,nil)
-      verify_doc_result(@s, {:md => @data,
-                             :md_size => @data.size.to_s})
-    end
+  include_examples "SharedRhoconnectHelper", :rhoconnect_data => true
+  before(:each) do
+    @s = Source.load(@s_fields[:name],@s_params)
+    @s.query_queue = 'query'
+    @s.cud_queue = 'cud'
+  end
 
-    it "should perform process_cud" do
-      expected = {'backend_id'=>@product1}
-      @product1['link'] = 'abc'
-      @s.push_queue(:create, @c.id, [[@s.name, [['1', @product1]]]])
-      SourceJob.perform('cud',@s.id,@s.app_id,@s.user_id,nil)
-      verify_source_queue_data(@s, :create => [])
-      verify_doc_result(@s, {:md => expected,
-                             :md_size => expected.size.to_s})
-      verify_doc_result(@c, {:cd => expected,
-                             :cd_size => expected.size.to_s})
-    end
-  end  
+  it "should perform process_query" do
+    set_state('test_db_storage' => @data)
+    SourceJob.perform('query',@s.id,@s.app_id,@s.user_id,nil)
+    verify_doc_result(@s, {:md => @data,
+                           :md_size => @data.size.to_s})
+  end
+
+  it "should perform process_cud" do
+    expected = {'backend_id'=>@product1}
+    @product1['link'] = 'abc'
+    @s.push_queue(:create, @c.id, [[@s.name, [['1', @product1]]]])
+    SourceJob.perform('cud',@s.id,@s.app_id,@s.user_id,nil)
+    verify_source_queue_data(@s, :create => [])
+    verify_doc_result(@s, {:md => expected,
+                           :md_size => expected.size.to_s})
+    verify_doc_result(@c, {:cd => expected,
+                           :cd_size => expected.size.to_s})
+  end
 end

data/spec/license_spec.rb

@@ -1,68 +1,67 @@
 require File.join(File.dirname(__FILE__),'spec_helper')
 
 describe "License" do
-  it_behaves_like "SharedRhoconnectHelper", :rhoconnect_data => false do
-    before(:each) do
-      Store.put_value(License::CLIENT_DOCKEY,nil)
-    end
+  include_examples "SharedRhoconnectHelper", :rhoconnect_data => false
 
-    it "should decrypt license" do
-      license = License.new
-      license.rhoconnect_version.should == 'Version 1'
-      license.licensee.should == 'Rhomobile'
-      license.seats.should == 10
-      license.issued.should == 'Fri Apr 23 17:20:13 -0700 2010'
-    end
+  before(:each) do
+    Store.put_value(License::CLIENT_DOCKEY,nil)
+  end
 
-    it "should use generator license file when there is license load error" do
-      path_to_license = File.join(File.dirname(__FILE__), '..', 'generators', 'templates', 'application', 'settings', 'license.key')
-      license = IO.read(path_to_license).strip
-      Rhoconnect.stub!(:get_config).and_return({Rhoconnect.environment.to_sym => {}})
-      License.new.license.should == license
-      # lambda { License.new }.should raise_error(LicenseException, "Error verifying license.")
-    end
+  it "should decrypt license" do
+    license = License.new
+    license.rhoconnect_version.should == 'Version 1'
+    license.licensee.should == 'Rhomobile'
+    license.seats.should == 10
+    license.issued.should == 'Fri Apr 23 17:20:13 -0700 2010'
+  end
 
-    it "should verify # of seats before adding" do
-      License.new.check_and_use_seat
-      Store.get_value(License::CLIENT_DOCKEY).to_i.should == 1
-    end
+  it "should use generator license file when there is license load error" do
+    path_to_license = File.join(File.dirname(__FILE__), '..', 'generators', 'templates', 'application', 'settings', 'license.key')
+    license = IO.read(path_to_license).strip
+    Rhoconnect.stub(:get_config).and_return({Rhoconnect.environment.to_sym => {}})
+    License.new.license.should == license
+    # lambda { License.new }.should raise_error(LicenseException, "Error verifying license.")
+  end
 
-    it "should raise exception when seats are exceeded" do
-      Store.put_value(License::CLIENT_DOCKEY,10)
-      lambda { License.new.check_and_use_seat }.should raise_error(
-        LicenseSeatsExceededException, "WARNING: Maximum # of devices exceeded for this license."
-      )
-    end
+  it "should verify # of seats before adding" do
+    License.new.check_and_use_seat
+    Store.get_value(License::CLIENT_DOCKEY).to_i.should == 1
+  end
 
-    it "should free license seat" do
-      Store.put_value(License::CLIENT_DOCKEY,5)
-      License.new.free_seat
-      Store.get_value(License::CLIENT_DOCKEY).to_i.should == 4
-    end
+  it "should raise exception when seats are exceeded" do
+    Store.put_value(License::CLIENT_DOCKEY,10)
+    lambda { License.new.check_and_use_seat }.should raise_error(
+      LicenseSeatsExceededException, "WARNING: Maximum # of devices exceeded for this license."
+    )
+  end
 
-    it "should get # of available seats" do
-      license = License.new
-      license.check_and_use_seat
-      license.available.should == 9
-    end
+  it "should free license seat" do
+    Store.put_value(License::CLIENT_DOCKEY,5)
+    License.new.free_seat
+    Store.get_value(License::CLIENT_DOCKEY).to_i.should == 4
+  end
+
+  it "should get # of available seats" do
+    license = License.new
+    license.check_and_use_seat
+    license.available.should == 9
+  end
 
-    it "should use RHOCONNECT_LICENSE env var" do
-      ENV['RHOCONNECT_LICENSE'] = 'b749cbe6e029400e688360468624388e2cb7f6a1e72c91d4686a1b8c9d37b72c3e1872ec9f369d481220e10759c18e16'
-      license = License.new
-      license.licensee.should == 'Rhohub'
-      license.seats.should == 5
-      license.issued.should == 'Tue Aug 10 16:14:24 -0700 2010'
-      ENV.delete('RHOCONNECT_LICENSE')
-    end
-    
-    it "should use RHOSYNC_LICENSE env var" do
-      ENV['RHOSYNC_LICENSE'] = 'b749cbe6e029400e688360468624388e2cb7f6a1e72c91d4686a1b8c9d37b72c3e1872ec9f369d481220e10759c18e16'
-      license = License.new
-      license.licensee.should == 'Rhohub'
-      license.seats.should == 5
-      license.issued.should == 'Tue Aug 10 16:14:24 -0700 2010'
-      ENV.delete('RHOSYNC_LICENSE')
-    end
+  it "should use RHOCONNECT_LICENSE env var" do
+    ENV['RHOCONNECT_LICENSE'] = 'b749cbe6e029400e688360468624388e2cb7f6a1e72c91d4686a1b8c9d37b72c3e1872ec9f369d481220e10759c18e16'
+    license = License.new
+    license.licensee.should == 'Rhohub'
+    license.seats.should == 5
+    license.issued.should == 'Tue Aug 10 16:14:24 -0700 2010'
+    ENV.delete('RHOCONNECT_LICENSE')
   end
 
+  it "should use RHOSYNC_LICENSE env var" do
+    ENV['RHOSYNC_LICENSE'] = 'b749cbe6e029400e688360468624388e2cb7f6a1e72c91d4686a1b8c9d37b72c3e1872ec9f369d481220e10759c18e16'
+    license = License.new
+    license.licensee.should == 'Rhohub'
+    license.seats.should == 5
+    license.issued.should == 'Tue Aug 10 16:14:24 -0700 2010'
+    ENV.delete('RHOSYNC_LICENSE')
+  end
 end