stormpath-rails 0.3.4 → 0.4.0

data/.travis.yml

@@ -4,3 +4,5 @@ rvm:
 gemfile:
   - Gemfile
 script: bundle exec rake spec
+services:
+  - mongodb

data/Guardfile

@@ -0,0 +1,23 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+guard 'rspec' do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/#{m[1]}_spec.rb" }
+  watch('spec/spec_helper.rb')  { "spec" }
+
+  # Rails example
+  watch(%r{^app/(.+)\.rb$})                           { |m| "spec/#{m[1]}_spec.rb" }
+  watch(%r{^app/(.*)(\.erb|\.haml)$})                 { |m| "spec/#{m[1]}#{m[2]}_spec.rb" }
+  watch(%r{^app/controllers/(.+)_(controller)\.rb$})  { |m| ["spec/routing/#{m[1]}_routing_spec.rb", "spec/#{m[2]}s/#{m[1]}_#{m[2]}_spec.rb", "spec/acceptance/#{m[1]}_spec.rb"] }
+  watch(%r{^spec/support/(.+)\.rb$})                  { "spec" }
+  watch('config/routes.rb')                           { "spec/routing" }
+  watch('app/controllers/application_controller.rb')  { "spec/controllers" }
+
+  # Capybara features specs
+  watch(%r{^app/views/(.+)/.*\.(erb|haml)$})          { |m| "spec/features/#{m[1]}_spec.rb" }
+
+  # Turnip features and steps
+  watch(%r{^spec/acceptance/(.+)\.feature$})
+  watch(%r{^spec/acceptance/steps/(.+)_steps\.rb$})   { |m| Dir[File.join("**/#{m[1]}.feature")][0] || 'spec/acceptance' }
+end

data/lib/stormpath/rails/account.rb

@@ -11,24 +11,29 @@ module Stormpath
       STORMPATH_FIELDS = [ :email, :password, :username, :given_name, :middle_name, :surname ]
 
       included do
-        self.partial_updates = false
+        #AR specific workaround
+        self.partial_updates = false if self.respond_to?(:partial_updates)
+
+        #Mongoid specific declaration
+        #TODO index?
+        field(:stormpath_url, type: String) if self.respond_to?(:field)
 
         attr_accessor *STORMPATH_FIELDS
         attr_accessible  *STORMPATH_FIELDS
 
         after_initialize do |user|
-          return unless user.stormpath_url
+          return true unless user.stormpath_url
           begin
             account = Client.find_account(user.stormpath_url)
             (STORMPATH_FIELDS - [:password]).each { |field| self.send("#{field}=", account.send("get_#{field}")) }
           rescue ResourceError => error
-            #somehow mark as data not loaded
+            Logger.new(STDERR).warn "Error loading Stormpath account (#{error})"
           end
         end
 
         before_create do
           begin
-            account = Client.create_account!(Hash[*STORMPATH_FIELDS.map { |f| { f => self.send(f) } }.map(&:to_a).flatten])
+            account = Stormpath::Rails::Client.create_account!(Hash[*STORMPATH_FIELDS.map { |f| { f => self.send(f) } }.map(&:to_a).flatten])
           rescue ResourceError => error
             self.errors[:base] << error.to_s
             return false
@@ -37,26 +42,24 @@ module Stormpath
         end
 
         before_update do
+          return true unless self.stormpath_url
           begin
-            account = Client.update_account!(self.stormpath_url, Hash[*STORMPATH_FIELDS.map { |f| { f => self.send(f) } }.map(&:to_a).flatten])
+            Client.update_account!(self.stormpath_url, Hash[*STORMPATH_FIELDS.map { |f| { f => self.send(f) } }.map(&:to_a).flatten])
           rescue ResourceError => error
             self.errors[:base] << error.to_s
-            return nil
+            return false
           end
         end
 
         after_destroy do
+          return true unless self.stormpath_url
           begin
             account = Client.delete_account!(self.stormpath_url)
           rescue ResourceError => error
-            self.errors[:base] << error.to_s
-            return false
+            Logger.new(STDERR).warn "Error destroying Stormpath account (#{error})"
           end
         end
       end
-
-      module ClassMethods
-      end
     end
   end
 end

data/lib/stormpath/rails/client.rb

@@ -6,7 +6,9 @@ include Stormpath::Authentication
 module Stormpath
   module Rails
     class Client
-      cattr_accessor :connection
+      class << self
+        attr_accessor :connection
+      end
 
       def self.authenticate_account(login, password)
         application = self.ds.get_resource Config[:application], ::Application
@@ -14,6 +16,16 @@ module Stormpath
         auth_result.get_account
       end
 
+      def self.send_password_reset_email(login_or_email)
+        application = self.ds.get_resource Config[:application], ::Application
+        application.send_password_reset_email login_or_email
+      end
+
+      def self.verify_password_reset_token(token)
+        application = self.ds.get_resource Config[:application], ::Application
+        application.verify_password_reset_token token
+      end
+
       def self.create_account!(attributes)
         account = self.ds.instantiate ::Account
         attributes.each { |field, value| account.send("set_#{field}", value) }

data/lib/stormpath/rails/config.rb

@@ -0,0 +1,14 @@
+module Stormpath
+  module Rails
+    class Config
+      class << self
+        attr_accessor :vars
+      end
+
+      def self.[](name)
+        self.vars ||= YAML.load(ERB.new(File.read("#{::Rails.root}/config/stormpath.yml")).result)
+        self.vars["common"].update(self.vars[::Rails.env])[name.to_s]
+      end
+    end
+  end
+end

data/lib/stormpath/rails/version.rb

@@ -1,5 +1,5 @@
 module Stormpath
   module Rails
-    VERSION = "0.3.4"
+    VERSION = "0.4.0"
   end
 end

data/lib/stormpath-rails.rb

@@ -1,16 +1 @@
-require "stormpath/rails/version"
-require "stormpath/rails/client"
-require "stormpath/rails/account"
-
-module Stormpath
-  module Rails
-    class Config
-      cattr_accessor :vars
-
-      def self.[](name)
-        self.vars ||= YAML.load(ERB.new(File.read("#{::Rails.root}/config/stormpath.yml")).result)
-        self.vars["common"].update(self.vars[::Rails.env])[name.to_s]
-      end
-    end
-  end
-end
+Dir[File.join(File.dirname(__FILE__), "stormpath/**/*.rb")].sort.each {|f| require f}

data/spec/fixtures/config/stormpath.yml

@@ -0,0 +1,12 @@
+common:
+  href: stormpath_url
+  application: application_url
+
+development:
+  root: development_root
+
+test:
+  root: test_root
+
+production:
+  root: production_root

data/spec/{lib/generators → generators}/stormpath/rails/install/install_generator_spec.rb

@@ -1,16 +1,7 @@
 require "spec_helper"
-
-module Rails
-  module Generators
-    class Base
-    end
-  end
-end
-
 require "generators/stormpath/rails/install/install_generator"
 
 describe Stormpath::Rails::Generators::InstallGenerator do
-
   it "creates a configuration file" do
     subject.should_receive(:create_file).with("config/stormpath.yml", kind_of(String))
     subject.create_initializer_file

data/spec/integration/active_record_spec.rb

@@ -0,0 +1,23 @@
+require "spec_helper"
+require "active_record"
+require "stormpath-rails"
+
+describe "ActiveRecord record" do
+  class ArEntity < ActiveRecord::Base
+    include Stormpath::Rails::Account
+  end
+
+  subject { ArEntity.new }
+
+  before(:each) do
+    ActiveRecord::Base.establish_connection :adapter => 'sqlite3', :database => ':memory:'
+    ActiveRecord::Migration.verbose = false
+    ActiveRecord::Migration.create_table :ar_entities do |t|
+      t.string :id
+      t.string :stormpath_url
+    end
+  end
+
+  it_should_behave_like "stormpath account"
+
+end

data/spec/integration/mongoid_spec.rb

@@ -0,0 +1,19 @@
+require "spec_helper"
+require 'mongoid'
+require "stormpath-rails"
+
+describe "Mongoid document" do
+  class MongoidEntity
+    include Mongoid::Document
+    include Stormpath::Rails::Account
+  end
+
+  subject { MongoidEntity.new }
+
+  before(:each) do
+    Mongoid::Config.connect_to("stormpath_rails_test")
+  end
+
+  it_should_behave_like "stormpath account"
+
+end

data/spec/spec_helper.rb

@@ -1,17 +1,5 @@
-# This file was generated by the `rspec --init` command. Conventionally, all
-# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
-# Require this file using `require "spec_helper"` to ensure that it is only
-# loaded once.
-#
-# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
-RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
-  config.run_all_when_everything_filtered = true
-  config.filter_run :focus
+Dir["./spec/support/**/*.rb"].sort.each {|f| require f}
 
-  # Run specs in random order to surface order dependencies. If you find an
-  # order dependency and want to debug it, you can fix the order by providing
-  # the seed, which is printed after each run.
-  #     --seed 1234
+RSpec.configure do |config|
   config.order = 'random'
 end

data/spec/stormpath/rails/config_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+require 'stormpath/rails/config'
+
+describe Stormpath::Rails::Config do
+  before(:each) do
+    Rails.stub!(:root).and_return(File.join File.dirname(__FILE__), "..", "..", "fixtures")
+    Rails.stub!(:env).and_return("test")
+  end
+
+  context "file IO" do
+    let(:path) { "#{Rails.root}/config/stormpath.yml" }
+
+    before(:each) do
+      described_class.vars = nil
+    end
+
+    it "should read yaml from rails app config/stormpath.yml" do
+      File.should_receive(:read).with(path).and_call_original
+      described_class[:href].should == 'stormpath_url'
+    end
+
+    it "should not re-red yaml once loaded" do
+      File.should_receive(:read).with(path).and_call_original
+      described_class[:href].should == 'stormpath_url'
+      described_class[:href].should == 'stormpath_url'
+    end
+  end
+
+  %w{test development production}.each do |environment|
+    it "should read root directory from #{environment} yaml section" do
+      Rails.stub!(:env).and_return(environment)
+      described_class[:root].should == "#{environment}_root"
+    end
+
+    it "should mix shared vars to #{environment} env" do
+      Rails.stub!(:env).and_return(environment)
+      described_class[:href].should == 'stormpath_url'
+      described_class[:application].should == 'application_url'
+    end
+  end
+end

data/spec/support/rails.rb

@@ -0,0 +1,6 @@
+module Rails
+  module Generators
+    class Base
+    end
+  end
+end

data/spec/support/stormpath_account_shared_examples.rb

@@ -0,0 +1,120 @@
+shared_examples "stormpath account" do
+  context "after initialize" do
+    let(:account) { mock("account", get_href: "account_href") }
+
+    let!(:logger) { Logger.new(STDERR) }
+
+    before(:each) do
+      subject.class::STORMPATH_FIELDS.each do |field_name|
+        account.stub!("get_#{field_name}").and_return(field_name.to_s)
+      end
+      Logger.stub!(:new).and_return(logger)
+      Stormpath::Rails::Client.stub!(:find_account).and_return(account)
+    end
+
+    it "should silently skip stormpath load if no stormpath_url set" do
+      Stormpath::Rails::Client.should_not_receive(:find_account)
+      expect { subject }.to_not raise_error
+    end
+
+    context "on find" do
+      before(:each) do
+        Stormpath::Rails::Client.stub!(:create_account!).and_return(account)
+        subject.save!
+      end
+
+      it "should find account at stormpath" do
+        Stormpath::Rails::Client.should_receive(:find_account).with(subject.stormpath_url)
+        subject.class.all.first.stormpath_url.should == subject.stormpath_url
+      end
+
+      it "should setup object with data from stormpath" do
+        Stormpath::Rails::Client.should_receive(:find_account).with(subject.stormpath_url).and_return(account)
+        found = subject.class.where(stormpath_url: 'account_href').first
+        (subject.class::STORMPATH_FIELDS - [:password]).each do |field_name|
+          found.send(field_name).should == account.send("get_#{field_name}")
+        end
+      end
+
+      it "should log warning if stormpath account update failed" do
+        Stormpath::Rails::Client.stub!(:find_account).and_raise(ResourceError.new(mock("error", get_message: "Find failed")))
+        logger.should_receive(:warn).with("Error loading Stormpath account (Find failed)")
+        found = subject.class.where(stormpath_url: 'account_href').first
+        subject.class::STORMPATH_FIELDS.each do |field_name|
+          found.send(field_name).should be_nil
+        end
+      end
+    end
+
+  end
+
+
+  context "before create" do
+    it "should create account at stormpath and assign stormpath_url" do
+      Stormpath::Rails::Client.should_receive(:create_account!).and_return(mock("account", get_href: "account_href"))
+      subject.save!
+      subject.stormpath_url.should == "account_href"
+    end
+
+    it "should add error if stormpath account creation failed" do
+      Stormpath::Rails::Client.stub!(:create_account!).and_raise(ResourceError.new(mock("error", get_message: "Create failed")))
+      subject.save
+      subject.errors[:base].should == ["Create failed"]
+    end
+  end
+
+  context "before update" do
+    let(:account) { mock("account", get_href: "account_href") }
+
+    before(:each) do
+      Stormpath::Rails::Client.stub!(:create_account!).and_return(account)
+      subject.save!
+    end
+
+    it "should silently skip stormpath update if no stormpath_url set" do
+      subject.stormpath_url = nil
+      Stormpath::Rails::Client.should_not_receive(:update_account!)
+      expect { subject.save! }.to_not raise_error
+    end
+
+    it "should update account at stormpath" do
+      Stormpath::Rails::Client.should_receive(:update_account!).with(subject.stormpath_url, anything())
+      subject.save!
+    end
+
+    it "should add error if stormpath account update failed" do
+      Stormpath::Rails::Client.stub!(:update_account!).and_raise(ResourceError.new(mock("error", get_message: "Update failed")))
+      subject.save.should be_false
+      subject.errors[:base].should == ["Update failed"]
+    end
+  end
+
+  context "after destroy" do
+    let(:account) { mock("account", get_href: "account_href") }
+
+    let!(:logger) { Logger.new(STDERR) }
+
+    before(:each) do
+      Logger.stub!(:new).and_return(logger)
+      Stormpath::Rails::Client.stub!(:create_account!).and_return(account)
+      subject.save!
+    end
+
+    it "should silently skip stormpath delete if no stormpath_url set" do
+      subject.stormpath_url = nil
+      Stormpath::Rails::Client.should_not_receive(:delete_account!)
+      expect { subject.destroy }.to_not raise_error
+    end
+
+    it "should destroy account at stormpath" do
+      Stormpath::Rails::Client.should_receive(:delete_account!).with(subject.stormpath_url)
+      subject.destroy
+    end
+
+    it "should log warning if stormpath account update failed" do
+      Stormpath::Rails::Client.stub!(:delete_account!).and_raise(ResourceError.new(mock("error", get_message: "Delete failed")))
+      logger.should_receive(:warn).with("Error destroying Stormpath account (Delete failed)")
+      subject.destroy.should be_true
+    end
+  end
+end

data/stormpath-rails.gemspec

@@ -20,6 +20,10 @@ Gem::Specification.new do |gem|
 
   gem.add_development_dependency('rake', '~> 10.0.2')
   gem.add_development_dependency('rspec', '~> 2.12.0')
+  gem.add_development_dependency('activerecord')
+  gem.add_development_dependency('sqlite3')
+  gem.add_development_dependency('mongoid')
+  gem.add_development_dependency('railties')
   gem.add_development_dependency('guard-rspec', '~> 2.2.1')
   gem.add_development_dependency('rb-inotify', '~> 0.8.8')
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stormpath-rails
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.4.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-14 00:00:00.000000000 Z
+date: 2013-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -75,6 +75,70 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.12.0
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: mongoid
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: railties
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -118,6 +182,7 @@ files:
 - .rspec
 - .travis.yml
 - Gemfile
+- Guardfile
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -125,9 +190,16 @@ files:
 - lib/stormpath-rails.rb
 - lib/stormpath/rails/account.rb
 - lib/stormpath/rails/client.rb
+- lib/stormpath/rails/config.rb
 - lib/stormpath/rails/version.rb
-- spec/lib/generators/stormpath/rails/install/install_generator_spec.rb
+- spec/fixtures/config/stormpath.yml
+- spec/generators/stormpath/rails/install/install_generator_spec.rb
+- spec/integration/active_record_spec.rb
+- spec/integration/mongoid_spec.rb
 - spec/spec_helper.rb
+- spec/stormpath/rails/config_spec.rb
+- spec/support/rails.rb
+- spec/support/stormpath_account_shared_examples.rb
 - stormpath-rails.gemspec
 homepage: http://stormpath.com
 licenses: []
@@ -154,5 +226,11 @@ signing_key:
 specification_version: 3
 summary: Stormpath SDK API Wrapper
 test_files:
-- spec/lib/generators/stormpath/rails/install/install_generator_spec.rb
+- spec/fixtures/config/stormpath.yml
+- spec/generators/stormpath/rails/install/install_generator_spec.rb
+- spec/integration/active_record_spec.rb
+- spec/integration/mongoid_spec.rb
 - spec/spec_helper.rb
+- spec/stormpath/rails/config_spec.rb
+- spec/support/rails.rb
+- spec/support/stormpath_account_shared_examples.rb