1402878 - Add some more sanity-type tests, including perl critic

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Continous Integration, enhancement)

Description

[Dylan Hardison [:dylan] (he/him)]

As more people contribute to BMO on a one-off or infrequent basis, it becomes more important to check for common errors.

A lot of errors can be found by perl critic -- and perl critic can be used by the very user-friendly Visual Studio Code editor -- which means that pretty much anyone could make use of it.

So in this bug I'm adding a perlcriticrc file and a test that checks the code against it. Of course, even with a lot of tweaks the code still has perl critic errors -- but the test doesn't care about errors that already exist. It only cares that the number of errors doesn't grow.

Meanwhile, while writing this I realized I had sloppily added dependencies to cpanfile in the bmo-systems repo -- so I've properly added that (Test::Selenium::Firefox) to the Makefile.PL.

Finally, we pin the version of bmo-slim (our base docker container) in both Dockerfile and the circle ci config. I added a simple test that will detect if those are ever in disagreement.

Comment 1

[Bug Butler]

Attachment: github pull request