Closed MelissaAutumn closed 7 months ago
I ran the squish merge on the new docker image (separate PR) and we may have some test regressions lol. But much better than before:
======== 143 failed, 5542 passed, 48 xfailed, 1 xpassed, 31 warnings, 2 error in 1569.96 seconds ========
This should be good after a master rebase.
Resolves #207
This is a huge PR mostly due to the inclusion of the latest products json files (which the docker container generates on build iirc?) This update happened to fix some tests. 🤔
We should also merge the docker-fixes to master so we can change the target branch of this PR to master.
Unfortunately this was mostly an exercise of whack-a-mole. Not the cleanest, but I figured if we can at least get the API routes solidly tested, we can move ahead with any major upgrades.
So here's some notes:
src/olympia/core/tests/test_certifi.py
there's two tests that ping google and amazon for 200 statuses. These fail when running the complete suite, but succeed when run alone. I actually have no clue as to why, so I didn't mark them as xfail.We should definitely squish this PR, lmk if I should pre-squish these commits.
(It would also be nice to maybe get a CI going with this project.)