tldr 0.9.1 → 0.9.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f6d49bfa66698f5e33451a20655ab442e6a6e50e76ec3b048b49d93baf7d8c66
-  data.tar.gz: f64c38ecd633686bc7616d4cf377fd49d2d91387dadaad09d81bf960ae2c39b6
+  metadata.gz: 0c87b13131b7bdbd2307b55c7cc4f60dd9033f086063786aed52f3dbe9be0516
+  data.tar.gz: b4d5f7589dee0f5f94b585d930c8d225beb10ba42c3dad22c5321e57b80e96ec
 SHA512:
-  metadata.gz: 524382d1fe9004b821bc7c7ea9b33779221ed60b35f51cf78924412c15a2c3cf07bb2ad46f8d5815f0890f3bd2ccfec868310521b00def3196ab93069e9153b0
-  data.tar.gz: 11313f640b6f2443063dbd79083f6810dca5778411c8093a36ebd5284b306c3db6be9dbc9e7bf53c4f63500734075371051243e75a665fb7e0e952ce28fdbe23
+  metadata.gz: 1a5196a70159c64afc23eff6278b310fa44c1ed5c86a50a9aecc55229309c10d46f20aedd7fb04e0f743053b53859b80a09509cd2c56ca8f359532e98bcfd40d
+  data.tar.gz: 9d31131b6b72e80c4ff1c81e2e13bbb7e942d25adaf0d3e7524032ff293d3a94458886320f2b3f3ab6c924f8fdd2ae83acab87e0f45cf4e487a39ecf04faed52

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+* Print how many tests ran vs. didn't even when suppressing TLDR summary
+
+## [0.9.2]
+
+* Don't redundantly print out dotfile config values for re-run instructions
+
 ## [0.9.1]
 
 * Correctly clear the screen between runs

data/lib/tldr/reporters/default.rb

@@ -39,7 +39,7 @@ class TLDR
         @err.print "\n\n"
 
         if @config.yes_i_know
-          @err.print "🚨 TLDR! Display summary by omitting --yes-i-know"
+          @err.print "🚨 TLDR after completing #{test_results.size} of #{planned_tests.size} tests! Print full summary by omitting --yes-i-know"
         else
           wrap_in_horizontal_rule do
             @err.print [
@@ -121,7 +121,7 @@ class TLDR
           "#{index + 1}) #{describe(result.test, result.relevant_location)} #{result.failure? ? "failed" : "errored"}:",
           result.error.message.chomp,
           "\n  Re-run this test:",
-          "    #{tldr_command} #{@config.to_single_path_args(result.test.location.locator)}",
+          "    #{tldr_command} #{@config.to_single_path_args(result.test.location.locator, exclude_dotfile_matches: true)}\n",
           (TLDR.filter_backtrace(result.error.backtrace).join("\n") if @config.verbose)
         ].compact.reject(&:empty?).join("\n").strip
       end
@@ -160,7 +160,7 @@ class TLDR
         ].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    ")}
+            #{tldr_command} #{@config.to_full_args(exclude: [:paths], exclude_dotfile_matches: true)} #{suggested_locators.join(" \\\n    ")}
         MSG
       end
 

data/lib/tldr/value/config.rb

@@ -142,14 +142,14 @@ class TLDR
       }.compact
     end
 
-    def to_full_args exclude: [], ensure_args: []
+    def to_full_args exclude: [], ensure_args: [], exclude_dotfile_matches: false
       argv = to_cli_argv(
-        CONFLAGS.keys -
-        exclude - [
+        CONFLAGS.keys - exclude - [
           (:seed unless seed_set_intentionally),
           :watch,
           :i_am_being_watched
-        ]
+        ],
+        exclude_dotfile_matches:
       )
 
       ensure_args.each do |arg|
@@ -159,19 +159,20 @@ class TLDR
       argv.join(" ")
     end
 
-    def to_single_path_args path
+    def to_single_path_args path, exclude_dotfile_matches: false
       argv = to_cli_argv(CONFLAGS.keys - [
         :seed, :parallel, :names, :fail_fast, :paths, :prepend_paths,
         :no_prepend, :exclude_paths, :watch, :i_am_being_watched
-      ])
+      ], exclude_dotfile_matches:)
 
       (argv + [stringify(:paths, path)]).join(" ")
     end
 
     private
 
-    def to_cli_argv options = CONFLAGS.keys
+    def to_cli_argv options = CONFLAGS.keys, exclude_dotfile_matches:
       defaults = Config.build_defaults(cli_defaults: true)
+      defaults = defaults.merge(dotfile_args) if exclude_dotfile_matches
       options.map { |key|
         flag = CONFLAGS[key]
 
@@ -239,23 +240,28 @@ class TLDR
     end
 
     def merge_dotfile_args args
-      return args if args[:no_dotfile] || !File.exist?(".tldr.yml")
-      require "yaml"
-
-      dotfile_args = YAML.load_file(".tldr.yml").transform_keys { |k| k.to_sym }
-      # Since we don't have shell expansion, we have to glob any paths ourselves
-      if dotfile_args.key?(:paths)
-        dotfile_args[:paths] = dotfile_args[:paths].flat_map { |path| Dir[path] }
-      end
-      # The argv parser normally does this:
-      if dotfile_args.key?(:reporter)
-        dotfile_args[:reporter] = Kernel.const_get(dotfile_args[:reporter])
-      end
-      if (invalid_args = dotfile_args.except(*CONFIG_ATTRIBUTES)).any?
-        raise Error, "Invalid keys in .tldr.yml file: #{invalid_args.keys.join(", ")}"
-      end
+      return args if args[:no_dotfile]
 
       dotfile_args.merge(args)
     end
+
+    def dotfile_args
+      return {} unless File.exist?(".tldr.yml")
+
+      require "yaml"
+      @dotfile_args ||= YAML.load_file(".tldr.yml").transform_keys { |k| k.to_sym }.tap do |dotfile_args|
+        # Since we don't have shell expansion, we have to glob any paths ourselves
+        if dotfile_args.key?(:paths)
+          dotfile_args[:paths] = dotfile_args[:paths].flat_map { |path| Dir[path] }
+        end
+        # The argv parser normally does this:
+        if dotfile_args.key?(:reporter)
+          dotfile_args[:reporter] = Kernel.const_get(dotfile_args[:reporter])
+        end
+        if (invalid_args = dotfile_args.except(*CONFIG_ATTRIBUTES)).any?
+          raise Error, "Invalid keys in .tldr.yml file: #{invalid_args.keys.join(", ")}"
+        end
+      end
+    end
   end
 end

data/lib/tldr/version.rb

@@ -1,3 +1,3 @@
 class TLDR
-  VERSION = "0.9.1"
+  VERSION = "0.9.3"
 end

data/lib/tldr/watcher.rb

@@ -5,14 +5,14 @@ class TLDR
       tldr_command = "#{"bundle exec " if defined?(Bundler)}tldr #{config.to_full_args(ensure_args: ["--i-am-being-watched"])}"
       command = "fswatch -o #{config.load_paths.reverse.join(" ")} | xargs -n1 -I{} #{tldr_command}"
 
-      puts <<~MSG
-
-        Watching for changes in #{config.load_paths.map(&:inspect).join(", ")}...
+      print <<~MSG.chomp
+        Waiting for changes in --load-path directories: #{config.load_paths.map(&:inspect).join(", ")}
 
         When a file changes, TLDR will run this command:
 
         $ #{tldr_command}
 
+        Watching...
       MSG
 
       exec command

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tldr
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.9.3
 platform: ruby
 authors:
 - Justin Searls