Better partial summary if 100% test coverage.

Author: samuel-williams-shopify

lib/covered/partial_summary.rb

@@ -41,8 +41,11 @@ def print_coverage(terminal, coverage, before: 4, after: 4)
 		def call(wrapper, output = $stdout, **options)
 			terminal = self.terminal(output)
 			complete_files = []
+			partial_files_count = 0
 
 			statistics = self.each(wrapper) do |coverage|
+				partial_files_count += 1
+				
 				path = wrapper.relative_path(coverage.path)
 				terminal.puts ""
 				terminal.puts path, style: :path
@@ -66,8 +69,8 @@ def call(wrapper, output = $stdout, **options)
 			terminal.puts
 			statistics.print(output)
 
-			# Show information about files with 100% coverage
-			if complete_files.any?
+			# Only show information about files with 100% coverage if there were files with partial coverage shown above
+			if complete_files.any? && partial_files_count > 0
 				terminal.puts ""
 				if complete_files.size == 1
 					terminal.puts "1 file has 100% coverage and is not shown above:"

test/covered/partial_summary.rb

@@ -33,4 +33,41 @@
 
 		expect(io.string).to be(:include?, "               :\n")
 	end
+	
+	it "shows 100% coverage files when there are partial files" do
+		# Create a scenario with mixed coverage
+		partial_file = __FILE__
+		complete_file = File.join(__dir__, "../covered/summary.rb")
+		
+		# Mark partial coverage for this file
+		files.mark(partial_file, 1, 1)
+		files.mark(partial_file, 2, 0)  # uncovered line
+		
+		# Mark complete coverage for summary.rb
+		files.mark(complete_file, 1, 1)
+		files.mark(complete_file, 2, 1)
+		
+		summary.call(files, io)
+		
+		# Should show the message about 100% coverage files
+		expect(io.string).to be(:include?, "100% coverage and is not shown above:")
+		expect(io.string).to be(:include?, "summary.rb")
+	end
+	
+	it "does not show 100% coverage files when all files are 100%" do
+		# Create a scenario where all files have 100% coverage
+		file1 = __FILE__
+		file2 = File.join(__dir__, "../covered/summary.rb")
+		
+		# Mark complete coverage for both files
+		files.mark(file1, 1, 1)
+		files.mark(file1, 2, 1)
+		files.mark(file2, 1, 1)
+		files.mark(file2, 2, 1)
+		
+		summary.call(files, io)
+		
+		# Should NOT show the message about 100% coverage files (would be redundant)
+		expect(io.string).not.to be(:include?, "100% coverage and is not shown above:")
+	end
 end