stormpath-sdk 1.6.0 → 1.7.0

data/lib/stormpath-sdk/resource/saml_identity_provider.rb

@@ -0,0 +1,14 @@
+module Stormpath
+  module Resource
+    class SamlIdentityProvider < Stormpath::Resource::Instance
+      prop_reader :sso_login_endpoint, :signature_algorithm, :sha_fingerprint, :created_at, :modified_at
+      prop_accessor :status
+
+      has_one :metadata, class_name: :samlIdentityProviderMetadata
+      has_one :attribute_statement_mapping_rules
+      has_one :x509_signing_cert, class_name: :x509Certificate
+      has_many :registered_saml_service_providers
+      has_many :saml_service_provider_registrations, can: [:get, :create]
+    end
+  end
+end

data/lib/stormpath-sdk/resource/saml_identity_provider_metadata.rb

@@ -0,0 +1,9 @@
+module Stormpath
+  module Resource
+    class SamlIdentityProviderMetadata < Stormpath::Resource::Instance
+      prop_reader :entity_id
+      belongs_to :identity_provider, class_name: :samlIdentityProvider
+      has_one :x509_signing_cert, class_name: :x509Certificate
+    end
+  end
+end

data/lib/stormpath-sdk/resource/saml_policy.rb

@@ -0,0 +1,10 @@
+module Stormpath
+  module Resource
+    class SamlPolicy < Stormpath::Resource::Instance
+      prop_reader :created_at, :modified_at
+
+      has_one :service_provider, class_name: :samlServiceProvider
+      has_one :identity_provider, class_name: :samlIdentityProvider
+    end
+  end
+end

data/lib/stormpath-sdk/resource/saml_service_provider.rb

@@ -0,0 +1,7 @@
+module Stormpath
+  module Resource
+    class SamlServiceProvider < Stormpath::Resource::Instance
+      prop_reader :created_at, :modified_at, :sso_initiation_endpoint
+    end
+  end
+end

data/lib/stormpath-sdk/{provider/saml/saml_mapping_rules.rb → resource/saml_service_provider_metadata.rb}

@@ -13,10 +13,11 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-class Stormpath::Provider::SamlMappingRules < Stormpath::Provider::Provider
-  prop_reader :href, :created_at, :modified_at, :items
-
-  def set_options(options)
-    set_property :href, options[:href]
+module Stormpath
+  module Resource
+    class SamlServiceProviderMetadata < Stormpath::Resource::Instance
+      prop_reader :href, :created_at, :modified_at, :entity_id, :x509_signing_cert,
+                  :assertion_consumer_service_post_endpoint
+    end
   end
 end

data/lib/stormpath-sdk/resource/saml_service_provider_registration.rb

@@ -0,0 +1,11 @@
+module Stormpath
+  module Resource
+    class SamlServiceProviderRegistration < Stormpath::Resource::Instance
+      prop_reader :created_at, :modified_at
+      prop_accessor :status, :default_relay_state
+
+      has_one :service_provider, class_name: :registeredSamlServiceProvider
+      has_one :identity_provider, class_name: :samlIdentityProvider
+    end
+  end
+end

data/lib/stormpath-sdk/resource/schema.rb

@@ -13,9 +13,13 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-class Stormpath::Resource::Schema < Stormpath::Resource::Instance
-  prop_reader :created_at, :modified_at
+module Stormpath
+  module Resource
+    class Schema < Stormpath::Resource::Instance
+      prop_reader :created_at, :modified_at
 
-  has_many :fields
-  belongs_to :directory
+      has_many :fields
+      belongs_to :directory
+    end
+  end
 end

data/lib/stormpath-sdk/resource/tenant.rb

@@ -13,14 +13,17 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-class Stormpath::Resource::Tenant < Stormpath::Resource::Instance
-  include Stormpath::Resource::CustomDataStorage
+module Stormpath
+  module Resource
+    class Tenant < Stormpath::Resource::Instance
+      include Stormpath::Resource::CustomDataStorage
 
-  prop_reader :name, :key, :created_at, :modified_at
-
-  has_many :applications
-  has_many :directories
-  has_many :organizations
-  has_one :custom_data
+      prop_reader :name, :key, :created_at, :modified_at
 
+      has_many :applications
+      has_many :directories
+      has_many :organizations
+      has_one :custom_data
+    end
+  end
 end

data/lib/stormpath-sdk/resource/user_info_mapping_rules.rb

@@ -13,7 +13,11 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-class Stormpath::Resource::UserInfoMappingRules < Stormpath::Resource::Instance
-  prop_accessor :items
-  prop_reader :href, :created_at, :modified_at
+module Stormpath
+  module Resource
+    class UserInfoMappingRules < Stormpath::Resource::Instance
+      prop_accessor :items
+      prop_reader :href, :created_at, :modified_at
+    end
+  end
 end

data/lib/stormpath-sdk/resource/utils.rb

@@ -21,16 +21,15 @@ module Stormpath
 
       def inspect
         ''.tap do |str|
-          str << %Q[#<#{class_name_with_id} @properties={]
+          str << %(#<#{class_name_with_id} @properties={)
           @read_lock.lock
           begin
             str << properties.map do |key, value|
-              if printable_property? key
-                if value.kind_of? Hash and value.has_key? Stormpath::Resource::Base::HREF_PROP_NAME
-                  value = %Q[{ "#{Stormpath::Resource::Base::HREF_PROP_NAME}" => "#{value[Stormpath::Resource::Base::HREF_PROP_NAME]}" }]
-                end
-                %Q["#{key} => #{value}"]
+              next unless printable_property? key
+              if value.is_a?(Hash) && value.key?(Stormpath::Resource::Base::HREF_PROP_NAME)
+                value = %({ "#{Stormpath::Resource::Base::HREF_PROP_NAME}" => "#{value[Stormpath::Resource::Base::HREF_PROP_NAME]}" })
               end
+              %("#{key} => #{value}")
             end.compact.join(',')
           ensure
             @read_lock.unlock
@@ -49,9 +48,7 @@ module Stormpath
 
           begin
             properties_yaml = properties.each do |key, value|
-              if printable_property? key
-                " #{key}: #{value} \n"
-              end
+              " #{key}: #{value} \n" if printable_property? key
             end.compact.join("\n")
             unless properties_yaml.empty?
               yaml << " properties\n "
@@ -64,7 +61,7 @@ module Stormpath
       end
 
       def class_name_with_id
-        object_id_hex = '%x' % (self.object_id << 1)
+        object_id_hex = '%x' % (object_id << 1)
         "#{self.class.name}:0x#{object_id_hex}"
       end
     end

data/lib/stormpath-sdk/resource/verification_email.rb

@@ -1,5 +1,9 @@
-class Stormpath::Resource::VerificationEmail < Stormpath::Resource::Instance
-  prop_accessor :login, :account_store
+module Stormpath
+  module Resource
+    class VerificationEmail < Stormpath::Resource::Instance
+      prop_accessor :login, :account_store
 
-  belongs_to :application
+      belongs_to :application
+    end
+  end
 end

data/lib/stormpath-sdk/resource/x_509_certificate.rb

@@ -0,0 +1,7 @@
+module Stormpath
+  module Resource
+    class X509Certificate < Stormpath::Resource::Instance
+      prop_accessor :encoded_x_509_certificate, :encoded_private_key
+    end
+  end
+end

data/lib/stormpath-sdk/util/assert.rb

@@ -16,13 +16,12 @@
 module Stormpath
   module Util
     module Assert
-
       def assert_not_nil(object, message)
         raise(ArgumentError, message, caller) if object.nil?
       end
 
       def assert_kind_of(clazz, object, message)
-        raise(ArgumentError, message, caller) unless object.kind_of? clazz
+        raise(ArgumentError, message, caller) unless object.is_a?(clazz)
       end
 
       def assert_true(arg, message)
@@ -32,7 +31,6 @@ module Stormpath
       def assert_false(arg, message)
         raise(ArgumentError, message, caller) if arg
       end
-
     end
   end
 end

data/lib/stormpath-sdk/version.rb

@@ -14,6 +14,6 @@
 # limitations under the License.
 #
 module Stormpath
-  VERSION = '1.6.0'
-  VERSION_DATE = '2017-01-26'
+  VERSION = '1.7.0'.freeze
+  VERSION_DATE = '2017-02-09'.freeze
 end

data/spec/auth/basic_authenticator_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'BasicAuthenticator', vcr: true do
+describe 'BasicAuthenticator', :vcr do
   let(:application) { test_api_client.applications.create(application_attrs) }
   let(:directory) { test_api_client.directories.create(directory_attrs) }
   let(:directory2) { test_api_client.directories.create(directory_attrs) }
@@ -10,16 +10,14 @@ describe 'BasicAuthenticator', vcr: true do
   end
   let(:password) { 'F00barfoo' }
   let(:invalid_password) { 'Wr00ngPassw0rd' }
-  let(:dir_account) do
-    directory.accounts.create(account_attrs(username: 'ruby_cilim_dir', password: password))
-  end
-  let(:org_account) do
-    organization.accounts.create(account_attrs(username: 'ruby_cilim_org', password: password))
-  end
+  let(:dir_account) { directory.accounts.create(account_attrs(password: password)) }
+  let(:org_account) { organization.accounts.create(account_attrs(password: password)) }
   let(:request) do
-    Stormpath::Authentication::UsernamePasswordRequest.new(account.username,
-                                                           password,
-                                                           account_store: account_store)
+    Stormpath::Authentication::UsernamePasswordRequest.new(
+      account.username,
+      password,
+      account_store: account_store
+    )
   end
   let(:authenticate) { authenticator.authenticate(application.href, request) }
 
@@ -61,8 +59,10 @@ describe 'BasicAuthenticator', vcr: true do
 
     context 'wrong password' do
       let(:request) do
-        Stormpath::Authentication::UsernamePasswordRequest.new(org_account.username,
-                                                               invalid_password)
+        Stormpath::Authentication::UsernamePasswordRequest.new(
+          org_account.username,
+          invalid_password
+        )
       end
 
       it_behaves_like 'an invalid username or password error'
@@ -75,9 +75,11 @@ describe 'BasicAuthenticator', vcr: true do
 
     context 'successful authentication' do
       let(:request) do
-        Stormpath::Authentication::UsernamePasswordRequest.new(org_account.username,
-                                                               password,
-                                                               account_store: organization)
+        Stormpath::Authentication::UsernamePasswordRequest.new(
+          org_account.username,
+          password,
+          account_store: organization
+        )
       end
 
       it_should_behave_like 'an AuthenticationResult'
@@ -85,9 +87,11 @@ describe 'BasicAuthenticator', vcr: true do
 
     context 'wrong password' do
       let(:request) do
-        Stormpath::Authentication::UsernamePasswordRequest.new(org_account.username,
-                                                               invalid_password,
-                                                               account_store: organization)
+        Stormpath::Authentication::UsernamePasswordRequest.new(
+          org_account.username,
+          invalid_password,
+          account_store: organization
+        )
       end
 
       it_behaves_like 'an invalid username or password error'
@@ -97,13 +101,13 @@ describe 'BasicAuthenticator', vcr: true do
       before { map_account_store(application, directory2, 1, false, false) }
       after { directory2.delete }
 
-      let(:another_account) do
-        directory2.accounts.create(account_attrs(username: 'ruby-dir-acc', password: password))
-      end
+      let(:another_account) { directory2.accounts.create(account_attrs(password: password)) }
       let(:request) do
-        Stormpath::Authentication::UsernamePasswordRequest.new(another_account.username,
-                                                               password,
-                                                               account_store: organization)
+        Stormpath::Authentication::UsernamePasswordRequest.new(
+          another_account.username,
+          password,
+          account_store: organization
+        )
       end
 
       it_behaves_like 'an invalid username or password error'

data/spec/auth/register_service_provider_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+describe 'RegisterServiceProvider', vcr: true do
+  let(:client) { test_api_client }
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:identity_provider) { application.saml_policy.identity_provider }
+  let(:assertion_consumer_service_url) { "http://example#{random_number}.zendesk.com/access/saml" }
+  let(:entity_id) { "unique-name-#{random_number}" }
+  let(:registered_service_provider) do
+    Stormpath::Authentication::RegisterServiceProvider.new(identity_provider, options).call
+  end
+  let(:options) do
+    {
+      assertion_consumer_service_url: assertion_consumer_service_url,
+      entity_id: entity_id
+    }
+  end
+
+  after { application.delete }
+
+  describe 'successfull service provider registration' do
+    after { registered_service_provider.delete }
+
+    context 'without optional parameters' do
+      it 'should successfully create a registered_service_provider' do
+        expect(registered_service_provider).to(
+          be_a(Stormpath::Resource::RegisteredSamlServiceProvider)
+        )
+      end
+
+      it 'should successfully map the registered_service_provider to the identity_provider' do
+        expect(identity_provider.registered_saml_service_providers).to(
+          include(registered_service_provider)
+        )
+      end
+    end
+
+    context 'with optional parameters' do
+      before do
+        options[:name] = "service-provider-name-#{random_number}"
+        options[:description] = 'stormpath example'
+        options[:name_id_format] = 'PERSISTENT'
+      end
+
+      it 'should successfully create a registered_service_provider' do
+        expect(registered_service_provider).to(
+          be_a(Stormpath::Resource::RegisteredSamlServiceProvider)
+        )
+      end
+
+      it 'should successfully map the registered_service_provider to the identity_provider' do
+        expect(identity_provider.registered_saml_service_providers).to(
+          include(registered_service_provider)
+        )
+      end
+    end
+  end
+
+  describe 'unsuccessfull service provider registration' do
+    before do
+      options.delete(:assertion_consumer_service_url)
+    end
+
+    it 'should raise Stormpath::Error' do
+      expect { registered_service_provider }.to raise_error(Stormpath::Error)
+    end
+  end
+end

data/spec/auth/sauthc1_signer_spec.rb

@@ -3,10 +3,10 @@ require 'timecop'
 
 describe Stormpath::Http::Authc::Sauthc1Signer do
   let(:fake_uuid_generator) do
-    Proc.new { 'fake-uuid' }
+    proc { 'fake-uuid' }
   end
   let(:signer) do
-    Stormpath::Http::Authc::Sauthc1Signer.new fake_uuid_generator
+    Stormpath::Http::Authc::Sauthc1Signer.new(fake_uuid_generator)
   end
 
   after do
@@ -19,11 +19,15 @@ describe Stormpath::Http::Authc::Sauthc1Signer do
         let(:fake_api_key) { Stormpath::ApiKey.new('foo', 'bar') }
 
         let(:empty_query_hash_request) do
-          Stormpath::Http::Request.new 'get', 'http://example.com/resources/abc123?q=red blue', nil, Hash.new, nil, test_api_key
+          Stormpath::Http::Request.new(
+            'get', 'http://example.com/resources/abc123?q=red blue', nil, {}, nil, test_api_key
+          )
         end
 
         let(:filled_query_hash_request) do
-          Stormpath::Http::Request.new 'get', 'http://example.com/resources/abc123', {'q' => 'red blue'}, Hash.new, nil, test_api_key
+          Stormpath::Http::Request.new(
+            'get', 'http://example.com/resources/abc123', { 'q' => 'red blue' }, {}, nil, test_api_key
+          )
         end
 
         before do

data/spec/cache/cache_entry_spec.rb

@@ -8,7 +8,7 @@ describe Stormpath::Cache::CacheEntry do
 
   context 'by default' do
     let(:cache_entry) do
-      Stormpath::Cache::CacheEntry.new 'foo'
+      Stormpath::Cache::CacheEntry.new('foo')
     end
 
     let(:now) { Time.now }
@@ -19,19 +19,19 @@ describe Stormpath::Cache::CacheEntry do
     end
 
     it 'initializes the creation time to now' do
-      #commenting because it passes locally but not on travis :/
-      #expect(cache_entry.created_at).to eq now
+      # commenting because it passes locally but not on travis :/
+      # expect(cache_entry.created_at).to eq now
     end
 
     it 'initializes the last accessed time to now' do
-      #commenting because it passes locally but not on travis :/
-      #expect(cache_entry.last_accessed_at).to eq now
+      # commenting because it passes locally but not on travis :/
+      # expect(cache_entry.last_accessed_at).to eq now
     end
   end
 
   describe '#touch' do
     let(:cache_entry) do
-      Stormpath::Cache::CacheEntry.new 'foo'
+      Stormpath::Cache::CacheEntry.new('foo')
     end
 
     let(:now) { Time.now }
@@ -42,8 +42,8 @@ describe Stormpath::Cache::CacheEntry do
     end
 
     it 'updates the last accessed at time' do
-      #commenting because it passes locally but not on travis :/
-      #expect(cache_entry.last_accessed_at).to eq now
+      # commenting because it passes locally but not on travis :/
+      # expect(cache_entry.last_accessed_at).to eq now
     end
   end
 
@@ -55,7 +55,7 @@ describe Stormpath::Cache::CacheEntry do
 
     context 'has not expired' do
       let(:cache_entry) do
-        Stormpath::Cache::CacheEntry.new 'foo'
+        Stormpath::Cache::CacheEntry.new('foo')
       end
 
       let(:expired) do
@@ -69,7 +69,7 @@ describe Stormpath::Cache::CacheEntry do
 
     context 'when TTL has expired' do
       before do
-        cache_entry = Stormpath::Cache::CacheEntry.new 'foo'
+        cache_entry = Stormpath::Cache::CacheEntry.new('foo')
         Timecop.freeze now + ttl_seconds + 1
 
         @expired = cache_entry.expired? ttl_seconds, tti_seconds
@@ -82,7 +82,7 @@ describe Stormpath::Cache::CacheEntry do
 
     context 'when TTI has expired' do
       before do
-        cache_entry = Stormpath::Cache::CacheEntry.new 'foo'
+        cache_entry = Stormpath::Cache::CacheEntry.new('foo')
         Timecop.freeze now + tti_seconds + 1
 
         @expired = cache_entry.expired? ttl_seconds, tti_seconds
@@ -96,40 +96,39 @@ describe Stormpath::Cache::CacheEntry do
 
   describe '#to_h' do
     let(:cache_entry) do
-      Stormpath::Cache::CacheEntry.new 'foo'
+      Stormpath::Cache::CacheEntry.new('foo')
     end
 
     let(:now) { Time.now }
     before { Timecop.freeze now }
 
     it 'returns a hash of the attributes' do
-      expect(cache_entry.to_h).to eq({
+      expect(cache_entry.to_h).to eq(
         'value' => cache_entry.value,
         'created_at' => cache_entry.created_at,
         'last_accessed_at' => cache_entry.last_accessed_at
-      })
+      )
     end
   end
 
   describe '.from_h=' do
     let(:hash) do
       {
-        "value" =>
+        'value' =>
           {
-            "href" => "https://api.stormpath.com/v1/applications/app1",
-            "name" => "application app1",
-            "tenant" => {
-              "href" => "https://api.stormpath.com/v1/tenants/ten2"
+            'href' => 'https://api.stormpath.com/v1/applications/app1',
+            'name' => 'application app1',
+            'tenant' => {
+              'href' => 'https://api.stormpath.com/v1/tenants/ten2'
             },
-            "groups" =>
-              {"href" => "https://api.stormpath.com/v1/applications/app3/groups",
-               "items" =>
-                 [{"href" => "https://api.stormpath.com/v1/groups/gro4"},
-                 {"href" => "https://api.stormpath.com/v1/groups/gro5"}]
-              }
-            },
-        "created_at" => "2013-06-05T10:01:31-07:00",
-        "last_accessed_at" => "2013-06-05T10:01:31-07:00"
+            'groups' =>
+              { 'href' => 'https://api.stormpath.com/v1/applications/app3/groups',
+                'items' =>
+                 [{ 'href' => 'https://api.stormpath.com/v1/groups/gro4' },
+                  { 'href' => 'https://api.stormpath.com/v1/groups/gro5' }] }
+          },
+        'created_at' => '2013-06-05T10:01:31-07:00',
+        'last_accessed_at' => '2013-06-05T10:01:31-07:00'
       }
     end
 
@@ -157,4 +156,4 @@ describe Stormpath::Cache::CacheEntry do
       expect(cache_entry.last_accessed_at).to eq last_accessed_at
     end
   end
-end
+end