strongmind-platform-sdk 3.21.0 → 3.21.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e6ef87966021eec7c833f5d1b603373ba547f1687d953fbd8923ee33af5f5aa2
-  data.tar.gz: fa98f5dbd4413fceb3cbeba32bef476d9bfebbc9c7afd13e23d76e3c41c2ffed
+  metadata.gz: 3e7d0d42072dcbe33f1f9cd7b6e4a51259c7aeb81e3df2075a8ad2d88b6f8f1e
+  data.tar.gz: 9319eb65bc4881be26cdd24b77f54c33616e2b38e1bf05997f65bbf236badcbf
 SHA512:
-  metadata.gz: cb0a228aa4c60a050b539858abb6f50399d5779793863c085b5d13f22cd166007c3528d2d297338e315dfb24f3eca6661efac8455d38e1e0128f46f1aae4ed06
-  data.tar.gz: 88c84634da5100b977627e3347d89ecee11849b64aebc55c919213cc9ca82e856e39dcd4a637b91bdadc9f015eabbcdaff92bc4a370171df34d4b8d9cd71ddcf
+  metadata.gz: f86bfa1c3e710781d1423ee2445d1a2df145d7badc96501632ea0729ac3e022eb09f3e695eb9e8bc0f64996bec4f05cdef27433195bf61ccbc38a0c1c98b6824
+  data.tar.gz: '05816f03571b880939cb9ab1ea797b1689ae32866c2289b03ace4db1787b734d8bb246d9c68444e1cb6fbdc040b055a4677900a46b39e51927d20c09c0ce92f0'

data/lib/platform_sdk/active_record/data_pipelineable.rb

@@ -13,15 +13,18 @@ module PlatformSdk
       end
 
       def send_pipeline_create
-        send_to_pipeline('created')
+        Jobs::SendNounToPipelineJob.perform_later(pipeline_payload('created'),
+                                                  one_roster_pipeline_payload)
       end
 
       def send_pipeline_update
-        send_to_pipeline('modified')
+        Jobs::SendNounToPipelineJob.perform_later(pipeline_payload('modified'),
+                                                  one_roster_pipeline_payload)
       end
 
       def send_pipeline_destroy
-        send_to_pipeline('destroyed')
+        Jobs::SendNounToPipelineJob.perform_later(pipeline_payload('destroyed'),
+                                                  one_roster_pipeline_payload(deleted: true))
       end
 
       def pipeline_payload(action)
@@ -67,20 +70,6 @@ module PlatformSdk
       def one_roster_pipeline_payload(deleted: false)
         {}
       end
-
-      def send_to_pipeline(action)
-        credentials = {
-          pipeline_host: ENV.fetch('DATA_PIPELINE_HOST', 'stage-di-data-pipeline-api.strongmind.com'),
-          pipeline_username: ENV.fetch('DATA_PIPELINE_USERNAME', 'canvas_prod'),
-          pipeline_password: ENV.fetch('DATA_PIPELINE_PASSWORD', '')
-        }
-        client = PlatformSdk::DataPipeline::Client.new(credentials)
-
-        client.post(pipeline_payload(action)) unless Rails.env.development?
-        return unless respond_to?(:one_roster_data_type) && !Rails.env.development? && one_roster_pipeline_payload(deleted: action == "destroyed").present?
-
-        client.post(one_roster_pipeline_payload(deleted: action == "destroyed"))
-      end
     end
   end
 end

data/lib/platform_sdk/canvas_api/client.rb

@@ -547,8 +547,10 @@ module PlatformSdk
       # @return [String]
       def parse_response_errors(message:, result:)
         error_messages = String.new
-        result[:errors].each do |error|
-          error_messages << "#{error[:message]} "
+        unless result[:errors].nil?
+          result[:errors].each do |error|
+            error_messages << "#{error[:message]} "
+          end
         end
         "#{message}: {#{error_messages}"
       end
@@ -567,7 +569,7 @@ module PlatformSdk
       # @param message [String]
       def handle_post_response_errors(response, message)
         result = MultiJson.load response.body, symbolize_keys: true
-        raise LockedModuleItemError, "#{message}: #{result[:message]}" if response.status == 403
+        raise LockedModuleItemError, "#{message}: #{result[:message]}" if response.status == 403 and result[:message] == 'The module item is locked.'
         raise MissingModuleItemError, "#{message}: #{result[:message]}" if response.status == 404
         raise CanvasClientError, parse_response_errors(message:, result:)
       end

data/lib/platform_sdk/jobs/send_noun_to_pipeline_job.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'active_job'
+
+module PlatformSdk
+  module Jobs
+    class SendNounToPipelineJob < ActiveJob::Base
+      queue_as :default
+
+      def perform(message, one_roster_message)
+        send_to_pipeline(message, one_roster_message)
+      end
+
+      private
+
+      def send_to_pipeline(message, one_roster_message)
+        credentials = {
+          pipeline_host: ENV.fetch('DATA_PIPELINE_HOST', 'stage-di-data-pipeline-api.strongmind.com'),
+          pipeline_username: ENV.fetch('DATA_PIPELINE_USERNAME', 'canvas_prod'),
+          pipeline_password: ENV.fetch('DATA_PIPELINE_PASSWORD', '')
+        }
+        client = PlatformSdk::DataPipeline::Client.new(credentials)
+
+        client.post(message) unless Rails.env.development?
+        return unless one_roster_message && !Rails.env.development?
+
+        client.post(one_roster_message)
+      end
+    end
+  end
+end

data/lib/platform_sdk/spec_support/shared_examples/data_pipelineable_examples.rb

@@ -1,6 +1,8 @@
 module PlatformSdk
   module SpecSupport
     RSpec.shared_examples 'DataPipelineable' do |params = {}|
+      include ActiveJob::TestHelper
+
       let(:shared_params) { params }
       let(:record) { build(described_class.to_s.underscore.to_sym) }
       let(:data_pipeline_client) { double(PlatformSdk::DataPipeline::Client) }
@@ -16,7 +18,9 @@ module PlatformSdk
       context 'create' do
 
         it 'posts a copy of our data to the data pipeline' do
-          record.save!
+          perform_enqueued_jobs do
+            record.save!
+          end
           expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                            noun: described_class.pipeline_noun,
                                                                            data: record.pipeline_data('created'),
@@ -31,8 +35,10 @@ module PlatformSdk
       context 'destroy' do
 
         it 'posts a copy of our data to the data pipeline' do
-          record.save!
-          record.destroy
+          perform_enqueued_jobs do
+            record.save!
+            record.destroy
+          end
 
           expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                            noun: described_class.pipeline_noun,
@@ -54,7 +60,9 @@ module PlatformSdk
           it 'posts a copy of our data to the data pipeline' do
             column = column_to_update(record)
             unless column.nil?
-              update_record(record, column)
+              perform_enqueued_jobs do
+                update_record(record, column)
+              end
               expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                                noun: described_class.pipeline_noun,
                                                                                data: record.pipeline_data('modified'),
@@ -71,10 +79,12 @@ module PlatformSdk
           let(:action) { 'modified' }
 
           it 'does not post to the data pipeline' do
-            record.save!
-            column = column_to_update(record)
-            update_record(record, column)
-            record.save!
+            perform_enqueued_jobs do
+              record.save!
+              column = column_to_update(record)
+              update_record(record, column)
+              record.save!
+            end
             expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                              noun: described_class.pipeline_noun,
                                                                              data: record.pipeline_data('modified'),

data/lib/platform_sdk/spec_support/shared_examples/one_roster_data_pipelineable_examples.rb

@@ -3,6 +3,8 @@
 module PlatformSdk
   module SpecSupport
     RSpec.shared_examples 'OneRosterDataPipelineable' do |record_keys|
+      include ActiveJob::TestHelper
+
       let(:data) do
         record_keys.each_with_object({}) do |key, hash|
           if key == '_type'
@@ -50,7 +52,9 @@ module PlatformSdk
         let(:action) { 'created' }
 
         it 'posts to the data pipeline' do
-          record.save!
+          perform_enqueued_jobs do
+            record.save!
+          end
           expect(data_pipeline_client).to have_received(:post).with(pipeline_payload)
         end
       end
@@ -68,8 +72,10 @@ module PlatformSdk
         end
 
         it 'posts to the data pipeline' do
-          record.save!
-          record.destroy
+          perform_enqueued_jobs do
+            record.save!
+            record.destroy
+          end
           expect(data_pipeline_client).to have_received(:post).with(pipeline_payload)
           expect(data_pipeline_client).to have_received(:post).with(pipeline_payload_deleted)
         end
@@ -86,7 +92,9 @@ module PlatformSdk
 
           it 'posts a copy of our data to the data pipeline' do
             column = column_to_update(record)
-            update_record(record, column)
+            perform_enqueued_jobs do
+              update_record(record, column)
+            end
             expect(data_pipeline_client).to have_received(:post).with(pipeline_payload)
           end
 
@@ -96,10 +104,12 @@ module PlatformSdk
           let(:action) { 'modified' }
 
           it 'does not post to the data pipeline' do
-            record.save!
-            column = column_to_update(record)
-            update_record(record, column)
-            record.save!
+            perform_enqueued_jobs do
+              record.save!
+              column = column_to_update(record)
+              update_record(record, column)
+              record.save!
+            end
             expect(data_pipeline_client).to have_received(:post).once.with(pipeline_payload)
           end
         end

data/lib/platform_sdk/version.rb

@@ -3,7 +3,7 @@
 module PlatformSdk
   MAJOR = 3
   MINOR = 21
-  PATCH = 0
+  PATCH = 2
 
   VERSION = "#{PlatformSdk::MAJOR}.#{PlatformSdk::MINOR}.#{PlatformSdk::PATCH}"
 end

data/lib/platform_sdk.rb

@@ -23,6 +23,7 @@ require "fog-aws"
 require "platform_sdk/asset_sync_initializer"
 require "platform_sdk/sidekiq/ecs_task_protection_middleware"
 require "platform_sdk/sidekiq"
+require "platform_sdk/jobs/send_noun_to_pipeline_job"
 
 module PlatformSdk
   class Error < StandardError; end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: strongmind-platform-sdk
 version: !ruby/object:Gem::Version
-  version: 3.21.0
+  version: 3.21.2
 platform: ruby
 authors:
 - Platform Team
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-11-11 00:00:00.000000000 Z
+date: 2024-11-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -273,6 +273,7 @@ files:
 - lib/platform_sdk/id_mapper/models/domain.rb
 - lib/platform_sdk/identity.rb
 - lib/platform_sdk/identity/clients.rb
+- lib/platform_sdk/jobs/send_noun_to_pipeline_job.rb
 - lib/platform_sdk/learnosity_api.rb
 - lib/platform_sdk/learnosity_api/client.rb
 - lib/platform_sdk/one_roster.rb