Fix regressions in worker unit tests.

Review Request #12159 — Created March 16, 2022 and submitted

chipx86
ReviewBot
release-3.0.x
reviewbot

Some unit tests have recently regressed, in part due to output changes
in recent versions of some of the tools we call.

cpplint and flake8 had some string changes, which are harmless, but
broke the test suite. We now test against the updated strings.

Something with the temp directory management also changed. I'm not sure
why it wasn't broken before, but basically, modules were importing
tmpdirs and tmpfiles for testing, and these global variables
themselves were being overridden when cleaning up the temp dirs/files.
This meant that modules were hanging onto old copies. Now, we keep the
same instances around, and they're just cleared.

Unit tests pass on Python 2.7 and 3.6-3.10.

Summary
Fix regressions in worker unit tests.
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (29b571c)
Loading...