startback 0.9.1 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c2a20858c9d0287af3cfaa8e60c6f59630f1d258de469f7c50ffe1e6529742ca
-  data.tar.gz: c5d3fb83ab9a7aa0b1bb6fbec2f0bfeef40b4c51c15c602d9a2bca85b17e8d26
+  metadata.gz: 10937b9e7877dd7393837c70185fe245493a35234d66a899efee5fbebb539b75
+  data.tar.gz: 5fd7234b34b09f875dcf9b21ad819eb5072ffed902ddd08ca6c2e6a68e3ec1df
 SHA512:
-  metadata.gz: ee763fb9d8be8ca1f6cc8a541b70df6b72ae982e2026b04286ec5b6a208aba31641ea8062330f7efc8d01e548bfbca408edb115678dbc6fefa707e3d98fc8f0e
-  data.tar.gz: ec3607dec57ff640d4b49afb147a5c6893e45844e5d53cf07e8ca26ea709341260c02f01a25bad7fc68f231e859986ce2bb0604de7e6031f59f97d1f0a06a464
+  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/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 = 9
-    TINY  = 1
+    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.9.1
+  version: 0.10.0
 platform: ruby
 authors:
 - Bernard Lambeau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-12-13 00:00:00.000000000 Z
+date: 2021-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -380,7 +380,6 @@ extra_rdoc_files: []
 files:
 - README.md
 - Rakefile
-- VERSION
 - lib/startback.rb
 - lib/startback/audit.rb
 - lib/startback/audit/prometheus.rb
@@ -411,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
@@ -438,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

data/VERSION

@@ -1 +0,0 @@
-0.9.1