Remove implementation-focused tests per review

test/test_util.py

@@ -50,13 +50,6 @@ class UtilTest(unittest.TestCase):
         util.interactive_open(["foo"], "bar")
         mock_execlp.assert_called_once_with("bar", "bar", "foo")
 
-    @patch("os.execlp")
-    def test_interactive_open_windows_start_command(self, mock_execlp):
-        util.interactive_open(["foo"], 'cmd /c start ""')
-        mock_execlp.assert_called_once_with(
-            "cmd", "cmd", "/c", "start", "", "foo"
-        )
-
     def test_sanitize_unix_replaces_leading_dot(self):
         with _common.platform_posix():
             p = util.sanitize_path("one/.two/three")

test/ui/commands/test_config.py

@@ -5,7 +5,6 @@ import pytest
 import yaml
 
 from beets import config, ui
-from beets.test import _common
 from beets.test.helper import BeetsTestCase, IOMixin
 
 
@@ -111,22 +110,6 @@ class ConfigCommandTest(IOMixin, BeetsTestCase):
             "please_open", "please_open", self.config_path
         )
 
-    def test_edit_config_with_windows_default_open(self):
-        with (
-            _common.system_mock("Windows"),
-            patch("os.execlp") as execlp,
-        ):
-            self.run_command("config", "-e")
-
-        execlp.assert_called_once_with(
-            "cmd",
-            "cmd",
-            "/c",
-            "start",
-            "",
-            self.config_path,
-        )
-
     def test_config_editor_not_found(self):
         msg_match = "Could not edit configuration.*here is problem"
         with (