[skip-changelog] add testing and linting infrastructure (#39)

* [skip-changelog] add integration testing infrastructure

* [skip-changelog] add linting infrastructure and apply formatting

* fix prettier

* move build native step to the bottom to fix test failing

* apply suggestions from code review

* add workflow to check general formatting using editorconfig

Author: umbynos

.flake8

@@ -0,0 +1,9 @@
+[flake8]
+doctests = True
+ignore =
+    # W503 and W504 are mutually exclusive, so one or the other must be ignored.
+    # PEP 8 recommends line break before, so we keep W504.
+    W503
+max-complexity = 10
+max-line-length = 120
+select = E,W,F,C,N

.github/workflows/check-general-formatting.yml

@@ -0,0 +1,22 @@
+name: Check General Formatting
+
+# See: https://docs.github.com/en/actions/reference/events-that-trigger-workflows
+on:
+  push:
+  pull_request:
+  schedule:
+    # Run every Tuesday at 8 AM UTC to catch breakage caused by changes to tools.
+    - cron: "0 8 * * TUE"
+  workflow_dispatch:
+  repository_dispatch:
+
+jobs:
+  check:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v2
+
+      - name: Check formatting
+        uses: editorconfig-checker/[email protected]

.github/workflows/lint-python.yml

@@ -0,0 +1,40 @@
+name: Lint Python code
+
+on:
+  push:
+    paths:
+      - "**.py"
+      - ".flake8"
+      - "pyproject.toml"
+      - "Taskfile.yml"
+  pull_request:
+    paths:
+      - "**.py"
+      - ".flake8"
+      - "pyproject.toml"
+      - "Taskfile.yml"
+
+jobs:
+  lint-python:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Install Taskfile
+        uses: arduino/setup-task@v1
+        with:
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          version: 3.x
+
+      - name: Install Python
+        uses: actions/setup-python@v2
+        with:
+          python-version: "3.8"
+
+      - name: Install Poetry
+        run: pip install poetry
+
+      - name: Lint Python files
+        run: task python:check

.github/workflows/test.yaml

@@ -48,5 +48,13 @@ jobs:
           GOARCH: "arm"
         run: task build
 
-      - name: Run unit tests
+      - name: Install Python
+        uses: actions/setup-python@v2
+        with:
+          python-version: "3.8"
+
+      - name: Install Poetry
+        run: pip install poetry
+
+      - name: Run unit and integration tests
         run: task test

.gitignore

@@ -4,6 +4,7 @@
 .idea
 coverage_*.txt
 /dist
+__pycache__/
 
 # Misc.
 .DS_Store

Taskfile.yml

@@ -33,9 +33,22 @@ tasks:
 
   test:
     desc: Run tests
+    cmds:
+      - task: test-unit
+      - task: test-integration
+
+  test-unit:
+    desc: Run unit tests
     cmds:
       - go test -short -race -run '.*' {{ default "-v" .GOFLAGS }} -coverprofile=coverage_unit.txt ./... {{.TEST_LDFLAGS}}
 
+  test-integration:
+    desc: Run integration tests
+    cmds:
+      - task: build
+      - poetry install --no-root
+      - poetry run pytest test
+
   check:
     desc: Check fmt and lint
     cmds:
@@ -44,6 +57,7 @@ tasks:
       - "'{{.GOLINTBIN}}' {{.GOLINTFLAGS}} ./..."
       - task: docs:check
       - task: config:check
+      - task: python:check
 
   config:check:
     desc: Check linting of configuration and supporting files
@@ -55,6 +69,22 @@ tasks:
     cmds:
       - npx {{ .PRETTIER }} --write "**/*.{yml,yaml}"
 
+  python:check:
+    cmds:
+      - task: python:lint
+
+  python:lint:
+    desc: Lint Python code
+    cmds:
+      - poetry install --no-root
+      - poetry run flake8
+
+  python:format:
+    desc: Automatically formats Python files
+    cmds:
+      - poetry install --no-root
+      - poetry run black .
+
 vars:
   PROJECT_NAME: "FirmwareUploader"
   DIST_DIR: "dist"

go.sum

@@ -307,6 +307,7 @@ golang.org/x/lint v0.0.0-20190301231843-5614ed5bae6f/go.mod h1:UVdnD1Gm6xHRNCYTk
 golang.org/x/lint v0.0.0-20190313153728-d0100b6bd8b3/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc=
 golang.org/x/lint v0.0.0-20190409202823-959b441ac422/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc=
 golang.org/x/lint v0.0.0-20190909230951-414d861bb4ac/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc=
+golang.org/x/lint v0.0.0-20190930215403-16217165b5de h1:5hukYrvBGR8/eNkX5mdUezrA6JiaEZDtJb9Ei+1LlBs=
 golang.org/x/lint v0.0.0-20190930215403-16217165b5de/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc=
 golang.org/x/mobile v0.0.0-20190312151609-d3739f865fa6/go.mod h1:z+o9i4GpDbdi3rU15maQ/Ox0txvL9dWGYEHz965HBQE=
 golang.org/x/mobile v0.0.0-20190719004257-d2bd2a29d028/go.mod h1:E/iHnbuqvinMTCcRqshq8CkpyQDoeVncDDYHnLhea+o=