Align test & lint

[grahamalama]

Chatted with @leplatrem about #81, and I added some more changes on top of that PR here.

The idea with this PR is that infra/lint.sh and infra/test.sh are the "source of truth" for how we want to run linting and tests respectively. We then use these scripts in precommit, make, Github Actions, and wherever else we want to run tests and linting.

For Github Actions, we also now run linting and tests directly in the action runner, rather than running through Docker. This resulted in a speed-up of the checks.

Fixes #79
Fixes #67

[bsieber-mozilla]

Overall, LGTM--thanks for the consolidation! 👍

[grahamalama]

Thanks for the approval @bsieber-mozilla. @leplatrem, looking for your review as well since I built on top of your initial PR.

[grahamalama]

(I should really get out of the habit of my rebase on main workflow since it causes me to force push 🤦🏻‍♂️)

[leplatrem]

Great 🎉

nit: the Github jobs are called lint-build and test-build, which was relevant when we were testing containers. Now they can be rename lint and test IMO

[leplatrem]

I just realized we exclude tests from certain checks.

When including tests in isort we face the issue I was having, where black unwraps and isort wraps back.

[grahamalama]

When including tests in isort we face the issue I was having, where black unwraps and isort wraps back

Something I just had to do that fixed this:

• make sure you have isort ^5.10.1 installed in your virtualenv
• make sure you don't have an old .isort.cfg file laying around -- I noticed I still had one in my directory, and I think that was the thing that was causing problems since that will take precedence over pyproject.toml if it's present.

I think when we do that, isort will respect the "black" profile in pyproject.toml