[skip changelog] Move integration tests to pytest (#564)

* move compilation tests to pytest

* moved core tests to pytest

* moved core install/upgrade/uninstall to pytest

* Added detected_boards fixture

Co-authored-by: Maurizio Branca <m.branca@arduino.cc>

* add upload tests, remove mocked ones

* use fixture

* docs + dont run go tests anymore

* remove debug prints, bump pytest to latest

* skip problematic test on win

Co-authored-by: Maurizio Branca <m.branca@arduino.cc>

Author: Massimiliano Pippi

CONTRIBUTING.md

@@ -60,6 +60,7 @@ see following paragraph):
 task test-integration
 ```
 ### Running only some tests
+
 By default, all tests from all go packages are run. To run only unit
 tests from one or more specific packages, you can set the TARGETS
 environment variable, e.g.:
@@ -75,11 +76,6 @@ Both can be combined as well, typically to run only a specific test:
 
     TEST_REGEX='^TestFindBoardWithFQBN$' TARGETS=./arduino/cores/packagemanager task test-unit
 
-For integration test, the same options are supported. Note that when not
-specified, `TEST_REGEX` defaults to "Integration" to select only
-integration tests, so if you specify a broader regex, this could cause
-non-integration tests to be run as well.
-
 ### Integration tests
 
 Being a command line interface, Arduino CLI is heavily interactive and it has to
@@ -95,7 +91,8 @@ assess the options are correctly understood and the output is what we expect.
 To run the full suite of integration tests you need an Arduino device attached
 to a serial port and a working Python environment. Chances are that you already
 have Python installed in your system, if this is not the case you can
-[download][3] the official distribution or use the package manager provided by your Operating System.
+[download][3] the official distribution or use the package manager provided by
+your Operating System.
 
 Some dependencies need to be installed before running the tests and to avoid
 polluting your global Python enviroment with dependencies that might be only
@@ -151,7 +148,8 @@ a list of items you can check before submitting a PR:
 * Maintain **clean commit history** and use **meaningful commit messages**.
   PRs with messy commit history are difficult to review and require a lot of
   work to be merged.
-* Your PR must pass all CI tests before we will merge it. If you're seeing an      error and don't think
+* Your PR must pass all CI tests before we will merge it. If you're seeing an
+  error and don't think
   it's your fault, it may not be! The reviewer will help you if there are test
   failures that seem
   not related to the change you are making.

Taskfile.yml

@@ -27,7 +27,6 @@ tasks:
   test-integration:
     desc: Run integration tests only
     cmds:
-      - go test -run '{{ default "Integration" .TEST_REGEX }}' {{ default "-v" .GOFLAGS }} -coverprofile=coverage_integ.txt {{ default .DEFAULT_TARGETS .TARGETS }} {{.TEST_LDFLAGS}}
       - pytest test
 
   test-legacy: