Add caret > to nudge the user to put a number

AlexB52 · Feb 25, 2024 · c10dd5a · c10dd5a
1 parent bf6fec4
commit c10dd5a
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 4 deletions.
diff --git a/features/ruby-app/retest/retest_test/matching_unmatching_command_test.rb b/features/ruby-app/retest/retest_test/matching_unmatching_command_test.rb
@@ -55,7 +55,7 @@ def test_displaying_options_on_matching_command
     assert_match "Test File Selected: foo_test.rb", stdout.tap(&:rewind).read
 
     modify_file('lib/bottles.rb')
-    assert_match <<~EXPECTED, stdout.tap(&:rewind).read
+    assert_match <<~EXPECTED.chomp, stdout.tap(&:rewind).read
       We found few tests matching: lib/bottles.rb
 
       [0] - test/bottles_test.rb
@@ -64,6 +64,7 @@ def test_displaying_options_on_matching_command
 
       Which file do you want to use?
       Enter the file number now:
+      > 
     EXPECTED
 
     assert_match "Test File Selected: foo_test.rb", stdout.tap(&:rewind).read

diff --git a/lib/retest/prompt.rb b/lib/retest/prompt.rb
@@ -29,6 +29,7 @@ def ask_which_test_to_use(path, files)
         Which file do you want to use?
         Enter the file number now:
       QUESTION
+      output.print("> ")
 
       options.values[input.gets.chomp.to_i]
     end

diff --git a/test/retest/prompt_test.rb b/test/retest/prompt_test.rb
@@ -33,7 +33,7 @@ def test_ask_which_test_to_use
       result = @subject.ask_which_test_to_use("app/models/valuation/holdings.rb", files)
 
       assert_equal "test/models/schedule/holdings_test.rb", result
-      assert_equal <<~EXPECTED, @subject.read_output
+      assert_equal <<~EXPECTED.chomp, @subject.read_output
         We found few tests matching: app/models/valuation/holdings.rb
 
         [0] - test/models/taxation/holdings_test.rb
@@ -45,6 +45,7 @@ def test_ask_which_test_to_use
 
         Which file do you want to use?
         Enter the file number now:
+        > 
       EXPECTED
     end
 

diff --git a/test/retest/repository/multiple_test_files_with_user_input.rb b/test/retest/repository/multiple_test_files_with_user_input.rb
@@ -18,7 +18,7 @@ def test_find_test_user_input_question
 
       @subject.find_test('app/models/billing_agent_customer.rb')
 
-      assert_equal <<~EXPECTED, @subject.prompt.read_output
+      assert_equal <<~EXPECTED.chomp, @subject.prompt.read_output
         We found few tests matching: app/models/billing_agent_customer.rb
 
         [0] - spec/models/billing_agent_customer_spec.rb
@@ -27,6 +27,7 @@ def test_find_test_user_input_question
 
         Which file do you want to use?
         Enter the file number now:
+        > 
       EXPECTED
     end
 

diff --git a/test/retest/repository_test.rb b/test/retest/repository_test.rb
@@ -131,7 +131,7 @@ def test_find_test_similar_files_but_no_exact_match
 
       out, _ = capture_subprocess_io { @subject.find_test('app/models/valuation/holdings.rb') }
 
-      assert_match <<~EXPECTED, out
+      assert_match <<~EXPECTED.chomp, out
         We found few tests matching: app/models/valuation/holdings.rb
 
         [0] - test/models/taxation/holdings_test.rb
@@ -143,6 +143,7 @@ def test_find_test_similar_files_but_no_exact_match
 
         Which file do you want to use?
         Enter the file number now:
+        > 
       EXPECTED
     end