urbanairship 2.3.1 → 2.3.2

data/lib/urbanairship.rb

@@ -19,7 +19,7 @@ module Urbanairship
     def register_device(device_token, options = {})
       body = parse_register_options(options).to_json
 
-      if (options[:provider] || @provider) == :android
+      if ( (options[:provider] || @provider) == :android ) || ( (options[:provider] || @provider) == 'android' )
         do_request(:put, "/api/apids/#{device_token}", :body => body, :authenticate_with => :application_secret)
       else
         do_request(:put, "/api/device_tokens/#{device_token}", :body => body, :authenticate_with => :application_secret)
@@ -27,7 +27,7 @@ module Urbanairship
     end
 
     def unregister_device(device_token, options = {})
-      if (options[:provider] || @provider) == :android
+      if ( (options[:provider] || @provider) == :android ) || ( (options[:provider] || @provider) == 'android' )
         do_request(:delete, "/api/apids/#{device_token}", :authenticate_with => :application_secret)
       else
         do_request(:delete, "/api/device_tokens/#{device_token}", :authenticate_with => :application_secret)

data/spec/urbanairship_spec.rb

@@ -398,17 +398,30 @@ shared_examples_for "an Urbanairship client" do
       FakeWeb.last_request.path.should == "/api/device_tokens/new_device_token"
     end
 
-    it "uses the android interface if 'provider' configuration option is set to :android" do
+    it "uses the android interface if 'provider' configuration option is set to :android Symbol" do
       subject.provider = :android
       subject.register_device("new_device_token")
       FakeWeb.last_request.path.should == "/api/apids/new_device_token"
       subject.provider = nil
     end
 
-    it "uses the android interface if 'provider' option is passed as :android" do
+    it "uses the android interface if 'provider' configuration option is set to 'android' String" do
+      subject.provider = 'android'
+      subject.register_device("new_device_token")
+      FakeWeb.last_request.path.should == "/api/apids/new_device_token"
+      subject.provider = nil
+    end
+
+    it "uses the android interface if :provider Symbol key is passed an :android Symbol value" do
       subject.register_device("new_device_token", :provider => :android)
       FakeWeb.last_request.path.should == "/api/apids/new_device_token"
     end
+
+    it "uses the android interface if 'provider' Symbol key is passed an 'android' String value" do
+      subject.register_device("new_device_token", :provider => "android")
+      FakeWeb.last_request.path.should == "/api/apids/new_device_token"
+    end
+
   end
 
   describe "::unregister_device" do
@@ -446,18 +459,30 @@ shared_examples_for "an Urbanairship client" do
       subject.unregister_device("key_to_delete").success?.should == false
     end
 
-    it "uses the android interface if 'provider' configuration option is set to :android" do
+    it "uses the android interface if 'provider' configuration option is set to :android Symbol" do
       subject.provider = :android
       subject.unregister_device("new_device_token")
       FakeWeb.last_request.path.should == "/api/apids/new_device_token"
       subject.provider = nil
     end
 
-    it "uses the android interface if 'provider' option is passed as :android" do
+    it "uses the android interface if 'provider' configuration option is set to 'android' String" do
+      subject.provider = 'android'
+      subject.unregister_device("new_device_token")
+      FakeWeb.last_request.path.should == "/api/apids/new_device_token"
+      subject.provider = nil
+    end
+
+    it "uses the android interface if :provider Symbol key is passed with an :android Symbol value" do
       subject.unregister_device("new_device_token", :provider => :android)
       FakeWeb.last_request.path.should == "/api/apids/new_device_token"
     end
 
+    it "uses the android interface if :provider Symbol key is passed with an 'android' String value" do
+      subject.unregister_device("new_device_token", :provider => "android")
+      FakeWeb.last_request.path.should == "/api/apids/new_device_token"
+    end
+
   end
 
   describe "::delete_scheduled_push" do
@@ -553,7 +578,7 @@ shared_examples_for "an Urbanairship client" do
       subject.push.success?.should == false
     end
 
-    it "converts aliases to strints" do
+    it "converts aliases to strings" do
       subject.push(@valid_params.merge(:aliases => [:one, 2]))
       request_json['aliases'].should == ['one', '2']
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: urbanairship
 version: !ruby/object:Gem::Version
-  version: 2.3.1
+  version: 2.3.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-18 00:00:00.000000000 Z
+date: 2013-06-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json