flake8 cleanup of some test files
Minor typo fixes
testGetTile() checks against getPlane(). See #11315
Update test for py.test
Add swapIfRequired to getTileDirect (Fix #10581)

The other `get*Direct` methods all contain a call to
`reader.swapIfRequired()`. The same logic in the non-
direct methods is handled by a call to `setOrder` on
the newly created `PixelData` object.
Prevent double swapping of tile bytes

Each of the `get*Direct` methods now call
swapIfRequired. Non-direct methods should
not call them since double swapping could
occur.
Modify testGetTile to use gatewaywrapper from fixture.
Minor refactoring of BFPPB for testing

In order to be able to intercept (and block) certain
methods of BFPPB for concurrency testing methods have
been changed to `protected` and an `init()` method has
been introduced to allow a ctor with less logic.
Passing test with intermediate fix

The change (basically to not lock on read) makes
this test pass where before it was failing (when
`Thread.sleep()` was artificially introduced for
calls to releaseLock.

It's unknown if this will be sufficiently safe
going forward.
Clean up code.
Remove tabs and whitespace.
Add check if all required fields are entered.
Add check and remove tab/whitespace.
Remove tabs and whitespace.
Improve code.
Make sure required parameters are populated.
Handle all the types returned.
Set global min/max according to pixel type. (see #11840)
If invalid interval set it to range wrt pixels type
Do not specify user.