test workflow 10

Author: Abhishek's Macbook Pro

.github/workflows/test.yaml

@@ -1,27 +1,3 @@
-# name: test
-
-# on: [push, pull_request]
-
-# jobs:
-#   run-tests:
-#     strategy:
-#       fail-fast: false
-#       matrix:
-#         st-version: [3, 4]
-#         os: ["ubuntu-latest", "macOS-latest", "windows-latest"]
-#     runs-on: ${{ matrix.os }}
-#     steps:
-#       - uses: actions/checkout@v2
-#       - uses: SublimeText/UnitTesting/actions/setup@v1
-#         with:
-#           sublime-text-version: ${{ matrix.st-version }}
-#       - uses: SublimeText/UnitTesting/actions/run-tests@v1
-#         with:
-#           coverage: true
-#           codecov-upload: true
-#       - uses: codecov/codecov-action@v3
-#         env:
-#           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
 name: test
 
 # on: [push, pull_request]
@@ -39,28 +15,15 @@ jobs:
         os: ["ubuntu-latest", "windows-latest"]
     runs-on: ${{ matrix.os }}
     steps:
-      # - name: Disable Cache
-      #   uses: actions/cache@v2
-      #   with:
-      #     path: |
-      #       # Specify paths to cache, but leave it empty to disable caching
-      #     key: ${{ runner.os }}-${{ hashFiles('**/*.lock') }}
       - uses: actions/checkout@v3
-      # - uses: codecov/codecov-action@v3
-      #   env:
-      #     CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
-      #   with:
-      #     files: ./coverage.xml
       - uses: SublimeText/UnitTesting/actions/setup@v1
         with:
           sublime-text-version: ${{ matrix.st-version }}
           package-name: ${{ env.PACKAGE }}
-      # - run: pip install coverage
-      # - run: sudo apt install at-spi2-core
-      #   shell: bash
-      # - run:
-      # https://github.com/wbond/package_control/releases/download/4.0.0-beta8/Package.Control.sublime-package
       - uses: SublimeText/UnitTesting/actions/run-tests@v1
+      - uses: codecov/codecov-action@v3
+        env:
+          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
         with:
-          coverage: true
-          codecov-upload: true
+          files: ./coverage.xml
+

tests/base.py

@@ -4,7 +4,6 @@
 
 class PyRockTestBase(DeferrableTestCase):
     def setUp(self):
-        sublime.log_commands(True)
         self.view = sublime.active_window().new_file(
             syntax='Packages/Python/Python.sublime-syntax'
         )
@@ -13,7 +12,6 @@ def setUp(self):
         self.sublime_settings.set("close_windows_when_empty", False)
 
     def tearDown(self):
-        sublime.log_commands(False)
         if self.view:
             self.view.set_scratch(True)
             self.view.window().focus_view(self.view)

tests/src/commands/test_import_symbol.py

@@ -36,10 +36,6 @@ def test_import_symbol(
         self.assertEqual(selected_text, "cmath")
 
         self.view.run_command("py_rock", args={"action": "import_symbol", "test": True})
-
-        # full_text = self.view.substr(sublime.Region(0, self.view.size()))
-        # self.assertEqual(full_text, "********")
-
         expected_import_statement = self.view.substr(
             self.view.find("import cmath", 0, flags=FindFlags.LITERAL)
         )
@@ -66,10 +62,6 @@ def test_multi_line_import(
         self.view.sel().add(region_to_select)
 
         self.view.run_command("py_rock", args={"action": "import_symbol", "test": True})
-
-        # full_text = self.view.substr(sublime.Region(0, self.view.size()))
-        # self.assertEqual(full_text, "*******1")
-
         expected_import_statement = self.view.substr(
             self.view.find("import cmath", 0, flags=FindFlags.LITERAL)
         )
@@ -110,10 +102,6 @@ def test_add_module_import_in_existing_import(
         self.view.sel().add(region_to_select)
 
         self.view.run_command("py_rock", args={"action": "import_symbol", "test": True})
-
-        # full_text = self.view.substr(sublime.Region(0, self.view.size()))
-        # self.assertEqual(full_text, "*******2")
-        
         expected_import_statement = self.view.substr(
             self.view.find(
                 "from cmath import sin, log10", 0, flags=FindFlags.LITERAL