Fix encoding and user problems in the Perforce unit tests.

Review Request #8803 — Created March 9, 2017 and submitted

Information

Review Board
release-2.0.x
2c9af41...

Reviewers

The Perforce unit tests didn't work if your local user account wasn't
registered on public.perforce.com. We didn't notice because ours were
all on there, but this probably failed for some contributors and
certainly for continuous integration.

Along with this, the encoding wasn't explicitly set and, client-side,
was defaulting to "utf8" on newer versions of p4python. This wasn't
compatible with public.perforce.com.

We now explicitly set the user "anonymous" and encoding "none" in order
to ensure tests will pass.

Successfully ran the Perforce unit tests locally and in CI.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/scmtools/tests/test_perforce.py
    
    
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (e2b1701)
Loading...