bridge_api 0.1.20 → 0.1.21

data/lib/bridge_api/client/role.rb

@@ -1,7 +1,6 @@
 module BridgeAPI
   class Client
     module Role
-
       def get_all_roles(params = {})
         get("#{API_PATH}#{AUTHOR_PATH}#{ROLE_PATH}", params)
       end
@@ -9,7 +8,6 @@ module BridgeAPI
       def get_role(role_id, params = {})
         get("#{API_PATH}#{AUTHOR_PATH}#{ROLE_PATH}/#{role_id}", params)
       end
-
     end
   end
-end
+end

data/lib/bridge_api/client/sub_account.rb

@@ -5,6 +5,10 @@ module BridgeAPI
         get("#{API_PATH}#{ADMIN_PATH}#{SUB_ACCOUNT_PATH}", params)
       end
 
+      def update_subaccount(subaccount_id, params = {})
+        put("#{API_PATH}#{ADMIN_PATH}#{SUB_ACCOUNT_PATH}/#{subaccount_id}", params)
+      end
+
       def share_course_with_subaccount(course_template_id, params = {})
         put("#{API_PATH}#{AUTHOR_PATH}#{COURSE_TEMPLATE_PATH}/#{course_template_id}/#{SUB_ACCOUNT_PATH}/share", params)
       end

data/lib/bridge_api/client/user.rb

@@ -1,7 +1,6 @@
 module BridgeAPI
   class Client
     module User
-
       def add_user(params = {})
         post("#{API_PATH}#{ADMIN_PATH}#{USER_PATH}", params)
       end
@@ -43,20 +42,26 @@ module BridgeAPI
       #   '2': 'Another Custom Value'
       # }
       def build_custom_values_payload(user_id, custom_field_values)
-        users = self.get_user(user_id, {'includes[]' => 'custom_fields'})
+        users = get_user(user_id, 'includes[]' => 'custom_fields')
         payload = []
-        return payload unless users.members.size > 0
+        return payload if users.members.empty?
         existing_values = users.linked['custom_field_values']
         custom_field_values.each do |field_id, value|
-          field_value_id = existing_values.find {|v|
-            v['links']['custom_field']['id'] == field_id.to_s rescue false
-          }['id'] rescue nil
-          payload << {'id' => field_value_id, 'custom_field_id' => field_id, 'value' => value}
+          field_value_id = begin
+                             existing_values.find do |v|
+                               begin
+                                 v['links']['custom_field']['id'] == field_id.to_s
+                               rescue StandardError
+                                 false
+                               end
+                             end['id']
+                           rescue StandardError
+                             nil
+                           end
+          payload << { 'id' => field_value_id, 'custom_field_id' => field_id, 'value' => value }
         end
         payload
       end
-
     end
-
   end
-end
+end

data/lib/bridge_api/version.rb

@@ -1,3 +1,3 @@
 module BridgeAPI
-  VERSION = '0.1.20' unless defined?(BridgeAPI::VERSION)
+  VERSION = '0.1.21'.freeze unless defined?(BridgeAPI::VERSION)
 end

data/spec/bridge_api/client/account_spec.rb

@@ -0,0 +1,14 @@
+require 'test_helper'
+
+describe BridgeAPI::Client::CourseTemplate do
+  before do
+    @client = BridgeAPI::Client.new(
+      prefix: 'http://test.bridge.com', token: 'test_token'
+    )
+  end
+
+  it 'should update an account' do
+    templates = @client.update_account(1)
+    expect(templates.status).to(eq(200))
+  end
+end

data/spec/bridge_api/client/course_template_spec.rb

@@ -1,9 +1,8 @@
 require 'test_helper'
 
 describe BridgeAPI::Client::CourseTemplate do
-
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should get all course templates' do
@@ -40,5 +39,4 @@ describe BridgeAPI::Client::CourseTemplate do
     expect(template.count).to(eq(1))
     expect(template.first['title']).to(eq('My only course'))
   end
-
 end

data/spec/bridge_api/client/custom_field_spec.rb

@@ -1,19 +1,18 @@
 require 'test_helper'
 require 'byebug'
 describe BridgeAPI::Client::CustomField do
-
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should add an custom field' do
     response = @client.add_custom_field
-    expect(response.first['id']).to(eq("30"))
+    expect(response.first['id']).to(eq('30'))
   end
 
   it 'should get a custom field' do
     response = @client.get_custom_field(1)
-    expect(response.first['id']).to(eq("30"))
+    expect(response.first['id']).to(eq('30'))
   end
 
   it 'should delete an custom field' do
@@ -23,13 +22,11 @@ describe BridgeAPI::Client::CustomField do
 
   it 'should update a custom field' do
     response = @client.update_custom_field(1)
-    expect(response.first['id']).to(eq("30"))
+    expect(response.first['id']).to(eq('30'))
   end
 
   it 'should get all custom field' do
     response = @client.get_all_custom_fields
     expect(response.length).to(eq(1))
   end
-
 end
-

data/spec/bridge_api/client/data_dump_spec.rb

@@ -2,7 +2,7 @@ require 'test_helper'
 
 describe BridgeAPI::Client::DataDump do
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should get all data dumps' do

data/spec/bridge_api/client/enrollment_spec.rb

@@ -1,9 +1,8 @@
 require 'test_helper'
 
 describe BridgeAPI::Client::Enrollment do
-
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should add an enrollment' do
@@ -14,12 +13,11 @@ describe BridgeAPI::Client::Enrollment do
   it 'should delete an enrollment' do
     response = @client.delete_enrollment(1)
     expect(response.status).to(eq(204))
-
   end
 
   it 'should update an enrollment' do
     enrollment = @client.update_enrollment(1)
-    expect(enrollment.first['id']).to(eq("3"))
+    expect(enrollment.first['id']).to(eq('3'))
   end
 
   it 'should get all enrollments' do
@@ -28,10 +26,8 @@ describe BridgeAPI::Client::Enrollment do
   end
 
   it 'should update due date for enrollments' do
-    enrollment = [{end_at: nil}]
+    enrollment = [{ end_at: nil }]
     response = @client.update_enrollment_due_date(1, enrollment)
     expect(response.status).to(eq 200)
   end
-
 end
-

data/spec/bridge_api/client/group_spec.rb

@@ -2,10 +2,10 @@ require 'test_helper'
 
 describe BridgeAPI::Client::Group do
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
-  let(:params) {{ name: 'Test Group', smart: true }}
+  let(:params) { { name: 'Test Group', smart: true } }
 
   it 'should create a group' do
     response = @client.add_group(groups: params)
@@ -22,4 +22,4 @@ describe BridgeAPI::Client::Group do
     expect(response.members.first['id']).to eq('6')
     expect(response.members.first['type']).to eq('smart')
   end
-end
+end

data/spec/bridge_api/client/live_course_enrollments_spec.rb

@@ -2,7 +2,7 @@ require 'test_helper'
 
 describe BridgeAPI::Client::LiveCourseEnrollment do
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should get a list of live course enrollments' do

data/spec/bridge_api/client/live_course_spec.rb

@@ -2,11 +2,11 @@ require 'test_helper'
 
 describe BridgeAPI::Client::LiveCourse do
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should get a list of live courses' do
-    response = @client.get_all_live_courses()
+    response = @client.get_all_live_courses
     expect(response.status).to eq(200)
     expect(response.members.count).to eq(2)
     expect(response.members.first['id']).to eq('1')

data/spec/bridge_api/client/manager_spec.rb

@@ -1,22 +1,20 @@
 require 'test_helper'
 
 describe BridgeAPI::Client::User do
-
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should get all managers' do
     response = @client.get_all_managers
-    expect(response.first['id']).to(eq("17"))
+    expect(response.first['id']).to(eq('17'))
   end
 
   it 'should get all the people who report to a manager' do
     response = @client.get_manager_direct_reports(17)
     expect(response.members.count).to(eq(2))
-    expect(response.members.first['id']).to(eq("3"))
+    expect(response.members.first['id']).to(eq('3'))
     expect(response.meta['direct_reports_count']).to(eq(2))
-    expect(response.meta['manager_id']).to(eq("17"))
+    expect(response.meta['manager_id']).to(eq('17'))
   end
-
 end

data/spec/bridge_api/client/program_enrollment_spec.rb

@@ -2,7 +2,7 @@ require 'test_helper'
 
 describe BridgeAPI::Client::ProgramEnrollment do
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should add a program enrollment' do

data/spec/bridge_api/client/sub_account_spec.rb

@@ -1,14 +1,12 @@
 require 'test_helper'
 
 describe BridgeAPI::Client::SubAccount do
-
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should get all subaccounts' do
     response = @client.get_sub_accounts
-    expect(response.first['id']).to(eq("141"))
+    expect(response.first['id']).to(eq('141'))
   end
-
 end

data/spec/bridge_api/client/user_spec.rb

@@ -1,19 +1,18 @@
 require 'test_helper'
 
 describe BridgeAPI::Client::User do
-
   before do
-    @client = BridgeAPI::Client.new(prefix: "http://test.bridge.com", token: "test_token")
+    @client = BridgeAPI::Client.new(prefix: 'http://test.bridge.com', token: 'test_token')
   end
 
   it 'should add an user' do
     response = @client.add_user
-    expect(response.first['id']).to(eq("10"))
+    expect(response.first['id']).to(eq('10'))
   end
 
   it 'should get a user' do
     response = @client.get_user(1)
-    expect(response.first['id']).to(eq("10"))
+    expect(response.first['id']).to(eq('10'))
   end
 
   it 'should delete an user' do
@@ -23,16 +22,14 @@ describe BridgeAPI::Client::User do
 
   it 'should update a user' do
     response = @client.update_user(1)
-    expect(response.first['id']).to(eq("10"))
+    expect(response.first['id']).to(eq('10'))
 
-    response = @client.update_user_with_custom_values(1, {'1' => 'test'})
-    expect(response.first['id']).to(eq("10"))
+    response = @client.update_user_with_custom_values(1, '1' => 'test')
+    expect(response.first['id']).to(eq('10'))
   end
 
   it 'should get all users' do
     response = @client.get_all_users
     expect(response.length).to(eq(2))
   end
-
 end
-

data/spec/bridge_api/client_spec.rb

@@ -1,27 +1,24 @@
 
 describe BridgeAPI::Client do
-
   it 'should set the auth header to basic auth' do
-    client = BridgeAPI::Client.new(prefix: "https://www.fake.com", api_key: "fake_token", api_secret: "fake_secret")
-    expect(client.connection.headers['Authorization']).to(eq("Basic ZmFrZV90b2tlbjpmYWtlX3NlY3JldA=="))
+    client = BridgeAPI::Client.new(prefix: 'https://www.fake.com', api_key: 'fake_token', api_secret: 'fake_secret')
+    expect(client.connection.headers['Authorization']).to(eq('Basic ZmFrZV90b2tlbjpmYWtlX3NlY3JldA=='))
   end
 
   it 'should set the auth header to bearer auth' do
-    client = BridgeAPI::Client.new(prefix: "https://www.fake.com", token: "test_token")
-    expect(client.connection.headers['Authorization']).to(eq("Bearer test_token"))
+    client = BridgeAPI::Client.new(prefix: 'https://www.fake.com', token: 'test_token')
+    expect(client.connection.headers['Authorization']).to(eq('Bearer test_token'))
   end
 
   it 'should set the meta property on client' do
-    client = BridgeAPI::Client.new(prefix: "https://www.fake.com", token: "test_token")
+    client = BridgeAPI::Client.new(prefix: 'https://www.fake.com', token: 'test_token')
     users = client.get_all_users
     expect(users.meta['next']).to(eq('http://bridge.bridgeapp.com/api/author/users?after=eyJ0eXAiOiJKV1QiLCJhSDiQQ'))
   end
 
   it 'should set the linked property on client' do
-    client = BridgeAPI::Client.new(prefix: "https://www.fake.com", token: "test_token")
+    client = BridgeAPI::Client.new(prefix: 'https://www.fake.com', token: 'test_token')
     users = client.get_all_users
     expect(users.linked['custom_fields'][0]['id']).to(eq('1'))
   end
-
-
-end
+end

data/spec/fixtures/accounts.json

@@ -0,0 +1,129 @@
+{
+  "meta": {},
+  "accounts": [
+    {
+      "id": "968",
+      "active": false,
+      "created_at": "2018-05-10T09:41:41.707-06:00",
+      "name": "psengtest",
+      "host": {
+        "subdomain": "psengtest",
+        "vanity_domain": null,
+        "vanity_domain_state": "vanity_domain_inactive"
+      },
+      "account_type": "paid",
+      "cluster": "cluster2",
+      "support_account": false,
+      "config": {
+        "contact_name": null,
+        "contact_email": null,
+        "contact_phone": null,
+        "import_passwords": false,
+        "support_level": "adminsOnly",
+        "show_custom_support_info": false,
+        "support_name": null,
+        "support_phone": null,
+        "support_email": null,
+        "use_custom_support": false,
+        "custom_support_chat_url": "https://lc.chat/now/8288951/2",
+        "custom_support_ticket_url": "https://bridgeapp.zendesk.com/hc/en-us/requests/new?ticket_form_id=794528",
+        "custom_support_phone_number": "855-383-4550",
+        "custom_back_to_my_learning_label": null,
+        "custom_back_to_my_learning_href": null,
+        "session_timeout_redirect_url": null,
+        "lynda_org_id": null,
+        "locale": "en",
+        "time_zone": "America/Denver",
+        "auto_csv": false,
+        "scorm_sftp": false,
+        "limited_authoring": false,
+        "limited_managers": false,
+        "bridge_retain": true,
+        "password_reset_allowed": true,
+        "create_user_from_auth_hash": true,
+        "salesforce_id": null,
+        "progressive_finance_customization": false,
+        "data_dump_enabled": false,
+        "data_dump_subaccount_enabled": false,
+        "cloneable": false,
+        "learner_library_enabled": true,
+        "learner_calendar_enabled": true,
+        "iframe_host": null,
+        "session_reissue_seconds": 1800,
+        "session_expire_seconds": 86400,
+        "user_limit": 0,
+        "expiration_date": "2018-06-09",
+        "used_free_trial_extension": false,
+        "lynda_library_enabled": false,
+        "external_course_id_enabled": false,
+        "learner_library_images_disabled": false,
+        "elastic_search_enabled": false,
+        "products": [
+          {
+            "product_name": "learn",
+            "subscription_type": "paid"
+          }
+        ],
+        "webhooks_enabled": false,
+        "calendar": {
+          "office365": null
+        },
+        "auth": {
+          "provider": "Basic",
+          "model": "User"
+        }
+      },
+      "import_profile": {
+        "csv_days": [
+          false,
+          false,
+          false,
+          false,
+          false,
+          false,
+          false
+        ],
+        "csv_password": null,
+        "csv_time": null,
+        "csv_url": null,
+        "csv_user": null,
+        "csv_public_key": null,
+        "try_key_based_auth": null
+      },
+      "import_scorm_profile": {
+        "sftp_days": [
+          false,
+          false,
+          false,
+          false,
+          false,
+          false,
+          false
+        ],
+        "sftp_time": null,
+        "sftp_url": null,
+        "sftp_user": null,
+        "sftp_password": null,
+        "sftp_public_key": null,
+        "registration_instancing_option": "never",
+        "sco_launch_type": "1",
+        "score_rollup_mode": "1",
+        "publish_on_upload": true,
+        "add_to_learner_library": null,
+        "default_days_until_due": null,
+        "email": null,
+        "try_key_based_auth": null
+      },
+      "domain": {
+        "tac_type": "domestic",
+        "tac_custom_body": null
+      },
+      "notification_profile": {
+        "notifications": false,
+        "chat_notifications": false,
+        "from_label": null,
+        "reply_to": null
+      }
+    }
+  ]
+}