De-duplicate unit tests for ports in Big Switch

This eliminates the separate classes that existed
for the VIF type testing. Each class was going
through all of the port tests when none of the
logic those tests were exercising was affected by
the VIF types.

This cut the number of tests run in test_restproxy_plugin
by close to 44% without a reduction in code coverage.

Closes-Bug: #1295448
Change-Id: I8c84e1599568fad36b6dd3d51e4ba8a3c012c6c5
Changed the message line of RouterInUse class

Implements: Changed the message line of RouterInUse class,
so user will not get confused with active or inactive ports
while deleting the router.

Closes-Bug: #1236372
Change-Id: I7a82550e2c6051f9e4940a8f1dd4401059e0a8d8
Fix quota_health_monitor opt name in neutron.conf

Reasons:
- quota_healt_monitor does not exist

Changes:
- Use quota_health_monitor instead of quota_health_monitors

Change-Id: I31d6a0394a786d26f7a063afbd3b6e07fae49b9b
Remove last parts of Quantum compatibility shim

Goodbye Quantum!

Change-Id: Idbd0384a892beaff3a937444f04cfc433cb805eb
Closes-Bug:1299046
Restore NOT NULL constraint lost by earlier migrations

Bring back nullable=False for columns in tables poolstatisticss
and ipsec_site_connections that disapeared because of
migrations abc88c33f74f and 338d7508968c correspondingly.

Closes-bug: #1288681

Change-Id: Idc4608616c2b4c06e7404157b2f708d46e91accc
Use os.uname() instead of calling uname in subprocess

Related-Bug: #1291535

Change-Id: I57e18c0b95904f318e3fff08cd18b32e28f48834
use floatingip's ID as key instead of itself

Change-Id: I267f174a0e2e611ec007404c7b44e2c73a359c38
Closes-Bug: 1294526
Revert changes removing OVSBridge return

This reverts changes from this review [1] which incorrectly removed
the return of the OVSBridge object when calling BaseOVS.add_bridge.
This broke all functional tests relying on this return.

[1] https://review.openstack.org/#/c/71142/

Related-Bug: #1301363

Change-Id: If32e155cc2bd368b2e842f14b1d1da0827ccfeff
Remove invalid copyright headers under API module

Change-Id: I629b639f6c94654117fbafb8b21840f7ba6b045a
Partial-bug: #1262424
fixes tests using called_once_ without assert

A few tests were using mock's called_once, or called_once_with_args
instead of assert_called_once or assert_called_once_with_args. Those
methods return a bool that needs to be actively checked.

The tests are fixed to avoid them from passing if the call condition
is not met.

Change-Id: I21e5257b26b2a08cc8f0b108233d1d5cc0b97b89
Closes-bug: #1297875
Explicitly import state_path opt in tests.base

This wasn't explicitly needed in the unit tests because
neutron.common.config was already loaded, but this broke for the
functional tests (which are non-voting) so add a CONF.import_opt line so
functional tests know where to find the definition for state_path.

Change-Id: I5b87a75317e2ba670adaa59b251bfa860920fd61
Merge "Revert changes removing OVSBridge return"
Merge "Remove last parts of Quantum compatibility shim"
Merge "Explicitly import state_path opt in tests.base"
Imported Translations from Transifex

Change-Id: Ifcadfb837fc2864214c6d070d90388110742b537
Merge "Use os.uname() instead of calling uname in subprocess"
Merge "Changed the message line of RouterInUse class"
Merge "Remove invalid copyright headers under API module"
Merge "Restore NOT NULL constraint lost by earlier migrations"
ignore build directory for pep8

this mirrors the excludes from nova.

Change-Id: I0ca5f02918842fc708c704fcf2845936dc1c5965