rookout 0.1.7 → 0.1.12

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

@@ -154,6 +154,22 @@ module Rookout
 
           attr_reader :level
 
+          def check_result result, first_obj, second_obj
+            return unless [true, false].include? result
+            if result
+              return
+            end
+
+            unless first_obj.nil?
+              raise Exceptions::RookNonPrimitiveObjectType, first_obj.class.to_s unless
+                PRIMITIVES.include? first_obj.class
+            end
+
+            return if second_obj.nil?
+            raise Exceptions::RookNonPrimitiveObjectType, second_obj.class.to_s unless
+              PRIMITIVES.include? second_obj.class
+          end
+
           def execute_operation first, second
             # Remove  wrapping of objects as needed
             first_obj = decapsulate_item first
@@ -166,6 +182,8 @@ module Rookout
               raise Exceptions::RookExceptionEvaluationFailed.new "", e
             end
 
+            check_result result, first_obj, second_obj
+
             # If we don't have a result
             if result.nil?
               # Verify objects are primitives

data/lib/rookout/processor/rook_error.rb

@@ -24,13 +24,16 @@ module Rookout
         end
       end
 
-      attr_reader :message
+      attr_reader :message, :exception, :parameters
 
       def dumps
         parameters = NamespaceSerializer.dump Namespaces::RubyObjectNamespace.new(@parameters), false
         exception = NamespaceSerializer.dump Namespaces::RubyObjectNamespace.new(@exception), false
 
-        backtrace_string = @exception.backtrace.join "\n\t"
+        backtrace_string = ""
+        unless @exception.backtrace.nil?
+          backtrace_string = @exception.backtrace.join "\n\t"
+        end
         backtrace_object = Namespaces::RubyObjectNamespace.new(backtrace_string).tailor_limits!
         traceback = NamespaceSerializer.dump backtrace_object, false
 

data/lib/rookout/services/position.rb

@@ -102,13 +102,16 @@ module Rookout
         elsif suggested_match? location, filename
           warning = Exceptions::RookSourceFilePathSuggestion.new location.filename, filename
           location.notify_warning warning
+          # Must return nil in order to skip this script
+          nil
         end
       end
 
       #########################################################################################
       # Utils
       def exact_match? location_filename, script_filename
-        script_filename.end_with? location_filename
+        return unless script_filename.end_with? location_filename
+        File.basename(script_filename) == File.basename(location_filename)
       end
 
       def suggested_match? location, filename
@@ -122,6 +125,31 @@ module Rookout
         Digest::SHA2.new(256).hexdigest content
       end
 
+      def handle_unique_line lines, location, filename
+        first_line = nil
+        second_found = false
+
+        lines.each_with_index do |line, index|
+          if crc_line(line) == location.line_crc
+            if first_line.nil?
+              first_line = index
+            else
+              second_found = true
+              break
+            end
+          end
+        end
+
+        if first_line && !second_found
+          updated_line = first_line + 1
+          location.notify_warning Processor::RookError.new Exceptions::RookLineMoved.new(filename,
+                                                                                         location.lineno,
+                                                                                         updated_line)
+          return updated_line
+        end
+        nil
+      end
+
       def find_updated_line filename, location
         return location.lineno if location.line_crc.nil?
 
@@ -130,23 +158,8 @@ module Rookout
         return location.lineno if location.line_crc == line_crc32
 
         if location.line_unique
-          first_line = nil
-          second_found = false
-
-          lines.each_with_index do |line, index|
-            if crc_line(line) == location.line_crc
-              if first_line.nil?
-                first_line = index
-              else
-                second_found = true
-                break
-              end
-            end
-          end
-
-          if first_line && !second_found
-            updated_line = first_line + 1
-            location.notify_warning Exceptions::RookLineMoved.new(filename, location.lineno, updated_line)
+          updated_line = handle_unique_line lines, location, filename
+          unless updated_line.nil?
             return updated_line
           end
         end

data/lib/rookout/version.rb

@@ -1,3 +1,3 @@
 module Rookout
-  VERSION = "0.1.7".freeze
+  VERSION = "0.1.12".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rookout
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.12
 platform: ruby
 authors:
 - Liran Haimovitch
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-13 00:00:00.000000000 Z
+date: 2020-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: binding_of_caller
@@ -53,33 +53,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.5.0
 - !ruby/object:Gem::Dependency
-  name: google-protobuf
+  name: event_emitter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: 0.2.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: 0.2.6
 - !ruby/object:Gem::Dependency
-  name: event_emitter
+  name: google-protobuf
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.6
+        version: 3.9.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.6
+        version: 3.9.2
 - !ruby/object:Gem::Dependency
   name: google-style
   requirement: !ruby/object:Gem::Requirement
@@ -253,7 +253,7 @@ licenses:
 - Proprietary
 metadata:
   homepage_uri: https://rookout.com
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -268,8 +268,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key:
+rubygems_version: 3.1.2
+signing_key: 
 specification_version: 4
 summary: rookout is the Ruby SDK for the Rookout Debugging Platform
 test_files: []