stormpath-sdk 1.3.0 → 1.3.1

data/spec/resource/password_policy_spec.rb

@@ -1,19 +1,16 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::PasswordPolicy, :vcr do
-  describe "instances should respond to attribute property methods" do
-    let(:application) { test_application }
-    let(:directory) { test_api_client.directories.create(name: random_directory_name) }
+  let(:application) { test_api_client.applications.create(build_application) }
+
+  after { application.delete }
+
+  describe 'instances should respond to attribute property methods' do
+    let(:directory) { test_api_client.directories.create(build_directory) }
     let(:password_policy) { directory.password_policy }
 
     before do
-      test_api_client.account_store_mappings.create(
-        application: application,
-        account_store: directory,
-        list_index: 1,
-        is_default_account_store: false,
-        is_default_group_store: false
-      )
+      map_account_store(application, directory, 1, false, false)
     end
 
     after { directory.delete }
@@ -42,17 +39,17 @@ describe Stormpath::Resource::PasswordPolicy, :vcr do
     end
 
     it 'can change reset_email_status' do
-      expect(directory.password_policy.reset_email_status).to eq("ENABLED")
-      password_policy.reset_email_status = "DISABLED"
+      expect(directory.password_policy.reset_email_status).to eq('ENABLED')
+      password_policy.reset_email_status = 'DISABLED'
       password_policy.save
-      expect(directory.password_policy.reset_email_status).to eq("DISABLED")
+      expect(directory.password_policy.reset_email_status).to eq('DISABLED')
     end
 
     it 'can change reset_success_email_status' do
-      expect(directory.password_policy.reset_success_email_status).to eq("ENABLED")
-      password_policy.reset_success_email_status = "DISABLED"
+      expect(directory.password_policy.reset_success_email_status).to eq('ENABLED')
+      password_policy.reset_success_email_status = 'DISABLED'
       password_policy.save
-      expect(directory.password_policy.reset_success_email_status).to eq("DISABLED")
+      expect(directory.password_policy.reset_success_email_status).to eq('DISABLED')
     end
   end
 end

data/spec/resource/password_strength_spec.rb

@@ -1,20 +1,17 @@
 require 'spec_helper'
 
 describe Stormpath::Resource::PasswordStrength, :vcr do
-  describe "instances should respond to attribute property methods" do
-    let(:application) { test_application }
-    let(:directory) { test_api_client.directories.create(name: random_directory_name) }
+  let(:application) { test_api_client.applications.create(build_application) }
+
+  after { application.delete }
+
+  describe 'instances should respond to attribute property methods' do
+    let(:directory) { test_api_client.directories.create(build_directory) }
     let(:password_policy) { directory.password_policy }
     let(:password_strength) { password_policy.strength }
 
     before do
-      test_api_client.account_store_mappings.create(
-        application: application,
-        account_store: directory,
-        list_index: 1,
-        is_default_account_store: false,
-        is_default_group_store: false
-      )
+      map_account_store(application, directory, 1, false, false)
     end
 
     after { directory.delete }
@@ -22,14 +19,14 @@ describe Stormpath::Resource::PasswordStrength, :vcr do
     it do
       expect(password_strength).to be_a Stormpath::Resource::PasswordStrength
 
-      [ :min_length,
-        :max_length,
-        :min_lower_case,
-        :min_upper_case,
-        :min_numeric,
-        :min_symbol,
-        :min_diacritic,
-        :prevent_reuse].each do |property_accessor|
+      [:min_length,
+       :max_length,
+       :min_lower_case,
+       :min_upper_case,
+       :min_numeric,
+       :min_symbol,
+       :min_diacritic,
+       :prevent_reuse].each do |property_accessor|
         expect(password_strength).to respond_to(property_accessor)
         expect(password_strength).to respond_to("#{property_accessor}=")
       end

data/spec/resource/status_spec.rb

@@ -1,67 +1,64 @@
 require 'spec_helper'
 
-describe 'status', :vcr do
-  def authenticate_user
-    auth_request = Stormpath::Authentication::UsernamePasswordRequest.new 'test@example.com', 'P@$$w0rd'
-    account_store_mapping unless account_store_mapping
-    application.authenticate_account auth_request
+describe 'StatusOnDirectoryAndAccount', :vcr do
+  let(:auth_request) do
+    Stormpath::Authentication::UsernamePasswordRequest.new("rubytest#{default_domain}", 'P@$$w0rd')
+  end
+  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!(:account) do
+    directory.accounts.create(build_account(email: 'rubytest', password: 'P@$$w0rd'))
   end
-
-  let(:directory) { test_api_client.directories.create name: random_directory_name, description: 'testDirectory for statuses' }
-
-  let(:application) { test_api_client.applications.create name: random_application_name, description: 'testDirectory for statuses' }
-
-  let(:group) { directory.groups.create name: 'testGroup', description: 'testGroup' }
-
-  let(:account_store_mapping) { test_api_client.account_store_mappings.create application: application, account_store: directory }
-
-  let!(:account) { directory.accounts.create email: 'test@example.com', password: 'P@$$w0rd', given_name: "Ruby SDK", surname: 'SDK' }
-
   let(:reloaded_account) { test_api_client.accounts.get account.href }
+  before { map_account_store(application, directory, 0, true, true) }
 
   after do
     application.delete if application
     directory.delete if directory
   end
 
-  it "should respond to status getter and setter" do
-    expect(directory.respond_to? :status).to be_truthy
-    expect(directory.respond_to? :status=).to be_truthy
+  it 'should respond to status getter and setter' do
+    expect(directory.respond_to?(:status)).to be_truthy
+    expect(directory.respond_to?(:status=)).to be_truthy
 
-    expect(application.respond_to? :status).to be_truthy
-    expect(application.respond_to? :status=).to be_truthy
+    expect(application.respond_to?(:status)).to be_truthy
+    expect(application.respond_to?(:status=)).to be_truthy
 
-    expect(group.respond_to? :status).to be_truthy
-    expect(group.respond_to? :status=).to be_truthy
+    expect(group.respond_to?(:status)).to be_truthy
+    expect(group.respond_to?(:status=)).to be_truthy
 
-    expect(account.respond_to? :status).to be_truthy
-    expect(account.respond_to? :status=).to be_truthy
+    expect(account.respond_to?(:status)).to be_truthy
+    expect(account.respond_to?(:status=)).to be_truthy
   end
 
-  it "users status by default should be ENABLED" do
-    expect(account.status).to eq("ENABLED")
+  it 'users status by default should be ENABLED' do
+    expect(account.status).to eq('ENABLED')
   end
 
-  it "change user status" do
-    account.status = "DISABLED"
+  it 'change user status' do
+    account.status = 'DISABLED'
     account.save
-    expect(reloaded_account.status).to eq("DISABLED")
+    expect(reloaded_account.status).to eq('DISABLED')
   end
 
-  it "authenticate user with status ENABLED" do
-    expect(authenticate_user.properties["account"]["href"]).to eq(account.href)
+  it 'authenticate user with status ENABLED' do
+    expect(authenticate_user.properties['account']['href']).to eq(account.href)
   end
 
   it "shouldn't authenticate users with status DISABLED, UNVERIFIED or LOCKED" do
-    ["DISABLED", "UNVERIFIED", "LOCKED"].each do |status|
+    ['DISABLED', 'UNVERIFIED', 'LOCKED'].each do |status|
       account.status = status
       account.save
-      expect{authenticate_user}.to raise_exception(Stormpath::Error)
+      expect { authenticate_user }.to raise_exception(Stormpath::Error)
     end
   end
 
   it 'assigning inappropriate status states should fail silently' do
-    account.status = "INVALID_STATUS_VALUE"
+    account.status = 'INVALID_STATUS_VALUE'
     expect { account.save }.to raise_error(Stormpath::Error)
   end
 end

data/spec/spec_helper.rb

@@ -26,149 +26,18 @@ VCR.configure do |c|
   c.ignore_request { |r| HIJACK_HTTP_REQUESTS_WITH_VCR == false }
 end
 
-RSpec::Matchers.define :be_boolean do
-  match do |actual|
-    actual.should satisfy { |x| x == true || x == false }
-  end
-end
-
-module Stormpath
-  module TestApiKeyHelpers
-    TEST_ENV_REQUIRED_VARS = {
-      STORMPATH_SDK_TEST_API_KEY_ID: 'The id form your Stormpath API Key',
-      STORMPATH_SDK_TEST_API_KEY_SECRET: 'The secret from your Stormpath API Key',
-      STORMPATH_SDK_TEST_APPLICATION_URL: 'The REST URL of a Stormpath Application reserved for testing.',
-      STORMPATH_SDK_TEST_DIRECTORY_URL: 'The REST URL of a Stormpath Directory associated to the test Application.',
-      STORMPATH_SDK_TEST_DIRECTORY_WITH_VERIFICATION_URL: 'The REST URL of a Stormpath Directory configured for email verification, associated to the test Application.'
-    }
-
-    def test_api_key_id
-      ENV['STORMPATH_SDK_TEST_API_KEY_ID']
-    end
-
-    def test_api_key_secret
-      ENV['STORMPATH_SDK_TEST_API_KEY_SECRET']
-    end
-
-    def test_directory_url
-      ENV['STORMPATH_SDK_TEST_DIRECTORY_URL']
-    end
-
-    def test_directory_with_verification_url
-      ENV['STORMPATH_SDK_TEST_DIRECTORY_WITH_VERIFICATION_URL']
-    end
-
-    def test_application_url
-      ENV['STORMPATH_SDK_TEST_APPLICATION_URL']
-    end
-
-    def test_api_key
-      Stormpath::ApiKey.new test_api_key_id, test_api_key_secret
-    end
-
-    def test_api_client
-      @test_api_client ||= Stormpath::Client.new api_key: test_api_key
-    end
-
-    def get_cache_data href
-      data_store = test_api_client.send :data_store
-      data_store.send :cache_for, href
-    end
-
-    def test_missing_env_vars
-      TEST_ENV_REQUIRED_VARS.reject do |var, _message|
-        ENV[var.to_s]
-      end
-    end
-
-    def test_application
-      test_api_client.applications.get test_application_url
-    end
-
-    def test_directory
-      test_api_client.directories.get test_directory_url
-    end
-
-    def test_directory_with_verification
-      test_api_client.directories.get test_directory_with_verification_url
-    end
-
-    def fixture_path
-      File.expand_path('../fixtures/response', __FILE__)
-    end
-
-    def fixture(file)
-      File.new(fixture_path + '/' + file)
-    end
-  end
-
-  module TestResourceHelpers
-    def build_account(opts = {})
-      opts.tap do |o|
-        o[:surname]    = (!opts[:surname].blank? && opts[:surname]) || 'surname'
-        o[:given_name] = (!opts[:given_name].blank? && opts[:given_name]) || 'givenname'
-        o[:username]   = (!opts[:username].blank? && opts[:username]) || random_user_name
-        o[:password]   = (!opts[:password].blank? && opts[:password]) || 'P@$$w0rd'
-        o[:email]      = (!opts[:email].blank? && opts[:email]) || random_email
-      end
-    end
-  end
-
-  module RandomResourceNameGenerator
-    include UUIDTools
-
-    %w(application directory organization group user).each do |resource|
-      define_method "random_#{resource}_name" do |suffix = nil|
-        "#{random_string}_#{resource}_#{suffix}"
-      end
-    end
-
-    def random_name_key(suffix = 'test')
-      "#{random_string}-namekey-#{suffix}"
-    end
-
-    def random_email
-      "#{random_string}@stormpath.com"
-    end
-
-    def random_string
-      if HIJACK_HTTP_REQUESTS_WITH_VCR
-        'test'
-      else
-        UUID.method(:random_create).call.to_s[0..9]
-      end
-    end
-  end
-end
-
-RSpec::Matchers.define :be_boolean do
-  match do |actual|
-    actual == true || actual == false
-  end
-end
-
 RSpec.configure do |c|
-  c.mock_with :rspec do |c|
-    c.syntax = :expect
+  c.mock_with :rspec do |cm|
+    cm.syntax = :expect
   end
 
-  c.expect_with :rspec do |c|
-    c.syntax = :expect
+  c.expect_with :rspec do |ce|
+    ce.syntax = :expect
   end
 
-  c.include Stormpath::TestApiKeyHelpers
-  c.include Stormpath::TestResourceHelpers
-  c.include Stormpath::RandomResourceNameGenerator
-
-  c.before(:all) do
-    unless test_missing_env_vars.empty?
-      set_up_message = "In order to run the specs of the Stormpath SDK you need to setup the following environment variables:\n\t"
-      set_up_message << test_missing_env_vars.map do |var, message|
-        "#{var.to_s} : #{message}"
-      end.join("\n\t")
-      set_up_message << "\nBe sure to configure these before running the specs again."
-      raise set_up_message
-    end
-  end
+  c.include Stormpath::Test::ApiKeyHelpers
+  c.include Stormpath::Test::EnvNamesWarning
+  c.include Stormpath::Test::ResourceHelpers
 
+  Stormpath::Test::EnvNamesWarning.check_env_variable_names
 end

data/spec/support/api_key_helpers.rb

@@ -0,0 +1,34 @@
+module Stormpath
+  module Test
+    module ApiKeyHelpers
+      def test_api_key_id
+        ENV['STORMPATH_CLIENT_APIKEY_ID'] || ENV['STORMPATH_SDK_TEST_API_KEY_ID']
+      end
+
+      def test_api_key_secret
+        ENV['STORMPATH_CLIENT_APIKEY_SECRET'] || ENV['STORMPATH_SDK_TEST_API_KEY_SECRET']
+      end
+
+      def test_api_key
+        Stormpath::ApiKey.new(test_api_key_id, test_api_key_secret)
+      end
+
+      def test_api_client
+        @test_api_client ||= Stormpath::Client.new(api_key: test_api_key)
+      end
+
+      def get_cache_data(href)
+        data_store = test_api_client.send :data_store
+        data_store.send :cache_for, href
+      end
+
+      def fixture_path
+        File.expand_path('../../fixtures/response', __FILE__)
+      end
+
+      def fixture(file)
+        File.new(fixture_path + '/' + file)
+      end
+    end
+  end
+end

data/spec/support/custom_data_storage_behavior.rb

@@ -1,33 +1,22 @@
 shared_examples_for 'account_custom_data' do
   context 'account' do
-    let(:custom_data_storage) do
-      directory.accounts.create username: random_user_name,
-         email: random_email,
-         givenName: "Jean-Luc",
-         surname: "Picard",
-         password: "uGhd%a8Kl!"
-    end
-
+    let(:custom_data_storage) { directory.accounts.create(build_account) }
     let(:custom_data_storage_w_nested_custom_data) do
-      directory.accounts.create username: random_user_name,
-         email: random_email,
-         given_name: "Jean-Luc",
-         surname: "Picard",
-         password: "uGhd%a8Kl!",
-         custom_data: {
-            rank: "Captain",
-            favorite_drink: "Earl Grey Tea",
-            favoriteDrink: "Camelized Tea"
-         }
-    end
-
-    let(:reloaded_custom_data_storage) do
-      test_api_client.accounts.get custom_data_storage.href
-    end
-
-    let(:reloaded_custom_data_storage_2) do
-      test_api_client.accounts.get custom_data_storage.href
+      directory.accounts.create(
+        username: 'ruby username',
+        email: "ruby#{default_domain}",
+        given_name: 'Jean-Luc',
+        surname: 'Picard',
+        password: 'uGhd%a8Kl!',
+        custom_data: {
+          rank: 'Captain',
+          favorite_drink: 'Earl Grey Tea',
+          favoriteDrink: 'Camelized Tea'
+        }
+      )
     end
+    let(:reloaded_custom_data_storage) { test_api_client.accounts.get custom_data_storage.href }
+    let(:reloaded_custom_data_storage_2) { test_api_client.accounts.get custom_data_storage.href }
 
     it_behaves_like 'custom_data_storage'
   end
@@ -35,40 +24,33 @@ end
 
 shared_examples_for 'group_custom_data' do
   context 'group' do
-    let(:custom_data_storage) do
-      directory.groups.create name: random_group_name
-    end
-
+    let(:custom_data_storage) { directory.groups.create(build_group) }
     let(:custom_data_storage_w_nested_custom_data) do
-      directory.groups.create name: random_group_name,
-         description: "Capital Group",
-         custom_data: {
-            rank: "Captain",
-            favorite_drink: "Earl Grey Tea",
-            favoriteDrink: "Camelized Tea"
-         }
-    end
-
-    let(:reloaded_custom_data_storage) do
-      test_api_client.groups.get custom_data_storage.href
+      directory.groups.create(
+        name: 'ruby group',
+        description: 'Capital Group',
+        custom_data: {
+          rank: 'Captain',
+          favorite_drink: 'Earl Grey Tea',
+          favoriteDrink: 'Camelized Tea'
+        }
+      )
     end
 
-    let(:reloaded_custom_data_storage_2) do
-      test_api_client.groups.get custom_data_storage.href
-    end
+    let(:reloaded_custom_data_storage) { test_api_client.groups.get custom_data_storage.href }
+    let(:reloaded_custom_data_storage_2) { test_api_client.groups.get custom_data_storage.href }
 
     it_behaves_like 'custom_data_storage'
   end
 end
 
-RESERVED_FIELDS = %w( createdAt modifiedAt meta spMeta spmeta ionMeta ionmeta )
+RESERVED_FIELDS = %w(createdAt modifiedAt meta spMeta spmeta ionMeta ionmeta).freeze
 
 shared_examples_for 'custom_data_storage' do
-
   it 'read reserved data' do
-    expect(custom_data_storage.custom_data["href"]).not_to eq(nil)
-    expect(custom_data_storage.custom_data["createdAt"]).not_to eq(nil)
-    expect(custom_data_storage.custom_data["modifiedAt"]).not_to eq(nil)
+    expect(custom_data_storage.custom_data['href']).not_to eq(nil)
+    expect(custom_data_storage.custom_data['createdAt']).not_to eq(nil)
+    expect(custom_data_storage.custom_data['modifiedAt']).not_to eq(nil)
   end
 
   it 'getters for timestamps work' do
@@ -79,30 +61,30 @@ shared_examples_for 'custom_data_storage' do
   RESERVED_FIELDS.each do |reserved_field|
     it "set reserved data #{reserved_field} should raise error" do
       custom_data_storage.custom_data[reserved_field] = 12
-      expect{ custom_data_storage.custom_data.save }.to raise_error Stormpath::Error
+      expect { custom_data_storage.custom_data.save }.to raise_error Stormpath::Error
     end
   end
 
   it 'should save properly when custom data is nested on creation' do
-    expect(custom_data_storage_w_nested_custom_data.custom_data["rank"]).to eq("Captain")
-    expect(custom_data_storage_w_nested_custom_data.custom_data["favorite_drink"]).to eq("Earl Grey Tea")
-    expect(custom_data_storage_w_nested_custom_data.custom_data["favoriteDrink"]).to eq("Camelized Tea")
+    expect(custom_data_storage_w_nested_custom_data.custom_data['rank']).to eq('Captain')
+    expect(custom_data_storage_w_nested_custom_data.custom_data['favorite_drink']).to eq('Earl Grey Tea')
+    expect(custom_data_storage_w_nested_custom_data.custom_data['favoriteDrink']).to eq('Camelized Tea')
   end
 
   it 'set custom data' do
-    custom_data_storage.custom_data[:rank] = "Captain"
-    expect(custom_data_storage.custom_data[:rank]).to eq("Captain")
+    custom_data_storage.custom_data[:rank] = 'Captain'
+    expect(custom_data_storage.custom_data[:rank]).to eq('Captain')
     custom_data_storage.custom_data.save
-    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq("Captain")
+    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq('Captain')
   end
 
   it 'set nested custom data' do
-    custom_data_storage.custom_data[:special_rank] = "Captain"
-    custom_data_storage.custom_data[:permissions] = {"crew_quarters" => "93-601"}
-    expect(custom_data_storage.custom_data[:permissions]).to eq({"crew_quarters" => "93-601"})
+    custom_data_storage.custom_data[:special_rank] = 'Captain'
+    custom_data_storage.custom_data[:permissions] = { 'crew_quarters' => '93-601' }
+    expect(custom_data_storage.custom_data[:permissions]).to eq('crew_quarters' => '93-601')
     custom_data_storage.custom_data.save
-    expect(reloaded_custom_data_storage.custom_data[:special_rank]).to eq("Captain")
-    expect(reloaded_custom_data_storage.custom_data[:permissions]).to eq({"crew_quarters" => "93-601"})
+    expect(reloaded_custom_data_storage.custom_data[:special_rank]).to eq('Captain')
+    expect(reloaded_custom_data_storage.custom_data[:permissions]).to eq('crew_quarters' => '93-601')
   end
 
   it 'not raise errors when saving a empty properties array' do
@@ -110,166 +92,165 @@ shared_examples_for 'custom_data_storage' do
   end
 
   it 'trigger custom data saving on custom_data_storage.save' do
-    custom_data_storage.custom_data[:rank] = "Captain"
+    custom_data_storage.custom_data[:rank] = 'Captain'
     custom_data_storage.save
-    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq("Captain")
+    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq('Captain')
   end
 
   it 'trigger custom data saving on custom_data_storage.save with complex custom data' do
-    custom_data_storage.custom_data[:permissions] = {"crew_quarters" => "93-601"}
+    custom_data_storage.custom_data[:permissions] = { 'crew_quarters' => '93-601' }
     custom_data_storage.save
-    expect(reloaded_custom_data_storage.custom_data[:permissions]).to eq({"crew_quarters" => "93-601"})
+    expect(reloaded_custom_data_storage.custom_data[:permissions]).to eq('crew_quarters' => '93-601')
   end
 
   it 'update custom data through custom_data_storage.save, cache should be cleared' do
-    custom_data_storage.custom_data[:permissions] = {"crew_quarters" => "93-601"}
+    custom_data_storage.custom_data[:permissions] = {'crew_quarters' => '93-601'}
     custom_data_storage.custom_data.save
 
-    expect(reloaded_custom_data_storage.custom_data[:permissions]).to eq({"crew_quarters" => "93-601"})
+    expect(reloaded_custom_data_storage.custom_data[:permissions]).to eq('crew_quarters' => '93-601')
 
-    reloaded_custom_data_storage.custom_data[:permissions] = {"crew_quarters" => "601-93"}
+    reloaded_custom_data_storage.custom_data[:permissions] = { 'crew_quarters' => '601-93' }
 
     reloaded_custom_data_storage.save
-    expect(reloaded_custom_data_storage_2.custom_data[:permissions]).to eq({"crew_quarters" => "601-93"})
+    expect(reloaded_custom_data_storage_2.custom_data[:permissions]).to eq('crew_quarters' => '601-93')
   end
 
   it 'first level keys can be saved as symbols or strings, they will default to the same (saved as strings)' do
-    custom_data_storage.custom_data[:permissions] = "Drive the boat"
-    expect(custom_data_storage.custom_data[:permissions]).to eq("Drive the boat");
-    expect(custom_data_storage.custom_data["permissions"]).to eq(custom_data_storage.custom_data[:permissions])
+    custom_data_storage.custom_data[:permissions] = 'Drive the boat'
+    expect(custom_data_storage.custom_data[:permissions]).to eq('Drive the boat');
+    expect(custom_data_storage.custom_data['permissions']).to eq(custom_data_storage.custom_data[:permissions])
 
     custom_data_storage.custom_data.save
-    expect(custom_data_storage.custom_data[:permissions]).to eq("Drive the boat");
-    expect(custom_data_storage.custom_data["permissions"]).to eq(custom_data_storage.custom_data[:permissions])
+    expect(custom_data_storage.custom_data[:permissions]).to eq('Drive the boat');
+    expect(custom_data_storage.custom_data['permissions']).to eq(custom_data_storage.custom_data[:permissions])
   end
 
   it "one shouldn't save deeply nested keys as symbols, as on return from the server they will be strings" do
-    custom_data_storage.custom_data[:permissions] = { driving_privelage: "Boat", can_swim: true }
-    expect(custom_data_storage.custom_data[:permissions]).to eq({ driving_privelage: "Boat", can_swim: true })
+    custom_data_storage.custom_data[:permissions] = { driving_privelage: 'Boat', can_swim: true }
+    expect(custom_data_storage.custom_data[:permissions]).to eq(driving_privelage: 'Boat', can_swim: true)
     custom_data_storage.custom_data.save
-    expect(custom_data_storage.custom_data[:permissions]).to eq({ "driving_privelage" => "Boat", "can_swim" => true })
-    expect(custom_data_storage.custom_data[:permissions]).not_to eq({ driving_privelage: "Boat", can_swim: true })
+    expect(custom_data_storage.custom_data[:permissions]).to eq('driving_privelage' => 'Boat', 'can_swim' => true)
+    expect(custom_data_storage.custom_data[:permissions]).not_to eq(driving_privelage: 'Boat', can_swim: true)
   end
 
   it 'delete all custom data and rebind without reloading' do
-    custom_data_storage.custom_data["a_map"] = {the_key: "this is the value"}
+    custom_data_storage.custom_data['a_map'] = { the_key: 'this is the value' }
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["a_map"]).to eq({"the_key" => "this is the value"})
+    expect(custom_data_storage.custom_data['a_map']).to eq('the_key' => 'this is the value')
 
-    custom_data_storage.custom_data.delete("a_map")
+    custom_data_storage.custom_data.delete('a_map')
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["a_map"]).to be_nil
+    expect(custom_data_storage.custom_data['a_map']).to be_nil
 
-    custom_data_storage.custom_data["rank"] = "Captain"
+    custom_data_storage.custom_data['rank'] = 'Captain'
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["rank"]).to eq("Captain")
+    expect(custom_data_storage.custom_data['rank']).to eq('Captain')
 
     custom_data_storage.custom_data.delete
 
-    expect(custom_data_storage.custom_data["a_map"]).to be_nil
-    expect(custom_data_storage.custom_data["rank"]).to be_nil
+    expect(custom_data_storage.custom_data['a_map']).to be_nil
+    expect(custom_data_storage.custom_data['rank']).to be_nil
 
-    custom_data_storage.custom_data['new_stuff'] = "the value"
+    custom_data_storage.custom_data['new_stuff'] = 'the value'
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["new_stuff"]).to eq("the value")
-    custom_data_storage.custom_data.delete("new_stuff")
+    expect(custom_data_storage.custom_data['new_stuff']).to eq('the value')
+    custom_data_storage.custom_data.delete('new_stuff')
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["new_stuff"]).to be_nil
+    expect(custom_data_storage.custom_data['new_stuff']).to be_nil
   end
 
   it 'delete all custom data and rebind without reloading through custom_data_storage#save' do
-    custom_data_storage.custom_data["a_map"] = {"the_key" => "this is the value"}
+    custom_data_storage.custom_data['a_map'] = { 'the_key' => 'this is the value' }
     custom_data_storage.save
 
-    expect(custom_data_storage.custom_data["a_map"]).to eq({"the_key" => "this is the value"})
+    expect(custom_data_storage.custom_data['a_map']).to eq('the_key' => 'this is the value')
 
-    custom_data_storage.custom_data.delete("a_map")
+    custom_data_storage.custom_data.delete('a_map')
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["a_map"]).to be_nil
+    expect(custom_data_storage.custom_data['a_map']).to be_nil
 
-    custom_data_storage.custom_data["rank"] = "Captain"
+    custom_data_storage.custom_data['rank'] = 'Captain'
     custom_data_storage.save
 
-    expect(custom_data_storage.custom_data["rank"]).to eq("Captain")
+    expect(custom_data_storage.custom_data['rank']).to eq('Captain')
 
     custom_data_storage.custom_data.delete
 
-    expect(custom_data_storage.custom_data["a_map"]).to be_nil
-    expect(custom_data_storage.custom_data["rank"]).to be_nil
+    expect(custom_data_storage.custom_data['a_map']).to be_nil
+    expect(custom_data_storage.custom_data['rank']).to be_nil
 
-    custom_data_storage.custom_data['new_stuff'] = "the value"
+    custom_data_storage.custom_data['new_stuff'] = 'the value'
     custom_data_storage.save
 
-    expect(custom_data_storage.custom_data["new_stuff"]).to eq("the value")
+    expect(custom_data_storage.custom_data['new_stuff']).to eq('the value')
 
-    custom_data_storage.custom_data.delete("new_stuff")
+    custom_data_storage.custom_data.delete('new_stuff')
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["new_stuff"]).to be_nil
+    expect(custom_data_storage.custom_data['new_stuff']).to be_nil
   end
 
   it 'delete all custom data' do
-    custom_data_storage.custom_data[:rank] = "Captain"
+    custom_data_storage.custom_data[:rank] = 'Captain'
     custom_data_storage.custom_data.save
-    expect(custom_data_storage.custom_data[:rank]).to eq("Captain")
+    expect(custom_data_storage.custom_data[:rank]).to eq('Captain')
     custom_data_storage.custom_data.delete
     expect(reloaded_custom_data_storage.custom_data[:rank]).to eq(nil)
   end
 
   it 'delete all custom data and re-add new custom data' do
-    custom_data_storage.custom_data[:rank] = "Captain"
+    custom_data_storage.custom_data[:rank] = 'Captain'
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data[:rank]).to eq("Captain")
+    expect(custom_data_storage.custom_data[:rank]).to eq('Captain')
 
     custom_data_storage.custom_data.delete
 
     expect(reloaded_custom_data_storage.custom_data[:rank]).to eq(nil)
 
-    reloaded_custom_data_storage.custom_data[:rank] = "Pilot"
+    reloaded_custom_data_storage.custom_data[:rank] = 'Pilot'
 
     reloaded_custom_data_storage.custom_data.save
 
-    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq("Pilot")
+    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq('Pilot')
 
-    expect(reloaded_custom_data_storage_2.custom_data[:rank]).to eq("Pilot")
+    expect(reloaded_custom_data_storage_2.custom_data[:rank]).to eq('Pilot')
   end
 
   it "shouldn't be the same if the key is lowercased or camelcased" do
+    favorite_drink = 'Earl Grey Tea'
+    custom_data_storage.custom_data['favorite_drink'] = favorite_drink
 
-    favorite_drink = "Earl Grey Tea";
-    custom_data_storage.custom_data["favorite_drink"] = favorite_drink
-
-    expect(custom_data_storage.custom_data["favorite_drink"]).to eq(favorite_drink)
-    expect(custom_data_storage.custom_data["favoriteDrink"]).to be_nil
+    expect(custom_data_storage.custom_data['favorite_drink']).to eq(favorite_drink)
+    expect(custom_data_storage.custom_data['favoriteDrink']).to be_nil
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["favorite_drink"]).to eq(favorite_drink)
-    expect(custom_data_storage.custom_data["favoriteDrink"]).to be_nil
+    expect(custom_data_storage.custom_data['favorite_drink']).to eq(favorite_drink)
+    expect(custom_data_storage.custom_data['favoriteDrink']).to be_nil
 
-    custom_data_storage.custom_data.delete("favorite_drink")
+    custom_data_storage.custom_data.delete('favorite_drink')
 
-    expect(custom_data_storage.custom_data["favorite_drink"]).to be_nil
-    expect(custom_data_storage.custom_data["favoriteDrink"]).to be_nil
+    expect(custom_data_storage.custom_data['favorite_drink']).to be_nil
+    expect(custom_data_storage.custom_data['favoriteDrink']).to be_nil
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data["favorite_drink"]).to be_nil
-    expect(custom_data_storage.custom_data["favoriteDrink"]).to be_nil
+    expect(custom_data_storage.custom_data['favorite_drink']).to be_nil
+    expect(custom_data_storage.custom_data['favoriteDrink']).to be_nil
   end
 
   it 'delete a specific custom data field' do
-    custom_data_storage.custom_data[:rank] = "Captain"
-    custom_data_storage.custom_data["favorite_drink"] = "Earl Grey Tea"
+    custom_data_storage.custom_data[:rank] = 'Captain'
+    custom_data_storage.custom_data['favorite_drink'] = 'Earl Grey Tea'
     custom_data_storage.custom_data.save
 
     custom_data_storage.custom_data.delete(:rank)
@@ -277,79 +258,81 @@ shared_examples_for 'custom_data_storage' do
 
     expect(reloaded_custom_data_storage.custom_data[:rank]).to eq(nil)
 
-    expect(reloaded_custom_data_storage.custom_data["favorite_drink"]).to eq("Earl Grey Tea")
+    expect(reloaded_custom_data_storage.custom_data['favorite_drink']).to eq('Earl Grey Tea')
   end
 
 
   it '#has_key?' do
-    expect(custom_data_storage.custom_data.has_key? "createdAt").to be_truthy
-    expect(custom_data_storage.custom_data.has_key? "created_at").to be_falsey
+    expect(custom_data_storage.custom_data.has_key?('createdAt')).to be_truthy
+    expect(custom_data_storage.custom_data.has_key?('created_at')).to be_falsey
   end
 
   it '#include?' do
-    expect(custom_data_storage.custom_data.include? "createdAt").to be_truthy
-    expect(custom_data_storage.custom_data.include? "created_at").to be_falsey
+    expect(custom_data_storage.custom_data.include?('createdAt')).to be_truthy
+    expect(custom_data_storage.custom_data.include?('created_at')).to be_falsey
   end
 
   it '#has_value?' do
-    custom_data_storage.custom_data[:rank] = "Captain"
+    custom_data_storage.custom_data[:rank] = 'Captain'
     custom_data_storage.custom_data.save
-    expect(reloaded_custom_data_storage.custom_data.has_value? "Captain").to be_truthy
+    expect(reloaded_custom_data_storage.custom_data.has_value?('Captain')).to be_truthy
   end
 
   it '#store' do
-    custom_data_storage.custom_data.store(:rank, "Captain")
+    custom_data_storage.custom_data.store(:rank, 'Captain')
     custom_data_storage.custom_data.save
-    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq("Captain")
+    expect(reloaded_custom_data_storage.custom_data[:rank]).to eq('Captain')
   end
 
   it '#store with a snakecased key' do
-    custom_data_storage.custom_data.store(:super_rank, "Captain")
+    custom_data_storage.custom_data.store(:super_rank, 'Captain')
 
-    expect(custom_data_storage.custom_data[:super_rank]).to eq("Captain")
-    expect(custom_data_storage.custom_data["super_rank"]).to eq("Captain")
+    expect(custom_data_storage.custom_data[:super_rank]).to eq('Captain')
+    expect(custom_data_storage.custom_data['super_rank']).to eq('Captain')
 
     expect(custom_data_storage.custom_data[:superRank]).to be_nil
-    expect(custom_data_storage.custom_data["superRank"]).to be_nil
+    expect(custom_data_storage.custom_data['superRank']).to be_nil
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data[:super_rank]).to eq("Captain")
-    expect(custom_data_storage.custom_data["super_rank"]).to eq("Captain")
+    expect(custom_data_storage.custom_data[:super_rank]).to eq('Captain')
+    expect(custom_data_storage.custom_data['super_rank']).to eq('Captain')
 
     expect(custom_data_storage.custom_data[:superRank]).to be_nil
-    expect(custom_data_storage.custom_data["superRank"]).to be_nil
+    expect(custom_data_storage.custom_data['superRank']).to be_nil
   end
 
-    it '#store with a lower camelCase key' do
-    custom_data_storage.custom_data.store(:superRank, "Captain")
+  it '#store with a lower camelCase key' do
+    custom_data_storage.custom_data.store(:superRank, 'Captain')
 
-    expect(custom_data_storage.custom_data[:superRank]).to eq("Captain")
-    expect(custom_data_storage.custom_data["superRank"]).to eq("Captain")
+    expect(custom_data_storage.custom_data[:superRank]).to eq('Captain')
+    expect(custom_data_storage.custom_data['superRank']).to eq('Captain')
 
     expect(custom_data_storage.custom_data[:super_rank]).to be_nil
-    expect(custom_data_storage.custom_data["super_rank"]).to be_nil
+    expect(custom_data_storage.custom_data['super_rank']).to be_nil
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data[:superRank]).to eq("Captain")
-    expect(custom_data_storage.custom_data["superRank"]).to eq("Captain")
+    expect(custom_data_storage.custom_data[:superRank]).to eq('Captain')
+    expect(custom_data_storage.custom_data['superRank']).to eq('Captain')
 
     expect(custom_data_storage.custom_data[:super_rank]).to be_nil
-    expect(custom_data_storage.custom_data["super_rank"]).to be_nil
+    expect(custom_data_storage.custom_data['super_rank']).to be_nil
   end
 
   it '#keys' do
     expect(custom_data_storage.custom_data.keys).to be_kind_of(Array)
     expect(custom_data_storage.custom_data.keys.count).to eq(3)
-    expect(custom_data_storage.custom_data.keys).to eq(custom_data_storage.custom_data.properties.keys)
+    expect(custom_data_storage.custom_data.keys)
+      .to eq(custom_data_storage.custom_data.properties.keys)
   end
 
   it '#values' do
-    custom_data_storage.custom_data[:permissions] = {"crew_quarters" => "93-601"}
+    custom_data_storage.custom_data[:permissions] = { 'crew_quarters' => '93-601' }
     custom_data_storage.custom_data.save
-    expect(reloaded_custom_data_storage.custom_data.values).to include({"crew_quarters" => "93-601"})
-    expect(reloaded_custom_data_storage.custom_data.values).to eq(reloaded_custom_data_storage.custom_data.properties.values)
+    expect(reloaded_custom_data_storage.custom_data.values).to include('crew_quarters' => '93-601')
+    expect(reloaded_custom_data_storage.custom_data.values)
+      .to eq(reloaded_custom_data_storage.custom_data.properties.values)
   end
 
   it 'inner property holders clearing properly' do
@@ -359,7 +342,7 @@ shared_examples_for 'custom_data_storage' do
 
     custom_data_storage.custom_data.save
 
-    expect(custom_data_storage.custom_data[:permissions]).to eq("NOOP")
+    expect(custom_data_storage.custom_data[:permissions]).to eq('NOOP')
     custom_data_storage.custom_data.delete(:permissions)
     expect(custom_data_storage.custom_data[:permissions]).to be_nil
 
@@ -371,7 +354,7 @@ shared_examples_for 'custom_data_storage' do
     expect(deleted_properties.count).to eq(0)
 
     custom_data_storage.custom_data[:permissions] = 'NOOP'
-    expect(custom_data_storage.custom_data[:permissions]).to eq("NOOP")
+    expect(custom_data_storage.custom_data[:permissions]).to eq('NOOP')
 
     custom_data_storage.custom_data.delete(:permissions)
     expect(custom_data_storage.custom_data[:permissions]).to be_nil
@@ -379,9 +362,9 @@ shared_examples_for 'custom_data_storage' do
     expect(deleted_properties.count).to eq(1)
 
     if custom_data_storage.is_a? Stormpath::Resource::Account
-      custom_data_storage.given_name = "Capt"
+      custom_data_storage.given_name = 'Capt'
     else
-      custom_data_storage.name = random_group_name
+      custom_data_storage.name = 'random_group_name'
     end
 
     custom_data_storage.save
@@ -391,6 +374,6 @@ shared_examples_for 'custom_data_storage' do
   end
 
   def deleted_properties
-    custom_data_storage.custom_data.instance_variable_get("@deleted_properties")
+    custom_data_storage.custom_data.instance_variable_get('@deleted_properties')
   end
 end