Fix unit test regressions in the Review Bot extension and ease testing.

Review Request #12451 — Created July 11, 2022 and submitted

Information

ReviewBot
release-3.x

Reviewers

The recent change to fix stale broker connections failed to actually
store the revised broker URL used to help indicate if a connection was
stale. This led to creating a brand-new Celery connection every time the
property was accessed, and broke unit tests in the process.

This change fixes that, and also improves test infrastructure by
supplying a default settings_local.py file that can be used. This
avoids needing to have one already set up in a dev server in order to
run tests.

Unit tests pass, without a settings_local.py being in the Python path.

Summary ID
Fix unit test regressions in the Review Bot extension and ease testing.
The recent change to fix stale broker connections failed to actually store the revised broker URL used to help indicate if a connection was stale. This led to creating a brand-new Celery connection every time the property was accessed, and broke unit tests in the process. This change fixes that, and also improves test infrastructure by supplying a default `settings_local.py` file that can be used. This avoids needing to have one already set up in a dev server in order to run tests.
72576834d116b87c1218ee24ed4b41c0c9c35a56
david
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (0ee76b4)
Loading...