stormpath-sdk 1.6.0 → 1.7.0

data/spec/resource/expansion_spec.rb

@@ -4,20 +4,20 @@ describe Stormpath::Resource::Expansion, :vcr do
   describe '#initialize' do
     context 'given a single property name' do
       let(:expansion) do
-        Stormpath::Resource::Expansion.new 'foo'
+        Stormpath::Resource::Expansion.new('foo')
       end
 
       it 'can be transmuted to a simple hash' do
-        expect(expansion.to_query).to eq({ expand: 'foo' })
+        expect(expansion.to_query).to eq(expand: 'foo')
       end
     end
     context 'given a list of property names' do
       let(:expansion) do
-        Stormpath::Resource::Expansion.new 'foo', 'bar'
+        Stormpath::Resource::Expansion.new('foo', 'bar')
       end
 
       it 'can be transmuted to a simple hash' do
-        expect(expansion.to_query).to eq({ expand: 'foo,bar' })
+        expect(expansion.to_query).to eq(expand: 'foo,bar')
       end
     end
 
@@ -32,7 +32,7 @@ describe Stormpath::Resource::Expansion, :vcr do
     end
   end
 
-  describe "#add_property" do
+  describe '#add_property' do
     context 'given a simple property name' do
       let(:expansion) { Stormpath::Resource::Expansion.new }
 
@@ -41,7 +41,7 @@ describe Stormpath::Resource::Expansion, :vcr do
       end
 
       it 'can be transmuted to a simple hash' do
-        expect(expansion.to_query).to eq({ expand: 'foo' })
+        expect(expansion.to_query).to eq(expand: 'foo')
       end
     end
 
@@ -54,7 +54,7 @@ describe Stormpath::Resource::Expansion, :vcr do
       end
 
       it 'can be transmuted to a simple hash' do
-        expect(expansion.to_query).to eq({ expand: 'foo,bar' })
+        expect(expansion.to_query).to eq(expand: 'foo,bar')
       end
     end
 
@@ -68,7 +68,7 @@ describe Stormpath::Resource::Expansion, :vcr do
       end
 
       it 'will not duplicate the property' do
-        expect(expansion.to_query).to eq({ expand: 'foo,bar' })
+        expect(expansion.to_query).to eq(expand: 'foo,bar')
       end
     end
 
@@ -80,7 +80,7 @@ describe Stormpath::Resource::Expansion, :vcr do
       end
 
       it 'can be transmuted to a simple hash' do
-        expect(expansion.to_query).to eq({ expand: 'foo(offset:5,limit:100)' })
+        expect(expansion.to_query).to eq(expand: 'foo(offset:5,limit:100)')
       end
     end
 
@@ -93,7 +93,7 @@ describe Stormpath::Resource::Expansion, :vcr do
       end
 
       it 'allows the last call to win out over the first' do
-        expect(expansion.to_query).to eq({ expand: 'foo(offset:25)' })
+        expect(expansion.to_query).to eq(expand: 'foo(offset:25)')
       end
     end
   end

data/spec/resource/factor_spec.rb

@@ -68,7 +68,7 @@ describe Stormpath::Resource::Factor, :vcr do
 
   describe 'factor associations' do
     let(:app) { test_api_client.applications.create(application_attrs) }
-    let(:application) { test_api_client.applications.get app.href }
+    let(:application) { test_api_client.applications.get(app.href) }
     let(:directory) { test_api_client.directories.create(directory_attrs) }
 
     before { map_account_store(app, directory, 1, true, true) }

data/spec/resource/group_spec.rb

@@ -5,7 +5,7 @@ describe Stormpath::Resource::Group, :vcr do
   after { directory.delete }
 
   describe 'instances should respond to attribute property methods' do
-    let(:group) { directory.groups.create name: 'RubyTestGroup', description: 'testDescription' }
+    let(:group) { directory.groups.create(group_attrs) }
 
     after { group.delete }
 

data/spec/resource/linked_account_spec.rb

@@ -1,16 +1,16 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::LinkedAccount, :vcr do
-  let(:application) do
-    test_api_client.applications.create(name: 'ruby sdk app', description: 'ruby sdk desc')
-  end
-  let(:directory1) { test_api_client.directories.create(name: 'ruby sdk dir 1') }
-  let(:directory2) { test_api_client.directories.create(name: 'ruby sdk dir 2') }
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:directory1) { test_api_client.directories.create(directory_attrs) }
+  let(:directory2) { test_api_client.directories.create(directory_attrs) }
+  let(:username_1) { "jekyll-#{random_number}" }
+  let(:username_2) { "hyde-#{random_number}" }
   let(:account1) do
-    directory1.accounts.create(account_attrs(email: 'jekyll', username: 'jekyll'))
+    directory1.accounts.create(account_attrs(email: username_1, username: username_1))
   end
   let(:account2) do
-    directory2.accounts.create(account_attrs(email: 'hyde', username: 'hyde'))
+    directory2.accounts.create(account_attrs(email: username_2, username: username_2))
   end
 
   before do

data/spec/resource/organization_spec.rb

@@ -1,12 +1,10 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::Organization, :vcr do
+  let(:organization_name_key) { "rubysdk-org-#{random_number}" }
   let(:organization) do
-    test_api_client.organizations.create(organization_attrs(name: 'test_ruby_organization',
-                                                            name_key: 'testorganization',
-                                                            description: 'test organization'))
+    test_api_client.organizations.create(organization_attrs(name_key: organization_name_key))
   end
-
   after { organization.delete if organization }
 
   describe 'instances should respond to attribute property methods' do
@@ -57,7 +55,7 @@ describe Stormpath::Resource::Organization, :vcr do
 
       it 'should raise Stormpath::Error' do
         expect do
-          test_api_client.organizations.create(organization_attrs(name_key: 'testorganization'))
+          test_api_client.organizations.create(organization_attrs(name_key: organization_name_key))
         end.to raise_error(Stormpath::Error)
       end
     end
@@ -68,7 +66,7 @@ describe Stormpath::Resource::Organization, :vcr do
     after { directory.delete }
 
     context 'groups' do
-      let(:group) { directory.groups.create(name: 'test_group') }
+      let(:group) { directory.groups.create(organization_attrs) }
       before { map_organization_store(group, organization) }
 
       it 'returns a collection of groups' do
@@ -78,8 +76,10 @@ describe Stormpath::Resource::Organization, :vcr do
     end
 
     context 'accounts' do
-      let(:account) { directory.accounts.create(account_attrs(email: 'rubysdk')) }
-      let(:org_account) { directory.accounts.create(account_attrs(email: 'rubysdk2')) }
+      let(:account_email_1) { "rubysdk-#{random_number}" }
+      let(:account_email_2) { "rubysdk-#{random_number}" }
+      let(:account) { directory.accounts.create(account_attrs(email: account_email_1)) }
+      let(:org_account) { directory.accounts.create(account_attrs(email: account_email_2)) }
 
       before { map_organization_store(directory, organization, true) }
 
@@ -118,14 +118,15 @@ describe Stormpath::Resource::Organization, :vcr do
   end
 
   describe 'update' do
+    let(:changed_org_name_key) { "rubysdk-org-#{random_number}" }
     before do
-      organization.name_key = 'changed-test-organization'
+      organization.name_key = changed_org_name_key
       organization.save
     end
 
     it 'can change the data of the existing organization' do
       org = test_api_client.organizations.get(organization.href)
-      expect(org.name_key).to eq('changed-test-organization')
+      expect(org.name_key).to eq(changed_org_name_key)
     end
   end
 
@@ -146,7 +147,7 @@ describe Stormpath::Resource::Organization, :vcr do
     end
 
     context 'given an account_store is a group' do
-      let(:group) { directory.groups.create(name: 'test_group') }
+      let(:group) { directory.groups.create(group_attrs) }
       let(:organization_account_store_mapping) { map_organization_store(group, organization) }
       let(:reloaded_mapping) do
         test_api_client.account_store_mappings.get(organization_account_store_mapping.href)

data/spec/resource/phone_spec.rb

@@ -36,7 +36,7 @@ describe Stormpath::Resource::Phone, :vcr do
 
   describe 'phone associations' do
     let(:app) { test_api_client.applications.create(application_attrs) }
-    let(:application) { test_api_client.applications.get app.href }
+    let(:application) { test_api_client.applications.get(app.href) }
     let(:directory) { test_api_client.directories.create(directory_attrs) }
 
     before { map_account_store(app, directory, 1, true, true) }

data/spec/resource/registered_saml_service_provider_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::RegisteredSamlServiceProvider, vcr: true do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:identity_provider) { application.saml_policy.identity_provider }
+  let(:assertion_consumer_service_url) { 'https://some.sp.com/saml/sso/post' }
+  let(:entity_id) { 'urn:sp:A1B2C3' }
+  let(:registered_service_provider) do
+    application.register_service_provider(
+      assertion_consumer_service_url: assertion_consumer_service_url, entity_id: entity_id
+    )
+  end
+
+  after do
+    registered_service_provider.delete
+    application.delete
+  end
+
+  it 'instances should respond to attribute property methods' do
+    expect(registered_service_provider).to be_a Stormpath::Resource::RegisteredSamlServiceProvider
+
+    [:created_at, :modified_at].each do |prop_reader|
+      expect(registered_service_provider).to respond_to(prop_reader)
+      expect(registered_service_provider.send(prop_reader)).to be_a String
+    end
+
+    [:name, :description, :assertion_consumer_service_url,
+     :entity_id, :name_id_format].each do |prop_accessor|
+       expect(registered_service_provider).to respond_to(prop_accessor)
+       expect(registered_service_provider).to respond_to("#{prop_accessor}=")
+     end
+
+    expect(registered_service_provider.encoded_x509_certificate).to be_nil
+  end
+end

data/spec/resource/saml_identity_provider_metadata_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::SamlIdentityProviderMetadata, vcr: true do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:metadata) { application.saml_policy.identity_provider.metadata }
+
+  after { application.delete }
+
+  it 'instances should respond to attribute property methods' do
+    expect(metadata).to be_a Stormpath::Resource::SamlIdentityProviderMetadata
+
+    [:entity_id].each do |property_getter|
+      expect(metadata).to respond_to(property_getter)
+      expect(metadata.send(property_getter)).to be_a String
+    end
+  end
+
+  describe 'saml identity provider metadata associations' do
+    it 'should respond to identity provider' do
+      expect(metadata.identity_provider).to be_a Stormpath::Resource::SamlIdentityProvider
+    end
+
+    it 'should respond to x509_signing_cert' do
+      expect(metadata.x509_signing_cert).to be_a Stormpath::Resource::X509Certificate
+    end
+  end
+end

data/spec/resource/saml_identity_provider_spec.rb

@@ -0,0 +1,94 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::SamlIdentityProvider, vcr: true do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:identity_provider) { application.saml_policy.identity_provider }
+
+  after { application.delete }
+
+  it 'instances should respond to attribute property methods' do
+    expect(identity_provider).to be_a Stormpath::Resource::SamlIdentityProvider
+
+    [:sso_login_endpoint].each do |property_getter|
+      expect(identity_provider).to respond_to(property_getter)
+      expect(identity_provider.send(property_getter)).to be_a Hash
+    end
+
+    [:signature_algorithm, :sha_fingerprint, :created_at, :modified_at].each do |property_getter|
+      expect(identity_provider).to respond_to(property_getter)
+      expect(identity_provider.send(property_getter)).to be_a String
+    end
+
+    [:status].each do |property_accessor|
+      expect(application).to respond_to(property_accessor)
+      expect(application).to respond_to("#{property_accessor}=")
+      expect(application.send(property_accessor)).to be_a String
+    end
+  end
+
+  describe 'saml identity provider associations' do
+    it 'should respond to registered_saml_service_providers' do
+      expect(identity_provider.registered_saml_service_providers).to(
+        be_a(Stormpath::Resource::Collection)
+      )
+    end
+
+    it 'should respond to saml_service_provider_registrations' do
+      expect(identity_provider.saml_service_provider_registrations).to(
+        be_a(Stormpath::Resource::Collection)
+      )
+    end
+
+    it 'should respond to metadata' do
+      expect(identity_provider.metadata).to be_a Stormpath::Resource::SamlIdentityProviderMetadata
+    end
+
+    describe 'attribute_statement_mapping_rules' do
+      let(:rule) do
+        { 'name' => 'email',
+          'nameFormat' => 'urn:oasis:names:tc:SAML:2.0:nameid-format:entity',
+          'accountAttributes' => ['email'] }
+      end
+      before do
+        identity_provider.attribute_statement_mapping_rules.items = [rule]
+        identity_provider.attribute_statement_mapping_rules.save
+      end
+
+      it 'should respond with attribute_statement_mapping_rules' do
+        expect(identity_provider.attribute_statement_mapping_rules).to(
+          be_a(Stormpath::Resource::AttributeStatementMappingRules)
+        )
+      end
+
+      it 'should contain the saved rule' do
+        expect(identity_provider.attribute_statement_mapping_rules.items).to include(rule)
+      end
+    end
+
+    it 'should respond to x509_signing_cert' do
+      expect(identity_provider.x509_signing_cert).to be_a Stormpath::Resource::X509Certificate
+    end
+  end
+
+  describe 'map existing registered service provider' do
+    let(:assertion_consumer_service_url) { 'https://some.sp.com/saml/sso/post' }
+    let(:entity_id) { 'urn:sp:A1B2C3' }
+    let!(:service_provider) do
+      test_api_client.registered_saml_service_providers.create(
+        assertion_consumer_service_url: assertion_consumer_service_url,
+        entity_id: entity_id
+      )
+    end
+
+    before do
+      identity_provider.saml_service_provider_registrations.create(
+        service_provider: { href: service_provider.href }
+      )
+    end
+    after { service_provider.delete }
+
+    it 'should successfully map with the identity provider' do
+      expect(identity_provider.registered_saml_service_providers).to include(service_provider)
+    end
+  end
+end

data/spec/resource/saml_policy_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::SamlPolicy, :vcr do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:saml_policy) { application.saml_policy }
+
+  after { application.delete }
+
+  it 'instances should respond to attribute property methods' do
+    expect(saml_policy).to be_a Stormpath::Resource::SamlPolicy
+
+    [:created_at, :modified_at].each do |property_getter|
+      expect(saml_policy).to respond_to(property_getter)
+      expect(saml_policy.send(property_getter)).to be_a String
+    end
+  end
+
+  describe 'saml policy associations' do
+    it 'should respond to service_provider' do
+      expect(saml_policy.service_provider).to be_a Stormpath::Resource::SamlServiceProvider
+    end
+
+    it 'should respond to identity_provider' do
+      expect(saml_policy.identity_provider).to be_a Stormpath::Resource::SamlIdentityProvider
+    end
+  end
+end

data/spec/resource/saml_service_provider_registration_spec.rb

@@ -0,0 +1,58 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::SamlServiceProviderRegistration, vcr: true do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:identity_provider) { application.saml_policy.identity_provider }
+  let(:assertion_consumer_service_url) { 'https://some.sp.com/saml/sso/post' }
+  let(:entity_id) { random_number }
+  let(:options) do
+    {
+      assertion_consumer_service_url: assertion_consumer_service_url,
+      entity_id: entity_id
+    }
+  end
+  let(:service_provider) do
+    Stormpath::Authentication::RegisterServiceProvider.new(identity_provider, options).call
+  end
+  let(:service_provider_registration) { identity_provider.saml_service_provider_registrations.first }
+
+  before do
+    service_provider
+    service_provider_registration.default_relay_state = 'example_jwt'
+    service_provider_registration.save
+  end
+
+  after do
+    service_provider.delete
+    application.delete
+  end
+
+  it 'instances should respond to attribute property methods' do
+    expect(service_provider_registration).to be_a Stormpath::Resource::SamlServiceProviderRegistration
+
+    [:created_at, :modified_at].each do |prop_reader|
+      expect(service_provider_registration).to respond_to(prop_reader)
+      expect(service_provider_registration.send(prop_reader)).to be_a String
+    end
+
+    [:status, :default_relay_state].each do |property_accessor|
+      expect(service_provider_registration).to respond_to(property_accessor)
+      expect(service_provider_registration).to respond_to("#{property_accessor}=")
+      expect(service_provider_registration.send(property_accessor)).to be_a String
+    end
+  end
+
+  describe 'associations' do
+    it 'should respond to identity_provider' do
+      expect(service_provider_registration.identity_provider).to(
+        be_a(Stormpath::Resource::SamlIdentityProvider)
+      )
+    end
+
+    it 'should respond to service_provider' do
+      expect(service_provider_registration.service_provider).to(
+        be_a(Stormpath::Resource::RegisteredSamlServiceProvider)
+      )
+    end
+  end
+end