Fix a handful of regressions and issues in unit tests.

Review Request #12091 — Created Feb. 27, 2022 and submitted

Information

ReviewBot
release-3.0.x

Reviewers

Some recent changes, and updates to third-party tools, have regressed
some unit tests. This change fixes them up.

Fixes include:

  • A cookie-related error assertion that didn't work on all versions of
    Python 2 and 3.
  • Full-repository unit tests never cleaned up their temp directories.
  • Modern versions of flake8 no longer report both styling errors and
    syntax errors in our test file.
  • RuboCop had a bad string check.
  • RuboCop error message checks now check in a way that doesn't depend on
    a given version of Ruby.
  • Shellcheck error checks have been updated for string changes in the
    tool.

All unit tests pass on Python 2.7 and 3.6-3.10 (except Infer, which I
can't yet test on Apple M1).

Summary ID
Fix a handful of regressions and issues in unit tests.
Some recent changes, and updates to third-party tools, have regressed some unit tests. This change fixes them up. Fixes include: * A cookie-related error assertion that didn't work on all versions of Python 2 and 3. * Full-repository unit tests never cleaned up their temp directories. * Modern versions of flake8 no longer report both styling errors and syntax errors in our test file. * RuboCop had a bad string check. * RuboCop error message checks now check in a way that doesn't depend on a given version of Ruby. * Shellcheck error checks have been updated for string changes in the tool.
69cb91851fb1307b2348c6def420b4ed58bd3cf6
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.0.x (dc04a33)