From 2403d6c5611bd796074918d6342a06b24568e2cb Mon Sep 17 00:00:00 2001
From: Alessio Perugini <alessioper.98@gmail.com>
Date: Thu, 14 Sep 2023 14:47:36 +0200
Subject: [PATCH] remove legacy targets from TaskFile

---
 .github/workflows/test-go-task.yml |  6 ------
 Taskfile.yml                       | 24 ++++--------------------
 2 files changed, 4 insertions(+), 26 deletions(-)

diff --git a/.github/workflows/test-go-task.yml b/.github/workflows/test-go-task.yml
index f0bac8cb077..e11c59ba91e 100644
--- a/.github/workflows/test-go-task.yml
+++ b/.github/workflows/test-go-task.yml
@@ -154,11 +154,6 @@ jobs:
       - name: Run tests
         run: task go:test
 
-      - name: Run unit tests on the legacy package
-        # Run legacy tests on one platform only
-        if: runner.os == 'Linux'
-        run: task test-legacy
-
       - name: Upload coverage data to workflow artifact
         if: runner.os == 'Linux'
         uses: actions/upload-artifact@v3
@@ -167,7 +162,6 @@ jobs:
           name: ${{ env.COVERAGE_ARTIFACT }}
           path: |
             ./coverage_unit.txt
-            ./coverage_legacy.txt
 
   coverage-upload:
     runs-on: ubuntu-latest
diff --git a/Taskfile.yml b/Taskfile.yml
index 1373e1adde8..b7a4522d033 100755
--- a/Taskfile.yml
+++ b/Taskfile.yml
@@ -284,21 +284,11 @@ tasks:
       - task: go:build
 
   test:
-    desc: Run the full testsuite, `legacy` will be skipped
+    desc: Run the full testsuite
     cmds:
       - task: go:test
       - task: go:integration-test
 
-  test-legacy:
-    desc: Run tests for the `legacy` package
-    cmds:
-      - |
-        go test \
-          {{ default "-v -failfast" .GOFLAGS }} \
-          -coverprofile=coverage_legacy.txt \
-          ./legacy/... \
-          {{.TEST_LDFLAGS}}
-
   test-unit-race:
     desc: Run unit tests only with race condition detection
     cmds:
@@ -311,18 +301,12 @@ tasks:
           {{.TEST_LDFLAGS}}
 
   check:
-    desc: Check fmt and lint, `legacy` will be skipped
+    desc: Check fmt and lint
     cmds:
       - task: go:vet
       - task: go:lint
       - task: protoc:check
 
-  check-legacy:
-    desc: Check fmt and lint for the `legacy` package
-    cmds:
-      - test -z $(go fmt ./legacy/...)
-      - go vet ./legacy/...
-
   rpc-client:
     desc: Run the rpc client test routine (server must be already started)
     cmds:
@@ -376,10 +360,10 @@ tasks:
 vars:
   PROJECT_NAME: "arduino-cli"
   DIST_DIR: "dist"
-  # all modules of this project except for "legacy/..." module and integration test
+  # all modules of this project except for integration test
   DEFAULT_GO_PACKAGES:
     sh: |
-      echo $(cd {{default "./" .GO_MODULE_PATH}} && go list ./... | grep -v internal/integrationtest | grep -v legacy | tr '\n' ' ' || echo '"ERROR: Unable to discover Go packages"')
+      echo $(cd {{default "./" .GO_MODULE_PATH}} && go list ./... | grep -v internal/integrationtest | tr '\n' ' ' || echo '"ERROR: Unable to discover Go packages"')
   DEFAULT_INTEGRATIONTEST_GO_PACKAGES:
     sh: |
       echo $(cd {{default "./" .GO_MODULE_PATH}} && go list ./... | grep internal/integrationtest | tr '\n' ' ' || echo '"ERROR: Unable to discover Go packages"')