test_launcher 1.5.0 → 1.5.1

data/test/test_launcher/queries/multi_term_query_test.rb

@@ -0,0 +1,138 @@
+require "test_helper"
+require "test_helpers/mocks"
+require "test_launcher/queries"
+
+module TestLauncher
+  module Queries
+    class MultiTermQueryTest < TestCase
+      include DefaultMocks
+
+      def searcher
+        @searcher ||= MockSearcher.new do |m|
+          m.impl :test_files do |search_string|
+            case search_string
+            when "not_found"
+              []
+            when "file_1"
+              ["file_1"]
+            when "file_2"
+              ["file_2"]
+            when "files"
+              ["file_3", "file_4"]
+            else
+              raise "can't find #{search_string}"
+            end
+          end
+        end
+      end
+
+      def create_mock_request(**attrs)
+        MockRequest.new(**attrs) do |m|
+          m.impl(:test_case) do |file:, example: nil, request:|
+            case file
+            when "file_1"
+              file_1_test_case
+            when "file_2"
+              file_2_test_case
+            when "file_3"
+              file_3_test_case
+            when "file_4"
+              file_4_test_case
+            else
+              raise "unmocked file: #{file}"
+            end
+          end
+        end
+      end
+
+      def file_1_test_case
+        @file_1_test_case ||= MockTestCase.new(file: "file_1", mtime: Time.now - 1)
+      end
+
+      def file_2_test_case
+        @file_2_test_case ||= MockTestCase.new(file: "file_2", mtime: Time.now - 2)
+      end
+
+      def file_3_test_case
+        @file_3_test_case ||= MockTestCase.new(file: "file_3", mtime: Time.now - 3)
+      end
+
+      def file_4_test_case
+        @file_4_test_case ||= MockTestCase.new(file: "file_4", mtime: Time.now - 4)
+      end
+
+
+      def test_command__finds_files_for_0_of_2_terms_asdf
+        request = create_mock_request(
+          search_string: "not_found not_found",
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher
+        )
+        command = MultiTermQuery.new(request, default_command_finder).command
+
+        assert_equal nil, command
+      end
+
+      def test_command__finds_files_for_0_of_2_terms__warns
+        request = create_mock_request(
+          search_string: "not_found not_found",
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher
+        )
+        command = MultiTermQuery.new(request, default_command_finder).command
+
+        assert_equal 1, default_shell.recall(:warn).size
+      end
+
+      def test_command__finds_files_for_1_of_2_terms
+        request = create_mock_request(
+          search_string: "file_1 not_found",
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher
+        )
+        command = MultiTermQuery.new(request, default_command_finder).command
+
+        assert_equal nil, command
+      end
+
+      def test_command__finds_files_for_1_of_2_terms__warns
+        request = create_mock_request(
+          search_string: "file_1 not_found",
+          shell: default_shell,
+          searcher: searcher
+        )
+        command = MultiTermQuery.new(request, default_command_finder).command
+
+        assert_equal 1, default_shell.recall(:warn).size
+      end
+
+      def test_command__finds_files_for_2_of_2_terms
+        request = create_mock_request(
+          search_string: "file_1 file_2",
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher,
+        )
+        command = MultiTermQuery.new(request, default_command_finder).command
+
+        assert_equal [[[file_1_test_case, file_2_test_case]]], default_runner.recall(:multiple_files)
+        assert_equal "multiple_files_return", command
+      end
+
+      def test_command__finds_files_for_2_of_2_terms__extra_files_found
+        request = create_mock_request(
+          search_string: "file_1 files",
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher,
+        )
+        command = MultiTermQuery.new(request, default_command_finder).command
+        assert_equal [[[file_1_test_case, file_3_test_case, file_4_test_case]]], default_runner.recall(:multiple_files)
+        assert_equal "multiple_files_return", command
+      end
+    end
+  end
+end

data/test/test_launcher/queries/path_query_test.rb

@@ -0,0 +1,192 @@
+require "test_helper"
+require "test_helpers/mocks"
+require "test_launcher/queries"
+
+module TestLauncher
+  module Queries
+    class PathQueryTest < TestCase
+      include DefaultMocks
+
+      def searcher
+        @searcher ||= MockSearcher.new do |m|
+          m.impl :test_files do |search_string|
+            case search_string
+            when "not_found"
+              []
+            when "one_file"
+              ["file_1"]
+            when "multiple_files"
+              ["file_2", "file_3", "file_4"]
+            else
+              raise "unmocked search_string: #{search_string}"
+            end
+          end
+        end
+      end
+
+      def create_mock_request(**attrs)
+        MockRequest.new(**attrs) do |m|
+          m.impl(:test_case) do |file:, request:|
+            case file
+            when "file_1"
+              normal_test_case
+            when "file_2"
+              old_test_case
+            when "file_3"
+              older_test_case
+            when "file_4"
+              new_test_case
+            else
+              raise "unmocked file: #{file}"
+            end
+          end
+        end
+      end
+
+      def normal_test_case
+        @normal_test_case ||= MockTestCase.new(file: "file_1")
+      end
+
+      def old_test_case
+        @old_test_case ||= MockTestCase.new(file: "file_2", mtime: Time.now - 1000)
+      end
+
+      def older_test_case
+        @older_test_case ||= MockTestCase.new(file: "file_3", mtime: Time.now - 10000)
+      end
+
+      def new_test_case
+        @new_test_case ||= MockTestCase.new(file: "file_4", mtime: Time.now)
+      end
+
+      def test_command__file_not_found__returns_nil
+        request = create_mock_request(
+          search_string: "not_found",
+          searcher: searcher
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert_equal nil, command
+      end
+
+      def test_command__file_not_found__does_not_warn
+        request = create_mock_request(
+          search_string: "not_found",
+          searcher: searcher
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert default_shell.recall(:warn).empty?
+      end
+
+      def test_command__one_file_found__runs_single_file
+        request = create_mock_request(
+          search_string: "one_file",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert_equal [[normal_test_case]], default_runner.recall(:single_file)
+        assert_equal "single_file_return", command
+      end
+
+      def test_command__one_file_found__has_correct_test_case
+        request = create_mock_request(
+          search_string: "one_file",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner
+        )
+
+        expected_test_case_args = [file: "file_1", request: request]
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert_equal [expected_test_case_args], request.recall(:test_case)
+      end
+
+      def test_command__one_file_found__notifies
+        request = create_mock_request(
+          search_string: "one_file",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert_equal [["Found 1 file."]], default_shell.recall(:notify)
+      end
+
+      def test_command__multiple_files_found__no_all__runs_single_file
+        request = create_mock_request(
+          search_string: "multiple_files",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner,
+          run_all?: false
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert_equal [[new_test_case]], default_runner.recall(:single_file)
+        assert_equal "single_file_return", command
+      end
+
+      def test_command__multiple_files_found__no_all__notifies
+        request = create_mock_request(
+          search_string: "multiple_files",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner,
+          run_all?: false
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        messages = [
+          ["Found 3 files."],
+          ["Running most recently edited. Run with '--all' to run all the tests."]
+        ]
+        assert_equal messages, default_shell.recall(:notify)
+      end
+
+      def test_command__multiple_files_found__all__runs_single_file
+        request = create_mock_request(
+          search_string: "multiple_files",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner,
+          run_all?: true
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        assert_equal [[[old_test_case, older_test_case, new_test_case]]], default_runner.recall(:multiple_files)
+        assert_equal "multiple_files_return", command
+      end
+
+      def test_command__multiple_files_found__all__notifies
+        request = create_mock_request(
+          search_string: "multiple_files",
+          shell: default_shell,
+          searcher: searcher,
+          runner: default_runner,
+          run_all?: true
+        )
+
+        command = PathQuery.new(request, default_command_finder).command
+
+        messages = [
+          ["Found 3 files."]
+        ]
+        assert_equal messages, default_shell.recall(:notify)
+      end
+    end
+  end
+end

data/test/test_launcher/queries/search_query_test.rb

@@ -0,0 +1,54 @@
+require "test_helper"
+require "test_helpers/mocks"
+require "test_launcher/queries"
+
+module TestLauncher
+  module Queries
+    class SearchQueryTest < TestCase
+      include DefaultMocks
+
+      def test__multi_search_term__hits
+        command_finder = Mock.new(Queries::CommandFinder, multi_search_term: :multi_search_term)
+        request = MockRequest.new(search_string: "a b")
+
+        assert_equal :multi_search_term, SearchQuery.new(request, command_finder).command
+      end
+
+      def test__multi_search_term__misses
+        command_finder = Mock.new(Queries::CommandFinder, multi_search_term: nil, single_search_term: :single_search_term)
+        request = MockRequest.new(search_string: "a b")
+
+        assert_equal :single_search_term, SearchQuery.new(request, command_finder).command
+      end
+
+      def test__line_number__hits
+        command_finder = Mock.new(Queries::CommandFinder, line_number: :line_number)
+        request = MockRequest.new(search_string: "a:1")
+
+        assert_equal :line_number, SearchQuery.new(request, command_finder).command
+      end
+
+      def test__line_number__misses
+        command_finder = Mock.new(Queries::CommandFinder, line_number: nil, single_search_term: :single_search_term)
+        request = MockRequest.new(search_string: "a:1")
+
+        assert_equal :single_search_term, SearchQuery.new(request, command_finder).command
+      end
+
+      def test__single_search_term
+        command_finder = Mock.new(Queries::CommandFinder, single_search_term: :single_search_term)
+        request = MockRequest.new(search_string: "a")
+
+        assert_equal :single_search_term, SearchQuery.new(request, command_finder).command
+      end
+
+      def test__none
+        command_finder = Mock.new(Queries::CommandFinder, single_search_term: nil)
+        request = MockRequest.new(search_string: "a")
+
+        assert_equal nil, SearchQuery.new(request, command_finder).command
+      end
+
+    end
+  end
+end

data/test/test_launcher/queries/single_term_query_test.rb

@@ -0,0 +1,36 @@
+require "test_helper"
+require "test_helpers/mocks"
+require "test_launcher/queries"
+
+module TestLauncher
+  module Queries
+    class SingleTermQueryTest < TestCase
+      include DefaultMocks
+
+      def test__by_path
+        command_finder = Mock.new(Queries::CommandFinder, by_path: :by_path)
+
+        assert_equal :by_path, SingleTermQuery.new(MockRequest.new, command_finder).command
+      end
+
+      def test__example_name
+        command_finder = Mock.new(Queries::CommandFinder, by_path: nil, example_name: :example_name)
+
+        assert_equal :example_name, SingleTermQuery.new(MockRequest.new, command_finder).command
+      end
+
+      def test__from_full_regex
+        command_finder = Mock.new(Queries::CommandFinder, by_path: nil, example_name: nil, from_full_regex: :from_full_regex)
+
+        assert_equal :from_full_regex, SingleTermQuery.new(MockRequest.new, command_finder).command
+      end
+
+      def test__none
+        command_finder = Mock.new(Queries::CommandFinder, by_path: nil, example_name: nil, from_full_regex: nil)
+
+        assert_equal nil, SingleTermQuery.new(MockRequest.new, command_finder).command
+      end
+
+    end
+  end
+end

data/test/test_launcher/queries/specified_name_query_test.rb

@@ -0,0 +1,112 @@
+require "test_helper"
+require "test_helpers/mocks"
+require "test_launcher/queries"
+
+module TestLauncher
+  module Queries
+    class SpecifiedNameQueryTest < TestCase
+      include DefaultMocks
+
+      def searcher
+        @searcher ||= MockSearcher.new do |m|
+          m.impl :test_files do |search_string|
+            case search_string
+            when "not_found"
+              []
+            when "multiple_matches"
+              ["file_1", "file_2"]
+            when "exact_match"
+              ["matched_file.rb"]
+            else
+              raise "can't find #{search_string}"
+            end
+          end
+        end
+      end
+
+      def test_command__file_not_found__returns_no_command
+        request = MockRequest.new(
+          search_string: "not_found",
+          shell: default_shell,
+          searcher: searcher
+        )
+        query = SpecifiedNameQuery.new(request, MockCommandFinder.new)
+
+        assert_equal nil, query.command
+      end
+
+      def test_command__file_not_found__warns
+        request = MockRequest.new(
+          search_string: "not_found",
+          shell: default_shell,
+          searcher: searcher
+        )
+        SpecifiedNameQuery.new(request, MockCommandFinder.new).command
+
+        assert_equal 1, default_shell.recall(:warn).size
+      end
+
+      def test_command__multiple_files_found
+        request = MockRequest.new(
+          search_string: "multiple_matches",
+          shell: default_shell,
+          searcher: searcher
+        )
+        query = SpecifiedNameQuery.new(request, MockCommandFinder.new)
+
+        assert_equal nil, query.command
+      end
+
+      def test_command__multiple_files_found__warns
+        request = MockRequest.new(
+          search_string: "multiple_matches",
+          shell: default_shell,
+          searcher: searcher
+        )
+        SpecifiedNameQuery.new(request, MockCommandFinder.new).command
+
+        assert_equal 1, default_shell.recall(:warn).size
+      end
+
+      def test_command__exact_match__command_is_single_example
+        request = MockRequest.new(
+          search_string: "exact_match",
+          example_name: "example_name",
+          test_case: default_test_case,
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher
+        ) do |m|
+          m.impl(:test_case) { default_test_case }
+        end # TODO: hack while implementing new searcher mocking
+        command = SpecifiedNameQuery.new(request, MockCommandFinder.new).command
+
+        expected_runner_args = [
+          default_test_case,
+        ]
+        assert_includes default_runner.recall(:single_example), expected_runner_args
+        assert_equal "single_example_return", command
+      end
+
+      def test_command__exact_match__creates_example_test_case
+        request = MockRequest.new(
+          search_string: "exact_match",
+          example_name: "example_name",
+          test_case: default_test_case,
+          shell: default_shell,
+          runner: default_runner,
+          searcher: searcher
+        )
+        command = SpecifiedNameQuery.new(request, MockCommandFinder.new).command
+
+        expected_test_case_args = [
+          file: "matched_file.rb",
+          example: "example_name",
+          request: request
+        ]
+
+        assert_includes request.recall(:test_case), expected_test_case_args
+      end
+    end
+  end
+end

data/test/test_launcher/rspec_integration_test.rb

@@ -1,83 +1,69 @@
 require "test_helper"
-require "test_launcher/request"
+require "test_helpers/integration_helper"
 
 module TestLauncher
   class RspecIntegrationTest < TestCase
+    include IntegrationHelper
+
     def test__single_method
-      launch("file_name_1 example_name_""1")
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example file_name_1\\ example_name_1", Shell::Runner.recall_exec
+      launch("file_name_1 example_name_""1", framework: "rspec")
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example file_name_1\\ example_name_1", shell_mock.recall_exec
     end
 
     def test__single_context
-      launch("file_name_1 con""text_1")
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example file_name_1\\ context_1", Shell::Runner.recall_exec
+      launch("file_name_1 con""text_1", framework: "rspec")
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example file_name_1\\ context_1", shell_mock.recall_exec
     end
 
     def test__single_describe
-      launch("Root1DummyTes""tClass1")
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example Root1DummyTes""tClass1", Shell::Runner.recall_exec
+      launch("Root1DummyTes""tClass1", framework: "rspec")
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example Root1DummyTes""tClass1", shell_mock.recall_exec
     end
 
     def test__multiple_methods__same_file
-      launch("file_name_1")
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example file_name_1", Shell::Runner.recall_exec
+      launch("file_name_1", framework: "rspec")
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} --example file_name_1", shell_mock.recall_exec
     end
 
     def test__multiple_methods__different_files
-      launch("multiple_files same_example")
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")} --example multiple_files\\ same_example", Shell::Runner.recall_exec
+      launch("multiple_files same_example", framework: "rspec")
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")} --example multiple_files\\ same_example", shell_mock.recall_exec
     end
 
     def test__single_file
-      launch("class_1_spec")
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")}", Shell::Runner.recall_exec
+      launch("class_1_spec", framework: "rspec")
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")}", shell_mock.recall_exec
     end
 
     def test__multiple_files__found
-      launch("Root1") # don't trigger the find in *this* file
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")} --example Roo""t1", Shell::Runner.recall_exec
+      launch("Root1", framework: "rspec") # don't trigger the find in *this* file
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")} --example Roo""t1", shell_mock.recall_exec
     end
 
     def test__multiple_files__found__all
-      launch("Root1""DummyTest""Class", run_all: true) # don't trigger the find in *this* file
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}", Shell::Runner.recall_exec
+      launch("Root1""DummyTest""Class", run_all: true, framework: "rspec") # don't trigger the find in *this* file
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}", shell_mock.recall_exec
     end
 
     def test__multiple_file_paths
-      launch("class_1_spec.rb class_2_spec.rb") # don't trigger the find in *this* file
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}", Shell::Runner.recall_exec
+      launch("class_1_spec.rb class_2_spec.rb", framework: "rspec") # don't trigger the find in *this* file
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}", shell_mock.recall_exec
     end
 
     def test__multiple_files__different_roots__all
-      launch("DummyTest""Class", run_all: true) # don't trigger the find in *this* file
+      launch("DummyTest""Class", run_all: true, framework: "rspec") # don't trigger the find in *this* file
       expected = "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_1_spec.rb")} #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}; cd -;\n\ncd #{system_path("test/test_launcher/fixtures/rspec/spec/different_root")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/different_root/spec/different_root_spec.rb")}"
-      assert_equal expected, Shell::Runner.recall_exec
+      assert_equal expected, shell_mock.recall_exec
     end
 
     def test__regex
-      launch("a_test_that_u""ses") # don't trigger the find in *this* file
-      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}", Shell::Runner.recall_exec
+      launch("a_test_that_u""ses", framework: "rspec") # don't trigger the find in *this* file
+      assert_equal "cd #{system_path("test/test_launcher/fixtures/rspec")} && rspec #{system_path("test/test_launcher/fixtures/rspec/spec/class_2_spec.rb")}", shell_mock.recall_exec
     end
 
     def test__not_found
-      launch("not_found""thing")
-      assert_equal nil, Shell::Runner.recall_exec
-    end
-
-    private
-
-    def system_path(relative_dir)
-      File.join(Dir.pwd, relative_dir)
-    end
-
-    def launch(query, run_all: false)
-      request = Request.new(
-        query: query,
-        run_all: run_all,
-        framework: "rspec"
-      )
-
-      TestLauncher.launch(request)
+      launch("not_found""thing", framework: "rspec")
+      assert_equal nil, shell_mock.recall_exec
     end
   end
 end

data/test/test_launcher/search/git_test.rb

@@ -1,9 +1,11 @@
 require "test_helper"
 require "test_launcher/search/git"
+require "test_helpers/mocks"
 
 module TestLauncher
   module Search
     class GitTest < TestCase
+      include DefaultMocks
 
       def setup
         super