startback 0.8.2 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 544da5f2ff4ff9484ed246bbe6c0068d19d9077f7fc56fef34adde6c7a298e1a
-  data.tar.gz: f56e2327c4b3a6fbdd29c8eac9dc6751d64b9da6b186c9cec093c3d8b97973b6
+  metadata.gz: 10937b9e7877dd7393837c70185fe245493a35234d66a899efee5fbebb539b75
+  data.tar.gz: 5fd7234b34b09f875dcf9b21ad819eb5072ffed902ddd08ca6c2e6a68e3ec1df
 SHA512:
-  metadata.gz: e206556b64bc960040527ec8ad17d2e0a610ff9a63d8f46343e687e762cc6d44ab2a462ca9e5897144bb328636c86587f11517847466c25dd360d3866636d488
-  data.tar.gz: 4965c3257c6a6c2a93b728a97821ffcd10613f2ab596d7c1073e39ff86dcdd66bda838241164f361fd9f4adc7edc5b9671914732e8c27c0c154bca7093452ca3
+  metadata.gz: 37df032165dd22b0680f8eaf874bada4810707253e06677c6a80b5d21afc87062643143e5f2b50e0f3adbe48d9df00368bb5f7070068e8458565e0d0fb8f1112
+  data.tar.gz: 754c4703f78bca862a4a7b2f8d973609ec44fab70368fe6d6d371050c140592a5706720ef0208ac4002ce434d85ad8e4d095d07840cd7993e2cb698cd116da5d

data/lib/startback/audit/trailer.rb

@@ -74,13 +74,13 @@ module Startback
 
     protected
 
-      def op_to_trail(op, time, ex = nil)
+      def op_to_trail(op, time = nil, ex = nil)
         log_msg = {
           op_took: time ? time.round(8) : nil,
           op: op_name(op),
           context: op_context(op),
           op_data: op_data(op)
-        }
+        }.compact
         log_msg[:error] = ex if ex
         log_msg
       end
@@ -104,6 +104,8 @@ module Startback
           op.input
         elsif op.respond_to?(:request, false)
           op.request
+        elsif op.is_a?(Operation::MultiOperation)
+          op.ops.map{ |sub_op| op_to_trail(sub_op) }
         end
         sanitize(data)
       end

data/lib/startback/bus/bunny/async.rb

@@ -35,6 +35,8 @@ module Startback
       class Async
         include Support::Robustness
 
+        CHANNEL_KEY = 'Startback::Bus::Bunny::Async::ChannelKey'
+
         DEFAULT_OPTIONS = {
           # (optional) The URL to use for connecting to RabbitMQ.
           url: ENV['STARTBACK_BUS_BUNNY_ASYNC_URL'],
@@ -65,11 +67,15 @@ module Startback
           try_max_times(10) do
             @bunny = ::Bunny.new(conn)
             @bunny.start
-            @channel = @bunny.create_channel
+            channel
             log(:info, {op: "#{self.class.name}#connect", op_data: conn}, options[:context])
           end
         end
-        attr_reader :channel, :options
+        attr_reader :options
+
+        def channel
+          Thread.current[CHANNEL_KEY] ||= @bunny.create_channel
+        end
 
         def emit(event)
           stop_errors(self, "emit", event.context) do

data/lib/startback/operation/multi_operation.rb

@@ -1,6 +1,6 @@
 module Startback
   class Operation
-    class MultiOperation
+    class MultiOperation < Operation
 
       def initialize(ops = [])
         @ops = ops

data/lib/startback/operation.rb

@@ -29,6 +29,7 @@ module Startback
   #     end
   #
   class Operation
+    extend Support::TransactionPolicy
     include Errors
     include Support::OperationRunner
     include Support::Hooks.new(:call)

data/lib/startback/support/transaction_manager.rb

@@ -0,0 +1,25 @@
+module Startback
+  module Support
+    class TransactionManager
+
+      def initialize(db, method = :transaction)
+        @db = db
+        @method = method
+      end
+
+      def call(runner, op, &then_block)
+        raise ArgumentError, "A block is required" unless then_block
+
+        before = (op.class.transaction_policy == :before_call)
+        if before
+          @db.send(@method) do
+            then_block.call
+          end
+        else
+          then_block.call
+        end
+      end
+
+    end # class TransactionManager
+  end # module Support
+end # module Startback

data/lib/startback/support/transaction_policy.rb

@@ -0,0 +1,33 @@
+module Startback
+  module Support
+    module TransactionPolicy
+
+      # Returns the operation's transaction policy
+      def transaction_policy
+        @transaction_policy || :before_call
+      end
+
+      # Sets the transaction policy to use. Valid values are:
+      # - before_call : the transaction is started by the operation
+      #   runner, right before calling the #call method on operation
+      #   instance
+      # - within_call: the transaction is started by the operation
+      #   itself, as part of its internal logic.
+      def transaction_policy=(policy)
+        unless [:before_call, :within_call].include?(policy)
+          raise ArgumentError, "Unknown policy `#{policy}`"
+        end
+        @transaction_policy = policy
+      end
+
+      def after_commit(&bl)
+        after_call do
+          db.after_commit do
+            instance_exec(&bl)
+          end
+        end
+      end
+
+    end # module TransactionPolicy
+  end # module Support
+end # module Startback

data/lib/startback/support.rb

@@ -19,3 +19,5 @@ require_relative 'support/logger'
 require_relative 'support/robustness'
 require_relative 'support/hooks'
 require_relative 'support/operation_runner'
+require_relative 'support/transaction_policy'
+require_relative 'support/transaction_manager'

data/lib/startback/version.rb

@@ -1,8 +1,8 @@
 module Startback
   module Version
     MAJOR = 0
-    MINOR = 8
-    TINY  = 2
+    MINOR = 10
+    TINY  = 0
   end
   VERSION = "#{Version::MAJOR}.#{Version::MINOR}.#{Version::TINY}"
 end

data/spec/unit/support/operation_runner/test_around_run.rb

@@ -68,7 +68,7 @@ module Startback
       end
 
       context 'the around feature with a class' do
-        class TransactionManager
+        class SomeTransactionManager
           include Singleton
 
           def initialize
@@ -87,7 +87,7 @@ module Startback
 
         class RunnerTest3
           include OperationRunner
-          around_run TransactionManager.instance
+          around_run SomeTransactionManager.instance
 
           def operation_world(op)
             { hello: "world" }
@@ -97,7 +97,7 @@ module Startback
         it 'calls the proc with expected parameters' do
           test = RunnerTest3.new
           got = test.run(op)
-          expect(TransactionManager.instance.called).to eql(true)
+          expect(SomeTransactionManager.instance.called).to eql(true)
           expect(got).to eql({
             seen_hello: "world"
           })
@@ -154,4 +154,3 @@ module Startback
     end # module OperationRunner
   end # module Support
 end # module Startback
-

data/spec/unit/support/test_transaction_manager.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+module Startback
+  module Support
+    describe TransactionManager do
+      subject do
+        TransactionManager.new(db).call(nil, op) do
+          op.call
+        end
+      end
+
+      class FakeDatabase
+        def initialize
+          @called = false
+        end
+        attr_reader :called
+
+        def transaction
+          @called = true
+          yield
+        end
+      end
+
+      let(:db) do
+        FakeDatabase.new
+      end
+
+      context 'when called with a default operation' do
+        class OperationNotManagingTransactions < Startback::Operation
+          def call
+            12
+          end
+        end
+
+        let(:op) do
+          OperationNotManagingTransactions.new
+        end
+
+        it 'calls db.transaction' do
+          expect(subject).to eql(12)
+          expect(db.called).to eql(true)
+        end
+      end
+
+      context 'when called with an operation that manages the transactions itself' do
+        class OperationManagingTransactions < Startback::Operation
+          self.transaction_policy = :within_call
+
+          def call
+            12
+          end
+        end
+
+        let(:op) do
+          OperationManagingTransactions.new
+        end
+
+        it 'calls db.transaction' do
+          expect(subject).to eql(12)
+          expect(db.called).to eql(false)
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: startback
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.10.0
 platform: ruby
 authors:
 - Bernard Lambeau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-05-05 00:00:00.000000000 Z
+date: 2021-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -210,7 +210,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.11.4
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -220,7 +220,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.11.4
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -230,20 +230,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.3'
+        version: 5.3.1
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.3'
+        version: 5.3.1
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: jwt
   requirement: !ruby/object:Gem::Requirement
@@ -410,6 +410,8 @@ files:
 - lib/startback/support/logger.rb
 - lib/startback/support/operation_runner.rb
 - lib/startback/support/robustness.rb
+- lib/startback/support/transaction_manager.rb
+- lib/startback/support/transaction_policy.rb
 - lib/startback/version.rb
 - lib/startback/web/api.rb
 - lib/startback/web/auto_caching.rb
@@ -437,6 +439,7 @@ files:
 - spec/unit/support/operation_runner/test_around_run.rb
 - spec/unit/support/operation_runner/test_before_after_call.rb
 - spec/unit/support/test_robusteness.rb
+- spec/unit/support/test_transaction_manager.rb
 - spec/unit/test_event.rb
 - spec/unit/test_operation.rb
 - spec/unit/test_support.rb
@@ -469,7 +472,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.2.32
 signing_key: 
 specification_version: 4
 summary: Got Your Ruby Back