accounts_client 0.0.3 → 0.0.4

data/lib/accounts/belongs_to_account.rb

@@ -1,4 +1,8 @@
 # this module assumes base class has an account_name attribute.
+# and thus responds to:
+# - account_name
+# - account_name=
+# - account_name_changed?
 module Accounts
   module BelongsToAccount
 
@@ -34,6 +38,7 @@ module Accounts
     # @raises 'This is the wrong account!'
     # @raises 'This is not a account!'
     def padma_account_setted_correctly
+      # refresh_cached_account_if_needed
       return if self.padma_account.nil?
       unless padma_account.is_a?(PadmaAccount)
         raise 'This is not a account!'
@@ -47,6 +52,13 @@ module Accounts
         end
       end
     end
+
+    def refresh_cached_account_if_needed
+      if self.account_name_changed?
+        self.account(force_service_call: true)
+      end
+    end
+
   end
 end
 

data/lib/accounts/is_a_user.rb

@@ -1,3 +1,9 @@
+# This module will include BelongsToUser and BelongsToAccount
+# It requires base class to respond to:
+# - username
+# - account_name
+# - account_name=
+# - account_name_changed?
 module Accounts
   module IsAUser
 
@@ -9,6 +15,8 @@ module Accounts
       base.send(:delegate, :verbose_help?, to: :padma_user)
     end
 
+    # @param [Boolean] cache
+    # @return [PadmaUser]
     def padma(cache=true)
       self.user(force_service_call: !cache)
     end
@@ -39,13 +47,13 @@ module Accounts
     # Validates that current_account_name
     # checking that user belongs to such account
     def has_access_to_current_account
-      return if self.current_account.nil? || !self.current_account_id_changed?
+      return if self.padma.try(:current_account).nil?
 
       if self.enabled_accounts.nil?
         # nil means error in connection
         self.errors.add(:current_account_id, I18n.t('user.couldnt_connect_to_check_current_account'))
       else
-        unless self.current_account.name.in?(self.enabled_accounts.map{|ea|ea.name})
+        unless self.padma.current_account_name.in?(self.enabled_accounts.map{|ea|ea.name})
           self.errors.add(:current_account_id, I18n.t('user.invalid_current_account'))
         end
       end

data/lib/accounts/shared_examples.rb

@@ -0,0 +1,16 @@
+shared_examples_for "it belongs to an account"  do
+  it { should validate_presence_of :account_name }
+  it "should return account on #account" do
+    PadmaAccount.should_receive(:find).with(object.account_name).and_return(PadmaAccount.new(account_name: object.account_name))
+    object.account.should be_a(PadmaAccount)
+  end
+end
+
+shared_examples_for "it belongs to a user"  do
+  it { should validate_presence_of :username}
+  it "should return user on #user" do
+    PadmaUser.stub(:find).with(object.username).and_return(PadmaUser.new(username: object.username))
+    object.user.should be_a(PadmaUser)
+  end
+  it "should use Rails.cache"
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: accounts_client
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-03 00:00:00.000000000 Z
+date: 2012-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -53,6 +53,7 @@ files:
 - app/models/padma_user.rb
 - app/models/padma_account.rb
 - lib/accounts/belongs_to_account.rb
+- lib/accounts/shared_examples.rb
 - lib/accounts/belongs_to_user.rb
 - lib/accounts/is_a_user.rb
 - lib/accounts/railties.rb