moto 0.9.17 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7fd7651cf81d31a19644da80e0837649daa58e57
-  data.tar.gz: a3ab1acad8e60e01cfb3ea2a40e2738226addb8a
+  metadata.gz: d2216e77b4bfad6cd5598a147586ed6567887fa0
+  data.tar.gz: d8fc5dc3f9cbc7d86c6798ddf0d38225ed556cd9
 SHA512:
-  metadata.gz: 8eca0bcbda114f618032f367494e1c53fa7d7fb9bfd5153baf40de7d8e487ec5889cb4a5c07f2f1e110711a122dd162c4c7294ffe3bdadd890c46befc5a7e281
-  data.tar.gz: 0e33fcc4ebbe3efcc9ca3142e00f70732edddf4c0f04d0516c4454b53dfbcc4ee366668462bcac0a2573c4d54478952f69ef7f3fd58dfc9b81e2aa2bb0847995
+  metadata.gz: 70c8d3fe2876552c6d14f8bdb8c547f68cde88c42fa791b184cbc72136d838adfb570899d2f39b3ba3ea81905acfcc25d1d519a72df7ac25613e8b99e5163b92
+  data.tar.gz: c2fd5dbd5e26d38b14f608d64ddc7cfea677d6990f7f46781db86ce88369368b90f214e0ea2d8c1296383a20cb3075a9f19dced7be6c1fb818bfb69a7f6d6479

data/bin/moto

@@ -1,6 +1,5 @@
 #!/usr/bin/env ruby
 
-require_relative '../lib/test_logging'
 require_relative '../lib/test/base'
 
 require_relative '../lib/reporting/listeners/base'
@@ -10,4 +9,5 @@ require_relative '../lib/reporting/listeners/junit_xml'
 require_relative '../lib/reporting/listeners/webui'
 
 require_relative '../lib/parameter_parser'
+
 Moto::ParameterParser.parse_user_input(ARGV)

data/lib/modes/run/thread_context.rb

@@ -18,10 +18,7 @@ module Moto
             FileUtils.mkdir_p(log_directory)
           end
 
-          file = File.open(@test.log_path, File::WRONLY | File::TRUNC | File::CREAT)
-          file.chmod(0o666)
-          Thread.current['logger'] = Logger.new(file)
-          Thread.current['logger'].level = config[:test_log_level] || Logger::DEBUG
+          setup_logger
         end
 
         def run
@@ -34,20 +31,25 @@ module Moto
           (1..max_attempts).each do |attempt|
 
             @test.before
-            Thread.current['logger'].info("Start: #{@test.name} attempt #{attempt}/#{max_attempts}")
+            logger.info("Start: #{@test.name} attempt #{attempt}/#{max_attempts}")
 
             begin
               @test.run_test
             rescue Exceptions::TestForcedPassed, Exceptions::TestForcedFailure, Exceptions::TestSkipped => e
-              Thread.current['logger'].info(e.message)
+              logger.info(e.message)
             rescue Exception => e
-              Thread.current['logger'].error("#{e.class.name}: #{e.message}")
-              Thread.current['logger'].error(e.backtrace.join("\n"))
+              logger.error("#{e.class.name}: #{e.message}")
+              logger.error(e.backtrace.join("\n"))
+
+              if config[:explicit_errors]
+                raise e
+              end
+
             end
 
             @test.after
 
-            Thread.current['logger'].info("Result: #{@test.status.results.last.code}")
+            logger.info("Result: #{@test.status.results.last.code}")
 
             # test should have another attempt in case of an error / failure / none at all
             unless (@test.status.results.last.code == Moto::Test::Result::ERROR && config[:test_reattempt_on_error]) ||
@@ -63,7 +65,7 @@ module Moto
           end # Make another attempt
 
           # Close and flush stream to file
-          Thread.current['logger'].close
+          logger.close
 
           # Reporting: end_test
           @test_reporter.report_end_test(@test.status)
@@ -75,6 +77,18 @@ module Moto
         end
         private :config
 
+        def setup_logger
+          file = File.open(@test.log_path, File::WRONLY | File::TRUNC | File::CREAT)
+          file.chmod(0o666)
+
+          Thread.current['logger'] = Logger.new(file)
+          logger.level = config[:test_log_level] || Logger::DEBUG
+        end
+
+        def logger
+          Thread.current['logger']
+        end
+
       end
     end
   end

data/lib/parameter_parser.rb

@@ -65,6 +65,7 @@ module Moto
         opts.on('--stop-on-fail') {options[:stop_on][:fail] = true}
         opts.on('--stop-on-skip') {options[:stop_on][:skip] = true}
         opts.on('--dry-run') {options[:dry_run] = true}
+        opts.on('-x', '--explicit_errors') {options[:explicit_errors] = true}
       end.parse!
 
       if options[:tests]
@@ -89,6 +90,7 @@ module Moto
       Moto::Lib::Config.moto[:test_runner][:thread_count] = options[:threads] if options[:threads]
       Moto::Lib::Config.moto[:test_runner][:test_attempt_max] = options[:attempts] if options[:attempts]
       Moto::Lib::Config.moto[:test_runner][:dry_run] = options[:dry_run] if options[:dry_run]
+      Moto::Lib::Config.moto[:test_runner][:explicit_errors] = options[:explicit_errors] if options[:explicit_errors]
 
       return options
     end
@@ -220,6 +222,11 @@ module Moto
        --stop-on-skip    Moto will stop test execution when a skip is encountered in test results
        --dry-run         Moto will list all test cases which would be run with provided arguments
 
+       --x, explicit_errors   Use for development of tests - each code error in test will be caught,
+                              logged as would normally do but will be also additionaly re-thrown.
+                              This will obviously stop execution of selected set of tests but will provide
+                              full stack and error message to the developer.
+
 
 
       ==============

data/lib/reporting/listeners/console_dots.rb

@@ -1,8 +1,15 @@
+require 'thread'
+
 module Moto
   module Reporting
     module Listeners
       class ConsoleDots < Base
 
+        def initialize(run_params)
+          @displayed_results = 0
+          @semaphore = Mutex.new
+        end
+
         def end_run(run_status)
           puts ''
           puts ''
@@ -46,15 +53,25 @@ module Moto
         end
 
         def end_test(test_status)
-          print case test_status.results.last.code
-          when Moto::Test::Result::PASSED   then '.'
-          when Moto::Test::Result::FAILURE  then 'F'
-          when Moto::Test::Result::ERROR    then 'E'
-          when Moto::Test::Result::SKIPPED  then 's'
+          @semaphore.synchronize do
+            @displayed_results += 1
+
+            representation =
+                case test_status.results.last.code
+                  when Moto::Test::Result::PASSED   then '.'
+                  when Moto::Test::Result::FAILURE  then 'F'
+                  when Moto::Test::Result::ERROR    then 'E'
+                  when Moto::Test::Result::SKIPPED  then 's'
+                end
+
+            if @displayed_results%50 != 0
+              print representation
+            else
+              puts representation
+            end
+
           end
-          STDOUT.flush
         end
-
       end
     end
   end

data/lib/test/base.rb

@@ -1,4 +1,5 @@
 require_relative 'status'
+require 'logger'
 
 module Moto
   module Test
@@ -140,7 +141,7 @@ module Moto
           line_number = caller.select { |l| l.match(/#{static_path}:\d*:in `run'/) }.first[/\d+/].to_i
 
           status.log_failure("ASSERTION FAILED in line #{line_number}: #{message}")
-          Thread.current['logger'].error(message)
+          logger.error(message)
         end
       end
 
@@ -151,6 +152,11 @@ module Moto
         Moto::Lib::Config.environment_const(key)
       end
 
+      # @return [Logger]
+      def logger
+        Thread.current['logger']
+      end
+
     end
   end
 end

data/lib/version.rb

@@ -1,3 +1,3 @@
 module Moto
-  VERSION = '0.9.17'
+  VERSION = '1.0.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: moto
 version: !ruby/object:Gem::Version
-  version: 0.9.17
+  version: 1.0.0
 platform: ruby
 authors:
 - Bartek Wilczek
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-17 00:00:00.000000000 Z
+date: 2017-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -85,7 +85,6 @@ extra_rdoc_files: []
 files:
 - bin/moto
 - lib/config.rb
-- lib/empty_listener.rb
 - lib/exceptions/base.rb
 - lib/exceptions/test_forced_failure.rb
 - lib/exceptions/test_forced_passed.rb
@@ -105,14 +104,12 @@ files:
 - lib/reporting/listeners/webui.rb
 - lib/reporting/run_status.rb
 - lib/reporting/test_reporter.rb
-- lib/runner_logging.rb
 - lib/test/base.rb
 - lib/test/generator.rb
 - lib/test/metadata.rb
 - lib/test/metadata_generator.rb
 - lib/test/result.rb
 - lib/test/status.rb
-- lib/test_logging.rb
 - lib/version.rb
 homepage: https://github.com/bwilczek/moto
 licenses:

data/lib/empty_listener.rb

@@ -1,17 +0,0 @@
-module Moto
-  module EmptyListener
-
-    def start_run
-    end
-    
-    def end_run
-    end
-    
-    def start_test(test)
-    end
-
-    def end_test(test)
-    end    
-
-  end
-end

data/lib/runner_logging.rb

@@ -1,26 +0,0 @@
-module Moto
-  module RunnerLogging
-
-    # TODO: merge it somehow with TestLogging. Parametrize logger object?
-    def self.included(cls)
-      def cls.method_added(name)
-        excluded_methods = Moto::EmptyListener.instance_methods(false)
-        excluded_methods << :new
-        excluded_methods << :initialize
-        # TODO: configure more excluded classes/methods
-        return if @added 
-        @added = true # protect from recursion
-        original_method = "original_#{name}"
-        alias_method original_method, name
-        define_method(name) do |*args|
-          Thread.current['logger'].debug("#{self.class.name}::#{__callee__} ENTER >>> #{args}") unless excluded_methods.include? name
-          result = send original_method, *args
-          Thread.current['logger'].debug("#{self.class.name}::#{__callee__} LEAVE <<< #{result} ") unless excluded_methods.include? name
-          result
-        end
-        @added = false
-      end
-
-    end    
-  end
-end

data/lib/test_logging.rb

@@ -1,46 +0,0 @@
-module Moto
-  module TestLogging
-
-    @@ignore_logging = []
-
-    def self.included(cls)
-
-      def cls.ignore_logging(method)
-        full_name = "#{self.name}::#{method}"
-        @@ignore_logging << full_name
-      end
-      
-      def cls.method_added(name)
-
-        @@ignore_logging << "#{self.name}::new"
-        @@ignore_logging << "#{self.name}::initialize"
-
-        return if @added
-        @added = true # protect from recursion
-        original_method = "original_#{name}"
-        alias_method original_method, name
-        define_method(name) do |*args|
-          full_name = "#{self.class.name}::#{__callee__}"
-          # TODO: use self.class.ancestors to figure out if ancestor::__callee__ is not in @@ignore_logging
-          skip_logging = @@ignore_logging.include? full_name
-          unless skip_logging
-            self.class.ancestors.each do |a|
-              ancestor_name = "#{a.name}::#{__callee__}"
-              if @@ignore_logging.include? ancestor_name
-                skip_logging = true
-                break
-              end
-            end
-          end
-          Thread.current['logger'].debug("ENTER >>> #{self.class.name}::#{__callee__}(#{args})") unless skip_logging
-          result = send original_method, *args
-          # Below is the hack to properly escape binary data (if any manages to make it to logs)
-          Thread.current['logger'].debug("LEAVE <<< #{self.class.name}::#{__callee__} => #{[result].to_s[1..-2]}") unless skip_logging
-          result
-        end
-        @added = false
-      end
-    end
-
-  end
-end