tldr 0.7.0 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3db38676fc9f10ee01f3332c6dfcafb5dc7c0b5405d9c687d13ba3c5b5be008e
-  data.tar.gz: 874cbd810ed0ebff0e2bc92bba4d703903c10beb51220817314a9e6528308a01
+  metadata.gz: 81e3d6734a718c5abbf80cfb088369d5a6bb89b81fc261020d97e34cbd6903bb
+  data.tar.gz: 1c4d0b5a53a948d868556b027f6fd6cdb4b9a96e67b459f0b2bf73e05a251d2f
 SHA512:
-  metadata.gz: c79d10782d1d15ba8bd782358ad1b45ecbbc7096542aba5d487fd97822a61163049247a6cff2e050de95226e112653418b5cc656278a0fdf55cd7c5b09ce5ccb
-  data.tar.gz: 429100174c63d3e9d917ef7a5b4f5ff14f105453c53a854a21369595272507b47e4629bd50edd067a6aaf46986f583f8327b9e4f66555e390440c86cd584c04b
+  metadata.gz: df9de67f0609adc2c5e0d2fc44a3bec44dd8a25a4ed0d128b8f1beaad6f45343dc9e5644e8d3b94b5bbe4db919f3becdf2e9a468a6eda0978669018135993d1f
+  data.tar.gz: 00e813b4947ef3d9e267e3ea4248def3254829b4b910719ad4cdd258cf809d751d7c5a539d527a708f9ff3471fe45fe24fc342d132f15948a56d7d5a56b38c7e

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## [0.8.0]
+
+* Add a `--yes-i-know` flag that will suppress the large warning when your test
+suite runs over the 1.8s limit
+
 ## [0.7.0]
 
 * Add a `tldt` alias for folks who have another executable named `tldr` on their

data/README.md

@@ -123,7 +123,8 @@ Usage: tldr [options] some_tests/**/*.rb some/path.rb:13 ...
         --no-emoji                   Disable emoji in the output
     -v, --verbose                    Print stack traces for errors
         --[no-]warnings              Print Ruby warnings (Default: true)
-        --comment COMMENT            No-op comment, used internally for multi-line execution instructions
+        --yes-i-know                 Suppress TLDR report when suite runs over 1.8s
+        --comment COMMENT            No-op; used for multi-line execution instructions
 ```
 
 After being parsed, all the CLI options are converted into a
@@ -336,12 +337,26 @@ with the `--reporter` command line option. It can be set to any fully-qualified
 class name that extends from
 [TLDR::Reporters::Base](/lib/tldr/reporters/base.rb).
 
+### I know my tests are over 1.8s, how do I suppress the huge output?
+
+Plenty of test suites are over 1.8s and having TLDR repeatedly print out the
+huge summary at the end of each test run can be distracting and make it harder
+to spot test failures. If you know your test suite is too slow, you can simply
+add the `--yes-i-know` flag
+
 ### What about mocking?
 
 TLDR is laser-focused on running tests, so it doesn't provide a built-in mocking
 facility. Might we interest you in a refreshing
 [mocktail](https://github.com/testdouble/mocktail), instead?
 
+### Contributing to TLDR
+
+If you want to submit PRs on this repo, please know that the code style is
+[Kirkland-style Ruby](https://mastodon.social/@searls/111137666157318482), where
+method definitions have parentheses omitted but parentheses are generally
+expected for method invocations.
+
 ## Acknowledgements
 
 Thanks to [George Sheppard](https://github.com/fuzzmonkey) for freeing up the

data/lib/tldr/argv_parser.rb

@@ -82,7 +82,11 @@ class TLDR
           options[:warnings] = warnings
         end
 
-        opts.on "--comment COMMENT", String, "No-op comment, used internally for multi-line execution instructions" do
+        opts.on CONFLAGS[:yes_i_know], "Suppress TLDR report when suite runs over 1.8s" do
+          options[:yes_i_know] = true
+        end
+
+        opts.on "--comment COMMENT", String, "No-op; used for multi-line execution instructions" do
           # See "--comment" in lib/tldr/reporters/default.rb for an example of how this is used internally
         end
       end.parse!(args)

data/lib/tldr/reporters/default.rb

@@ -31,29 +31,31 @@ class TLDR
         end
       end
 
-      def time_diff start, stop = Process.clock_gettime(Process::CLOCK_MONOTONIC, :microsecond)
-        ((stop - start) / 1000.0).round
-      end
-
       def after_tldr planned_tests, wip_tests, test_results
         stop_time = Process.clock_gettime Process::CLOCK_MONOTONIC, :microsecond
 
         @out.print @icons.tldr
         @err.print "\n\n"
-        wrap_in_horizontal_rule do
-          @err.print [
-            "too long; didn't run!",
-            "#{@icons.run} Completed #{test_results.size} of #{planned_tests.size} tests (#{((test_results.size.to_f / planned_tests.size) * 100).round}%) before running out of time.",
-            (<<~WIP.chomp if wip_tests.any?),
-              #{@icons.wip} #{plural wip_tests.size, "test was", "tests were"} cancelled in progress:
-              #{wip_tests.map { |wip_test| "  #{time_diff(wip_test.start_time, stop_time)}ms - #{describe(wip_test.test)}" }.join("\n")}
-            WIP
-            (<<~SLOW.chomp if test_results.any?),
-              #{@icons.slow} Your #{[10, test_results.size].min} slowest completed tests:
-              #{test_results.sort_by(&:runtime).last(10).reverse.map { |result| "  #{result.runtime}ms - #{describe(result.test)}" }.join("\n")}
-            SLOW
-            describe_tests_that_didnt_finish(planned_tests, test_results)
-          ].compact.join("\n\n")
+
+        if @config.yes_i_know
+          @err.print "🚨 TLDR! Display summary by omitting --yes-i-know"
+        else
+          wrap_in_horizontal_rule do
+            @err.print [
+              "too long; didn't run!",
+              "#{@icons.run} Completed #{test_results.size} of #{planned_tests.size} tests (#{((test_results.size.to_f / planned_tests.size) * 100).round}%) before running out of time.",
+              (<<~WIP.chomp if wip_tests.any?),
+                #{@icons.wip} #{plural wip_tests.size, "test was", "tests were"} cancelled in progress:
+                #{wip_tests.map { |wip_test| "  #{time_diff(wip_test.start_time, stop_time)}ms - #{describe(wip_test.test)}" }.join("\n")}
+              WIP
+              (<<~SLOW.chomp if test_results.any?),
+                #{@icons.slow} Your #{[10, test_results.size].min} slowest completed tests:
+                #{test_results.sort_by(&:runtime).last(10).reverse.map { |result| "  #{result.runtime}ms - #{describe(result.test)}" }.join("\n")}
+              SLOW
+              describe_tests_that_didnt_finish(planned_tests, test_results),
+              "🙈 Suppress this summary with --yes-i-know"
+            ].compact.join("\n\n")
+          end
         end
 
         after_suite test_results
@@ -102,6 +104,10 @@ class TLDR
 
       private
 
+      def time_diff start, stop = Process.clock_gettime(Process::CLOCK_MONOTONIC, :microsecond)
+        ((stop - start) / 1000.0).round
+      end
+
       def summarize_failures results
         failures = results.select { |result| result.failing? }
         return failures if failures.empty?
@@ -138,7 +144,7 @@ class TLDR
         rule = @icons.alarm + "=" * 20 + " ABORTED RUN " + "=" * 20 + @icons.alarm
         @err.print "#{rule}\n\n"
         yield
-        @err.print "\n\n#{rule}\n\n"
+        @err.print "\n\n#{rule}"
       end
 
       def describe_tests_that_didnt_finish planned_tests, test_results
@@ -150,7 +156,7 @@ class TLDR
         failed_locators = consolidate failed, exclude: unrun_locators
         suggested_locators = unrun_locators + [
           ("--comment \"Also include #{plural failed.size, "test"} that failed:\"" if failed_locators.any?)
-        ] + failed_locators
+        ].compact + failed_locators
         <<~MSG
           #{@icons.rock_on} Run the #{plural unrun.size, "test"} that didn't finish:
             #{tldr_command} #{@config.to_full_args exclude: [:paths]} #{suggested_locators.join(" \\\n    ")}

data/lib/tldr/value/config.rb

@@ -17,6 +17,7 @@ class TLDR
     base_path: "--base-path",
     no_dotfile: "--no-dotfile",
     warnings: "--[no-]warnings",
+    yes_i_know: "--yes-i-know",
     paths: nil
   }.freeze
 
@@ -26,7 +27,7 @@ class TLDR
     :paths, :seed, :no_helper, :verbose, :reporter,
     :helper_paths, :load_paths, :parallel, :names, :fail_fast, :no_emoji,
     :prepend_paths, :no_prepend, :exclude_paths, :exclude_names, :base_path,
-    :no_dotfile, :warnings,
+    :no_dotfile, :warnings, :yes_i_know,
     # Internal properties
     :config_intended_for_merge_only, :seed_set_intentionally, :cli_defaults
   ].freeze
@@ -64,7 +65,8 @@ class TLDR
         exclude_paths: [],
         exclude_names: [],
         base_path: nil,
-        warnings: true
+        warnings: true,
+        yes_i_know: false
       }
 
       if cli_defaults
@@ -101,7 +103,7 @@ class TLDR
       end
 
       # Booleans
-      [:no_helper, :verbose, :fail_fast, :no_emoji, :no_prepend, :warnings].each do |key|
+      [:no_helper, :verbose, :fail_fast, :no_emoji, :no_prepend, :warnings, :yes_i_know].each do |key|
         merged_args[key] = defaults[key] if merged_args[key].nil?
       end
 

data/lib/tldr/version.rb

@@ -1,3 +1,3 @@
 class TLDR
-  VERSION = "0.7.0"
+  VERSION = "0.8.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tldr
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Justin Searls
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-10-03 00:00:00.000000000 Z
+date: 2023-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: super_diff