rookout 0.1.0 → 0.1.56

data/lib/rookout/processor/namespaces/stack_namespace.rb

@@ -5,14 +5,16 @@ module Rookout
       require_relative "frame_namespace"
       require_relative "traceback_namespace"
 
+
       class StackNamespace < Namespace
-        def initialize backtrace, offset = 0
-          @backtrace = backtrace
+        def initialize stack_trace, offset = 0
+          super()
+          @stack_trace = stack_trace
           @offset = offset
         end
 
         def read_key key
-          FrameNamespace.new @backtrace[key + @offset]
+          FrameNamespace.new nil, @stack_trace[key + @offset]
         end
 
         def call_method name, args
@@ -22,7 +24,7 @@ module Rookout
 
         def create_traceback args
           args = 1000 if args == ""
-          TracebackNamespace.new @backtrace, @offset, args.to_i
+          TracebackNamespace.new @stack_trace, @offset, args.to_i
         end
       end
     end

data/lib/rookout/processor/namespaces/traceback_namespace.rb

@@ -3,18 +3,22 @@ module Rookout
     module Namespaces
       require_relative "namespace"
       require_relative "frame_namespace"
-
       require_relative "../../protobuf/variant_pb"
 
       class TracebackNamespace < Namespace
-        def initialize backtrace, offset, depth
-          @backtrace = backtrace
+        def initialize stack_trace, offset, depth
+          super()
+          @stack_trace = stack_trace
           @offset = offset
           @depth = depth
         end
 
+        attr_reader :stack_trace
+        attr_reader :offset
+        attr_reader :depth
+
         def read_key key
-          FrameNamespace.new @backtrace[key + @offset]
+          FrameNamespace.new nil, @stack_trace[key + @offset]
         end
 
         def dump _log_object_errors
@@ -22,12 +26,12 @@ module Rookout
 
           @depth.times do |i|
             position = i + @offset
-            break if position >= @backtrace.length
-            frame = @backtrace[position]
+            break if position >= @stack_trace.length
+            frame_info = @stack_trace[position]
 
-            code_object = Com::Rookout::Variant::CodeObject.new filename: frame.source_location[0],
-                                                                lineno: frame.source_location[1],
-                                                                name: frame.eval("__method__").to_s
+            code_object = Com::Rookout::Variant::CodeObject.new filename: frame_info.path,
+                                                                lineno: frame_info.lineno,
+                                                                name: frame_info.label
 
             traceback.locations << code_object
           end

data/lib/rookout/processor/operations/set_operation.rb

@@ -3,12 +3,16 @@ module Rookout
     module Operations
       require_relative "operation"
 
+      require_relative "../namespaces/ruby_object_namespace"
+
       require_relative "../rook_error"
       require_relative "../../logger"
       require_relative "../../user_warnings"
 
       class SetOperation < Operation
         def initialize configuration, factory
+          super()
+
           @paths = []
 
           configuration["paths"].each do |key, value|
@@ -29,7 +33,8 @@ module Rookout
           @paths.each do |dest_path, source_path|
             begin
               value = source_path.read_from namespace
-              if value.is_a?(RubyObjectNamespace) && value.dump_config == Namespaces::OBJECT_DUMP_CONFIG_DEFAULT
+              if value.is_a?(Namespaces::RubyObjectNamespace) &&
+                 value.dump_config == Namespaces::OBJECT_DUMP_CONFIG_DEFAULT
                 value.tailor_limits!
               end
 

data/lib/rookout/processor/paths/arithmetic_path.rb

@@ -13,6 +13,8 @@ module Rookout
 
       class ArithmeticPath < Path
         def initialize path
+          super()
+
           if path.is_a? Hash
             path = path["path"] || path[:path]
           end
@@ -20,7 +22,7 @@ module Rookout
           @raw_path = path
 
           if path.start_with?("NOT(") && path.end_with?(")")
-            @path = path["NOT(".length..-2]
+            @path = path[("NOT(".length)..-2]
             @negation = true
           else
             @path = path
@@ -33,7 +35,7 @@ module Rookout
         def read_from namespace
           begin
             result = Maps.parse @path, actions: Canopy::Actions.new(namespace)
-          rescue Maps::ParseError => e
+          rescue Maps::ParseError, NameError => e
             raise RookInvalidArithmeticPath.new(@path, e)
           end
 
@@ -71,7 +73,7 @@ module Rookout
 
           if result.is_a? Canopy::ToolExceptionMarker
             raise result.obj if result.obj.is_a? Exceptions::ToolException
-            raise Exceptions::RookInvalidArithmeticPath, @raw_path, result.obj
+            raise Exceptions::RookInvalidArithmeticPath.new(@raw_path, result.obj)
           end
 
           res = result.obj

data/lib/rookout/processor/paths/canopy/actions.rb

@@ -79,6 +79,8 @@ module Rookout
           end
 
           # rubocop:disable Metrics/AbcSize
+          # rubocop:disable Metrics/CyclomaticComplexity
+          # rubocop:disable Metrics/PerceivedComplexity
           def make_comp_exp _input, _start, _finish, elements
             # We can assume the following: atom ( opt_ atom )*
             # the first (which must be) will be simple atom
@@ -108,7 +110,7 @@ module Rookout
                 flat_elements.each_with_index do |e, index|
                   if e.is_a?(Opt) && e.level == level
                     result = e.execute_operation flat_elements[index - 1], flat_elements[index + 1]
-                    flat_elements = flat_elements[0...index - 1] + [result] + flat_elements[index + 2..-1]
+                    flat_elements = flat_elements[0...index - 1] + [result] + flat_elements[index + 2..]
                     restart_scan = true
                     break
                   end
@@ -122,6 +124,8 @@ module Rookout
 
             flat_elements[0]
           end
+          # rubocop:enable Metrics/PerceivedComplexity
+          # rubocop:enable Metrics/CyclomaticComplexity
           # rubocop:enable Metrics/AbcSize
 
           def make_opt _input, _start, _finish, elements

data/lib/rookout/processor/paths/canopy/consts.rb

@@ -50,9 +50,10 @@ module Rookout
             # If it's a dict, scan the keys, compensating for additional abstraction
             if b.is_a? Hash
               b.each_key do |key|
-                if key.is_a? ObjectMarker
+                case key
+                when ObjectMarker
                   return true if key.obj == a
-                elsif key == a
+                when a
                   return true
                 end
               end
@@ -62,9 +63,10 @@ module Rookout
             # If it's an array, scan the value, compensating for additional abstraction
             if b.is_a? Array
               b.each do |value|
-                if value.is_a? ObjectMarker
+                case value
+                when ObjectMarker
                   return true if value.obj == a
-                elsif value == a
+                when a
                   return true
                 end
               end