openstax_accounts 4.1.1 → 5.0.0

data/spec/lib/openstax/accounts/api_spec.rb

@@ -0,0 +1,232 @@
+module OpenStax
+  module Accounts
+    describe Api do
+
+      let!(:account) { OpenStax::Accounts::Account.create(username: 'some_user',
+                         first_name: 'Some', last_name: 'User', full_name: 'SomeUser',
+                         title: 'Sir', openstax_uid: 1, access_token: 'secret') }
+
+      it 'makes api requests' do
+        expect(::Api::DummyController.last_action).to be_nil
+        expect(::Api::DummyController.last_params).to be_nil
+        Api.request(:post, 'dummy', :params => {:test => true})
+        expect(::Api::DummyController.last_action).to eq :dummy
+        expect(::Api::DummyController.last_params).to include 'test' => 'true'
+      end
+
+      context 'users' do
+
+        let!(:account) { FactoryGirl.create :openstax_accounts_account }
+
+        it 'makes api call to users index' do
+          ::Api::UsersController.last_action = nil
+          ::Api::UsersController.last_params = nil
+          Api.search_accounts('something')
+          expect(::Api::UsersController.last_action).to eq :index
+          expect(::Api::UsersController.last_params).to include :q => 'something'
+        end
+
+        it 'makes api call to user update' do
+          ::Api::UsersController.last_action = nil
+          ::Api::UsersController.last_json = nil
+          Api.update_account(account)
+          expect(::Api::UsersController.last_action).to eq :update
+          expect(::Api::UsersController.last_json).to include(
+            account.attributes.slice('username', 'first_name',
+              'last_name', 'full_name', 'title'))
+        end
+
+        it 'makes api call to (temp) user create by email' do
+          ::Api::UsersController.last_action = nil
+          ::Api::UsersController.last_json = nil
+          Api.create_temp_account(email: 'dummy@dum.my')
+          expect(::Api::UsersController.last_action).to eq :create
+          expect(::Api::UsersController.last_json).to(
+            include('email' => 'dummy@dum.my')
+          )
+        end
+
+        it 'makes api call to (temp) user create by username' do
+          ::Api::UsersController.last_action = nil
+          ::Api::UsersController.last_json = nil
+          Api.create_temp_account(username: 'dummy')
+          expect(::Api::UsersController.last_action).to eq :create
+          expect(::Api::UsersController.last_json).to(
+            include('username' => 'dummy')
+          )
+        end
+
+      end
+
+      context 'application_users' do
+
+        it 'makes api call to application_users index' do
+          ::Api::ApplicationUsersController.last_action = nil
+          ::Api::ApplicationUsersController.last_params = nil
+          Api.search_application_accounts('something')
+          expect(::Api::ApplicationUsersController.last_action).to eq :index
+          expect(::Api::ApplicationUsersController.last_params).to include :q => 'something'
+        end
+
+        it 'makes api call to application_users updates' do
+          ::Api::ApplicationUsersController.last_action = nil
+          Api.get_application_account_updates
+          expect(::Api::ApplicationUsersController.last_action).to eq :updates
+        end
+
+        it 'makes api call to application_users updated' do
+          ::Api::ApplicationUsersController.last_action = nil
+          ::Api::ApplicationUsersController.last_json = nil
+          Api.mark_account_updates_as_read([{id: 1, read_updates: 1}])
+          expect(::Api::ApplicationUsersController.last_action).to eq :updated
+          expect(::Api::ApplicationUsersController.last_json).to include(
+            {'id' => 1, 'read_updates' => 1})
+        end
+
+      end
+
+      context 'groups' do
+
+        let!(:account) { FactoryGirl.create :openstax_accounts_account }
+        let!(:group)   { FactoryGirl.create :openstax_accounts_group }
+
+        it 'makes api call to groups create' do
+          ::Api::GroupsController.last_action = nil
+          ::Api::GroupsController.last_json = nil
+          Api.create_group(account, group)
+          expect(::Api::GroupsController.last_action).to eq :create
+          expect(::Api::GroupsController.last_json).to include(
+            {'name' => 'MyGroup', 'is_public' => false})
+        end
+
+        it 'makes api call to group update' do
+          group.save!
+          ::Api::GroupsController.last_action = nil
+          ::Api::GroupsController.last_params = nil
+          ::Api::GroupsController.last_json = nil
+          Api.update_group(account, group)
+          expect(::Api::GroupsController.last_action).to eq :update
+          expect(::Api::GroupsController.last_params).to include(
+            {'id' => group.openstax_uid.to_s})
+          expect(::Api::GroupsController.last_json).to include(
+            {'name' => group.name, 'is_public' => group.is_public})
+        end
+
+        it 'makes api call to group destroy' do
+          group.save!
+          ::Api::GroupsController.last_action = nil
+          ::Api::GroupsController.last_params = nil
+          Api.destroy_group(account, group)
+          expect(::Api::GroupsController.last_action).to eq :destroy
+          expect(::Api::GroupsController.last_params).to include(
+            {'id' => group.openstax_uid.to_s})
+        end
+
+      end
+
+      context 'group_members' do
+
+        let!(:account) { FactoryGirl.create :openstax_accounts_account }
+        let!(:group_member)   { FactoryGirl.build :openstax_accounts_group_member }
+
+        it 'makes api call to group_members create' do
+          ::Api::GroupMembersController.last_action = nil
+          ::Api::GroupMembersController.last_params = nil
+          Api.create_group_member(account, group_member)
+          expect(::Api::GroupMembersController.last_action).to eq :create
+          expect(::Api::GroupMembersController.last_params).to include(
+            {'group_id' => group_member.group_id.to_s,
+             'user_id' => group_member.user_id.to_s})
+        end
+
+        it 'makes api call to group_member destroy' do
+          group_member.save!
+          ::Api::GroupMembersController.last_action = nil
+          ::Api::GroupMembersController.last_params = nil
+          Api.destroy_group_member(account, group_member)
+          expect(::Api::GroupMembersController.last_action).to eq :destroy
+          expect(::Api::GroupMembersController.last_params).to include(
+            {'group_id' => group_member.group_id.to_s,
+             'user_id' => group_member.user_id.to_s})
+        end
+
+      end
+
+      context 'group_owners' do
+
+        let!(:account)      { FactoryGirl.create :openstax_accounts_account }
+        let!(:group_owner) { FactoryGirl.build :openstax_accounts_group_owner }
+
+        it 'makes api call to group_owners create' do
+          ::Api::GroupOwnersController.last_action = nil
+          ::Api::GroupOwnersController.last_params = nil
+          Api.create_group_owner(account, group_owner)
+          expect(::Api::GroupOwnersController.last_action).to eq :create
+          expect(::Api::GroupOwnersController.last_params).to include(
+            {'group_id' => group_owner.group_id.to_s,
+             'user_id' => group_owner.user_id.to_s})
+        end
+
+        it 'makes api call to group_owner destroy' do
+          group_owner.save!
+          ::Api::GroupOwnersController.last_action = nil
+          ::Api::GroupOwnersController.last_params = nil
+          Api.destroy_group_owner(account, group_owner)
+          expect(::Api::GroupOwnersController.last_action).to eq :destroy
+          expect(::Api::GroupOwnersController.last_params).to include(
+            {'group_id' => group_owner.group_id.to_s,
+             'user_id' => group_owner.user_id.to_s})
+        end
+
+      end
+
+      context 'group_nestings' do
+
+        let!(:account)      { FactoryGirl.create :openstax_accounts_account }
+        let!(:group_nesting) { FactoryGirl.build :openstax_accounts_group_nesting }
+
+        it 'makes api call to group_nestings (create)' do
+          ::Api::GroupNestingsController.last_action = nil
+          ::Api::GroupNestingsController.last_params = nil
+          Api.create_group_nesting(account, group_nesting)
+          expect(::Api::GroupNestingsController.last_action).to eq :create
+          expect(::Api::GroupNestingsController.last_params).to include(
+            {'group_id' => group_nesting.container_group_id.to_s,
+             'member_group_id' => group_nesting.member_group_id.to_s})
+        end
+
+        it 'makes api call to group_nesting (destroy)' do
+          group_nesting.save!
+          ::Api::GroupNestingsController.last_action = nil
+          ::Api::GroupNestingsController.last_params = nil
+          Api.destroy_group_nesting(account, group_nesting)
+          expect(::Api::GroupNestingsController.last_action).to eq :destroy
+          expect(::Api::GroupNestingsController.last_params).to include(
+            {'group_id' => group_nesting.container_group_id.to_s,
+             'member_group_id' => group_nesting.member_group_id.to_s})
+        end
+
+      end
+
+      context 'application_groups' do
+
+        it 'makes api call to application_groups_updates' do
+          ::Api::ApplicationGroupsController.last_action = nil
+          Api.get_application_group_updates
+          expect(::Api::ApplicationGroupsController.last_action).to eq :updates
+        end
+
+        it 'makes api call to application_groups_updated' do
+          ::Api::ApplicationGroupsController.last_action = nil
+          ::Api::ApplicationGroupsController.last_json = nil
+          Api.mark_group_updates_as_read([{id: 1, read_updates: 1}])
+          expect(::Api::ApplicationGroupsController.last_action).to eq :updated
+          expect(::Api::ApplicationGroupsController.last_json).to include(
+            {'id' => 1, 'read_updates' => 1})
+        end
+
+      end
+
+    end
+  end
+end

data/spec/lib/openstax/accounts/has_many_through_groups/active_record/base_spec.rb

@@ -0,0 +1,57 @@
+module OpenStax
+  module Accounts
+    module HasManyThroughGroups
+      module ActiveRecord
+        describe Base do
+          let!(:account_1) { FactoryGirl.create(:openstax_accounts_account,
+                             username: 'some_user',
+                             openstax_uid: 1) }
+          let!(:user_1)    { User.create(:account => account_1) }
+
+          let!(:account_2) { FactoryGirl.create(:openstax_accounts_account,
+                             username: 'another_user',
+                             openstax_uid: 2) }
+          let!(:user_2)    { User.create(:account => account_2) }
+
+          let!(:group_nesting) { FactoryGirl.create(:openstax_accounts_group_nesting) }
+
+          before(:each) do
+            group_nesting.member_group.add_member(account_1)
+          end
+
+          it 'allows users to retrieve all nested has_many_through_groups objects' do
+            expect(user_1.ownerships).to be_empty
+
+            o = Ownership.new
+            o.owner = user_2
+            o.save!
+
+            expect(user_1.reload.ownerships).to be_empty
+
+            o2 = Ownership.new
+            o2.owner = user_1
+            o2.save!
+
+            expect(user_1.reload.ownerships).to include(o2)
+
+            o3 = Ownership.new
+            o3.owner = group_nesting.member_group
+            o3.save!
+
+            expect(user_1.reload.ownerships).to include(o2)
+            expect(user_1.ownerships).to include(o3)
+
+            o4 = Ownership.new
+            o4.owner = group_nesting.container_group
+            o4.save!
+
+            expect(user_1.reload.ownerships).to include(o2)
+            expect(user_1.ownerships).to include(o3)
+            expect(user_1.ownerships).to include(o4)
+          end
+
+        end
+      end
+    end
+  end
+end

data/spec/routines/openstax/accounts/sync_accounts_spec.rb

@@ -17,15 +17,11 @@ module OpenStax
                                         unread_updates: 2, default_contact_info_id: 5}]
           end)
 
-        begin
-          OpenStax::Accounts.syncing = true
-          account = OpenStax::Accounts::Account.new
-          account.username = 'u'
-          account.openstax_uid = 2
-          account.save!
-        ensure
-          OpenStax::Accounts.syncing = false
-        end
+        account = OpenStax::Accounts::Account.new
+        account.username = 'u'
+        account.openstax_uid = 2
+        account.syncing = true
+        account.save!
 
         begin
           OpenStax::Accounts.configuration.enable_stubbing = false

data/spec/routines/openstax/accounts/sync_groups_spec.rb

@@ -24,45 +24,44 @@ module OpenStax
                                          unread_updates: 2, default_contact_info_id: 5}]
           end)
 
-        begin
-          OpenStax::Accounts.syncing = true
-          u = OpenStax::Accounts::Account.new
-          u.openstax_uid = 2
-          u.username = 'User'
-          u.save!
-
-          u2 = OpenStax::Accounts::Account.new
-          u2.openstax_uid = 3
-          u2.username = 'Fuego'
-          u2.save!
-
-          g = OpenStax::Accounts::Group.new
-          g.name = 'Member Group'
-          g.openstax_uid = 2
-          g.save!
-
-          gm = GroupMember.new
-          gm.group = g
-          gm.user = u
-          gm.save!
-
-          g2 = OpenStax::Accounts::Group.new
-          g2.name = 'Container Group'
-          g2.openstax_uid = 4
-          g2.save!
-
-          go = GroupOwner.new
-          go.group = g2
-          go.user = u
-          go.save!
-
-          gn = GroupNesting.new
-          gn.container_group = g2
-          gn.member_group = g
-          gn.save!
-        ensure
-          OpenStax::Accounts.syncing = false
-        end
+        u = OpenStax::Accounts::Account.new
+        u.openstax_uid = 2
+        u.username = 'User'
+        u.syncing = true
+        u.save!
+
+        u2 = OpenStax::Accounts::Account.new
+        u2.openstax_uid = 3
+        u2.username = 'Fuego'
+        u2.syncing = true
+        u2.save!
+
+        g = OpenStax::Accounts::Group.new
+        g.name = 'Member Group'
+        g.openstax_uid = 2
+        g.syncing = true
+        g.save!
+
+        gm = GroupMember.new
+        gm.group = g
+        gm.user = u
+        gm.save!
+
+        g2 = OpenStax::Accounts::Group.new
+        g2.name = 'Container Group'
+        g2.openstax_uid = 4
+        g2.syncing = true
+        g2.save!
+
+        go = GroupOwner.new
+        go.group = g2
+        go.user = u
+        go.save!
+
+        gn = GroupNesting.new
+        gn.container_group = g2
+        gn.member_group = g
+        gn.save!
 
         begin
           OpenStax::Accounts.configuration.enable_stubbing = false

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openstax_accounts
 version: !ruby/object:Gem::Version
-  version: 4.1.1
+  version: 5.0.0
 platform: ruby
 authors:
 - JP Slavinsky
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-20 00:00:00.000000000 Z
+date: 2015-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -81,13 +81,24 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.5.0
 - !ruby/object:Gem::Dependency
-  name: roar
+  name: representable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - "<"
+        version: '2.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: roar
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :runtime
@@ -95,9 +106,6 @@ dependencies:
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
-      - !ruby/object:Gem::Version
-        version: '0.8'
-    - - "<"
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
@@ -268,7 +276,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Rails common code and bindings and for the 'accounts' API
+description: This gem allows Rails apps to easily access the API's and login infrastructure
+  of OpenStax Accounts.
 email:
 - jps@kindlinglabs.com
 executables: []
@@ -330,11 +339,13 @@ files:
 - lib/generators/openstax/accounts/schedule/schedule_generator.rb
 - lib/generators/openstax/accounts/schedule/templates/schedule.rb
 - lib/omniauth/strategies/openstax.rb
+- lib/openstax/accounts/action_controller/base.rb
+- lib/openstax/accounts/api.rb
+- lib/openstax/accounts/configuration.rb
 - lib/openstax/accounts/current_user_manager.rb
 - lib/openstax/accounts/default_account_user_mapper.rb
 - lib/openstax/accounts/engine.rb
-- lib/openstax/accounts/extend_builtins.rb
-- lib/openstax/accounts/has_many_through_groups.rb
+- lib/openstax/accounts/has_many_through_groups/active_record/base.rb
 - lib/openstax/accounts/version.rb
 - lib/openstax_accounts.rb
 - spec/controllers/openstax/accounts/dev/accounts_controller_spec.rb
@@ -398,9 +409,9 @@ files:
 - spec/factories/openstax_accounts_group_owner.rb
 - spec/handlers/openstax/accounts/accounts_search_spec.rb
 - spec/handlers/openstax/accounts/dev/accounts_search_spec.rb
+- spec/lib/openstax/accounts/api_spec.rb
 - spec/lib/openstax/accounts/current_user_manager_spec.rb
-- spec/lib/openstax/accounts/has_many_through_groups_spec.rb
-- spec/lib/openstax_accounts_spec.rb
+- spec/lib/openstax/accounts/has_many_through_groups/active_record/base_spec.rb
 - spec/models/openstax/accounts/account_spec.rb
 - spec/models/openstax/accounts/anonymous_account_spec.rb
 - spec/models/openstax/accounts/group_spec.rb
@@ -430,7 +441,7 @@ rubyforge_project:
 rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
-summary: Rails common code and bindings and for the 'accounts' API
+summary: Rails common code and bindings for the 'accounts' API
 test_files:
 - spec/controllers/openstax/accounts/dev/accounts_controller_spec.rb
 - spec/controllers/openstax/accounts/sessions_controller_spec.rb
@@ -493,9 +504,9 @@ test_files:
 - spec/factories/openstax_accounts_group_owner.rb
 - spec/handlers/openstax/accounts/accounts_search_spec.rb
 - spec/handlers/openstax/accounts/dev/accounts_search_spec.rb
+- spec/lib/openstax/accounts/api_spec.rb
 - spec/lib/openstax/accounts/current_user_manager_spec.rb
-- spec/lib/openstax/accounts/has_many_through_groups_spec.rb
-- spec/lib/openstax_accounts_spec.rb
+- spec/lib/openstax/accounts/has_many_through_groups/active_record/base_spec.rb
 - spec/models/openstax/accounts/account_spec.rb
 - spec/models/openstax/accounts/anonymous_account_spec.rb
 - spec/models/openstax/accounts/group_spec.rb