Fix redundant test runs when specifying specific tests.

Review Request #8323 — Created Aug. 12, 2016 and submitted

Information

Djblets
release-0.10.x
d820c2f...

Reviewers

The test runner takes a list of test labels (module/class/function
paths) to run, which we were appending to the nose argument list.
However, in our standard cases, these may already be in the nose
argument list, leading to the tests being run more than once.

We still want to respect that argument, so we're now adding only the
entries not otherwise found in the argument list.

Ran a subset of Review Board's unit tests and saw that the tests were
only run once, not twice.

reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        djblets/testing/testrunners.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        djblets/testing/testrunners.py
    
    
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-0.10.x (ab9c97f)