stormpath-sdk 1.3.1 → 1.4.0

data/spec/resource/email_template_spec.rb

@@ -1,12 +1,12 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::EmailTemplate, :vcr do
-  let(:application) { test_api_client.applications.create(build_application) }
+  let(:application) { test_api_client.applications.create(application_attrs) }
 
   after { application.delete }
 
   describe 'instances should respond to attribute property methods' do
-    let(:directory) { test_api_client.directories.create(build_directory) }
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
     let(:password_policy) { directory.password_policy }
     let(:reset_email_template) { password_policy.reset_email_templates.first }
 

data/spec/resource/factor_spec.rb

@@ -0,0 +1,124 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::Factor, :vcr do
+  describe 'instances should respond to attribute property methods' do
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
+    let(:account) { directory.accounts.create(account_attrs) }
+
+    after do
+      factor.delete if factor
+      account.delete if account
+      directory.delete if directory
+    end
+
+    context 'type sms' do
+      let(:factor) do
+        account.factors.create(
+          type: 'SMS',
+          phone: {
+            number: '+12025550173',
+            name: 'test phone',
+            description: 'this is a testing phone number'
+          }
+        )
+      end
+
+      it do
+        [:type, :status].each do |property_accessor|
+          expect(factor).to respond_to(property_accessor)
+          expect(factor).to respond_to("#{property_accessor}=")
+          expect(factor.send(property_accessor)).to be_a String
+        end
+
+        [:verification_status].each do |property_getter|
+          expect(factor).to respond_to(property_getter)
+          expect(factor.send(property_getter)).to be_a String
+        end
+
+        expect(factor.account).to be_a Stormpath::Resource::Account
+        expect(factor.phone).to be_a Stormpath::Resource::Phone
+        expect(factor.challenges).to be_a Stormpath::Resource::Collection
+      end
+    end
+
+    context 'type google_authenticator' do
+      let(:factor) do
+        account.factors.create(
+          type: 'google-authenticator',
+          issuer: 'ACME'
+        )
+      end
+
+      it do
+        [:type, :status].each do |property_accessor|
+          expect(factor).to respond_to(property_accessor)
+          expect(factor).to respond_to("#{property_accessor}=")
+          expect(factor.send(property_accessor)).to be_a String
+        end
+
+        [:verification_status, :secret, :key_uri, :base64_q_r_image, :qr_code].each do |property_getter|
+          expect(factor).to respond_to(property_getter)
+          expect(factor.send(property_getter)).to be_a String
+        end
+
+        expect(factor.account).to be_a Stormpath::Resource::Account
+      end
+    end
+  end
+
+  describe 'factor associations' do
+    let(:app) { test_api_client.applications.create(application_attrs) }
+    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) }
+    let(:account) { directory.accounts.create(account_attrs) }
+    let(:phone_number) { '+12025550173' }
+
+    let(:factor) do
+      account.factors.create(
+        type: 'SMS',
+        phone: {
+          number: phone_number,
+          name: 'test phone',
+          description: 'this is a testing phone number'
+        }
+      )
+    end
+
+    it 'should belong_to account' do
+      expect(factor.account).to eq(account)
+    end
+
+    it 'should have one phone' do
+      expect(factor.phone.number).to eq(phone_number)
+    end
+
+    context 'challenges' do
+      before do
+        stub_request(:post, "#{factor.href}/challenges")
+          .to_return(body: Stormpath::Test.mocked_challenge)
+        stub_request(:get, factor.href)
+          .to_return(body: Stormpath::Test.mocked_factor_response)
+      end
+      let!(:challenge) { factor.challenges.create(message: 'Enter code: ${code}') }
+
+      it 'should have a collection of challenges' do
+        expect(factor.challenges).to be_a Stormpath::Resource::Collection
+      end
+
+      it 'should have the most recent challenge' do
+        most_recent_challenge = factor.challenges.create(message: 'Enter new code: ${code}')
+        reloaded_factor = account.factors.get(factor.href)
+        expect(reloaded_factor.most_recent_challenge.href).to eq(most_recent_challenge.href)
+      end
+    end
+
+    after do
+      application.delete if application
+      account.delete if account
+      directory.delete if directory
+      factor.delete if factor
+    end
+  end
+end

data/spec/resource/field_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::Field, :vcr do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:directory) { test_api_client.directories.create(directory_attrs) }
+  let(:schema) { directory.account_schema }
+  let(:field) { schema.fields.first }
+
+  after do
+    directory.delete
+    application.delete
+  end
+
+  describe 'instances should respond to attribute property methods' do
+    it do
+      expect(field).to be_a Stormpath::Resource::Field
+
+      [:name, :created_at, :modified_at].each do |property_getter|
+        expect(field).to respond_to(property_getter)
+        expect(field.send(property_getter)).to be_a String
+      end
+
+      expect(field).to respond_to(:required)
+      expect(field.required).to eq !!field.required
+    end
+  end
+
+  describe 'field associations' do
+    context '#schema' do
+      it 'should be able to get the account schema' do
+        expect(field.schema).to be_a Stormpath::Resource::Schema
+      end
+    end
+  end
+end

data/spec/resource/group_membership_spec.rb

@@ -7,9 +7,9 @@ describe Stormpath::Resource::GroupMembership, :vcr do
 
   describe '#add_account' do
     context 'given an account and a group' do
-      let(:directory) { test_api_client.directories.create(build_directory) }
-      let(:group) { directory.groups.create(build_group) }
-      let(:account) { directory.accounts.create(build_account) }
+      let(:directory) { test_api_client.directories.create(directory_attrs) }
+      let(:group) { directory.groups.create(group_attrs) }
+      let(:account) { directory.accounts.create(account_attrs) }
 
       before { group.add_account account }
 

data/spec/resource/group_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::Group, :vcr do
-  let(:directory) { test_api_client.directories.create(build_directory) }
+  let(:directory) { test_api_client.directories.create(directory_attrs) }
   after { directory.delete }
 
   describe 'instances should respond to attribute property methods' do
@@ -40,8 +40,8 @@ describe Stormpath::Resource::Group, :vcr do
 
   describe '#add_or_remove_account' do
     context 'given an account' do
-      let(:group) { directory.groups.create(build_group) }
-      let(:account) { directory.accounts.create(build_account) }
+      let(:group) { directory.groups.create(group_attrs) }
+      let(:account) { directory.accounts.create(account_attrs) }
 
       before { group.add_account(account) }
 

data/spec/resource/linked_account_spec.rb

@@ -7,10 +7,10 @@ describe Stormpath::Resource::LinkedAccount, :vcr do
   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(:account1) do
-    directory1.accounts.create(build_account(email: 'jekyll', username: 'jekyll'))
+    directory1.accounts.create(account_attrs(email: 'jekyll', username: 'jekyll'))
   end
   let(:account2) do
-    directory2.accounts.create(build_account(email: 'hyde', username: 'hyde'))
+    directory2.accounts.create(account_attrs(email: 'hyde', username: 'hyde'))
   end
 
   before do

data/spec/resource/organization_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe Stormpath::Resource::Organization, :vcr do
   let(:organization) do
-    test_api_client.organizations.create(build_organization(name: 'test_ruby_organization',
+    test_api_client.organizations.create(organization_attrs(name: 'test_ruby_organization',
                                                             name_key: 'testorganization',
                                                             description: 'test organization'))
   end
@@ -56,14 +56,14 @@ describe Stormpath::Resource::Organization, :vcr do
 
       it 'should raise Stormpath::Error' do
         expect do
-          test_api_client.organizations.create(build_organization(name_key: 'testorganization'))
+          test_api_client.organizations.create(organization_attrs(name_key: 'testorganization'))
         end.to raise_error(Stormpath::Error)
       end
     end
   end
 
   describe 'associations' do
-    let(:directory) { test_api_client.directories.create(build_directory) }
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
     after { directory.delete }
 
     context 'groups' do
@@ -77,8 +77,8 @@ describe Stormpath::Resource::Organization, :vcr do
     end
 
     context 'accounts' do
-      let(:account) { directory.accounts.create(build_account(email: 'rubysdk')) }
-      let(:org_account) { directory.accounts.create(build_account(email: 'rubysdk2')) }
+      let(:account) { directory.accounts.create(account_attrs(email: 'rubysdk')) }
+      let(:org_account) { directory.accounts.create(account_attrs(email: 'rubysdk2')) }
 
       before { map_organization_store(directory, organization, true) }
 
@@ -129,7 +129,7 @@ describe Stormpath::Resource::Organization, :vcr do
   end
 
   describe 'organization account store mapping' do
-    let(:directory) { test_api_client.directories.create(build_directory) }
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
     after { directory.delete }
 
     context 'given an account_store is a directory' do

data/spec/resource/password_policy_spec.rb

@@ -1,12 +1,12 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::PasswordPolicy, :vcr do
-  let(:application) { test_api_client.applications.create(build_application) }
+  let(:application) { test_api_client.applications.create(application_attrs) }
 
   after { application.delete }
 
   describe 'instances should respond to attribute property methods' do
-    let(:directory) { test_api_client.directories.create(build_directory) }
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
     let(:password_policy) { directory.password_policy }
 
     before do

data/spec/resource/password_strength_spec.rb

@@ -1,12 +1,12 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::PasswordStrength, :vcr do
-  let(:application) { test_api_client.applications.create(build_application) }
+  let(:application) { test_api_client.applications.create(application_attrs) }
 
   after { application.delete }
 
   describe 'instances should respond to attribute property methods' do
-    let(:directory) { test_api_client.directories.create(build_directory) }
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
     let(:password_policy) { directory.password_policy }
     let(:password_strength) { password_policy.strength }
 

data/spec/resource/phone_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::Phone, :vcr do
+  describe 'instances should respond to attribute property methods' do
+    let(:directory) { test_api_client.directories.create(directory_attrs) }
+    let(:account) { directory.accounts.create(account_attrs) }
+    let(:phone) do
+      account.phones.create(
+        number: '+12025550173',
+        name: 'test phone',
+        description: 'this is a testing phone number'
+      )
+    end
+
+    after do
+      phone.delete
+      account.delete
+      directory.delete
+    end
+
+    it do
+      [:number, :name, :description].each do |property_accessor|
+        expect(phone).to respond_to(property_accessor)
+        expect(phone).to respond_to("#{property_accessor}=")
+        expect(phone.send(property_accessor)).to be_a String
+      end
+
+      [:verification_status, :status, :created_at, :modified_at].each do |property_getter|
+        expect(phone).to respond_to(property_getter)
+        expect(phone.send(property_getter)).to be_a String
+      end
+
+      expect(phone.account).to be_a Stormpath::Resource::Account
+    end
+  end
+
+  describe 'phone associations' do
+    let(:app) { test_api_client.applications.create(application_attrs) }
+    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) }
+    let(:account) { directory.accounts.create(account_attrs) }
+    let(:phone) do
+      account.phones.create(
+        number: '+12025550173',
+        name: 'test phone',
+        description: 'this is a testing phone number'
+      )
+    end
+
+    it 'should belong_to account' do
+      expect(phone.account).to eq(account)
+    end
+
+    after do
+      application.delete if application
+      account.delete if account
+      directory.delete if directory
+      phone.delete if phone
+    end
+  end
+end

data/spec/resource/schema_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe Stormpath::Resource::Schema, :vcr do
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:directory) { test_api_client.directories.create(directory_attrs) }
+  let(:schema) { directory.account_schema }
+
+  after do
+    directory.delete
+    application.delete
+  end
+
+  describe 'instances should respond to attribute property methods' do
+    it do
+      expect(schema).to be_a Stormpath::Resource::Schema
+
+      [:created_at, :modified_at].each do |property_getter|
+        expect(schema).to respond_to(property_getter)
+        expect(schema.send(property_getter)).to be_a String
+      end
+    end
+  end
+
+  describe 'schema associations' do
+    context '#fields' do
+      let(:field) { directory.fields.first }
+
+      it 'should be able to get a list of fields' do
+        expect(schema.fields).to be_a Stormpath::Resource::Collection
+      end
+    end
+
+    context '#directory' do
+      it 'should be able to fetch the directory' do
+        expect(schema.directory).to eq directory
+      end
+    end
+  end
+end

data/spec/resource/status_spec.rb

@@ -7,11 +7,11 @@ describe 'StatusOnDirectoryAndAccount', :vcr do
   let(:authenticate_user) do
     application.authenticate_account(auth_request)
   end
-  let(:directory) { test_api_client.directories.create(build_directory) }
-  let(:application) { test_api_client.applications.create(build_application) }
-  let(:group) { directory.groups.create(build_group) }
+  let(:directory) { test_api_client.directories.create(directory_attrs) }
+  let(:application) { test_api_client.applications.create(application_attrs) }
+  let(:group) { directory.groups.create(group_attrs) }
   let!(:account) do
-    directory.accounts.create(build_account(email: 'rubytest', password: 'P@$$w0rd'))
+    directory.accounts.create(account_attrs(email: 'rubytest', password: 'P@$$w0rd'))
   end
   let(:reloaded_account) { test_api_client.accounts.get account.href }
   before { map_account_store(application, directory, 0, true, true) }

data/spec/spec_helper.rb

@@ -15,15 +15,18 @@ require 'uuidtools'
 
 Dir['./spec/support/*.rb'].each { |file| require file }
 
-HIJACK_HTTP_REQUESTS_WITH_VCR = ENV['STORMPATH_SDK_TEST_ENVIRONMENT'] != 'CI'
-
 WebMock.allow_net_connect!
 
 VCR.configure do |c|
   c.cassette_library_dir = 'spec/fixtures/vcr_cassettes'
   c.hook_into :webmock
   c.configure_rspec_metadata!
-  c.ignore_request { |r| HIJACK_HTTP_REQUESTS_WITH_VCR == false }
+
+  c.before_record do |i|
+    i.request.headers.delete('Authorization')
+    u = URI.parse(i.request.uri)
+    i.request.uri.sub!(/:\/\/.*#{Regexp.escape(u.host)}/, "://#{u.host}" )
+  end
 end
 
 RSpec.configure do |c|

data/spec/support/custom_data_storage_behavior.rb

@@ -1,6 +1,6 @@
 shared_examples_for 'account_custom_data' do
   context 'account' do
-    let(:custom_data_storage) { directory.accounts.create(build_account) }
+    let(:custom_data_storage) { directory.accounts.create(account_attrs) }
     let(:custom_data_storage_w_nested_custom_data) do
       directory.accounts.create(
         username: 'ruby username',
@@ -24,7 +24,7 @@ end
 
 shared_examples_for 'group_custom_data' do
   context 'group' do
-    let(:custom_data_storage) { directory.groups.create(build_group) }
+    let(:custom_data_storage) { directory.groups.create(group_attrs) }
     let(:custom_data_storage_w_nested_custom_data) do
       directory.groups.create(
         name: 'ruby group',