Guess I was wrong. Unit tests *were* broken

* New state-machine for non-IVS switches is unit tested
* No unit tests for IVS switches yet

* fail makeCookie if the app id hasn't been registered
* do static initialization
* fail if duplicate app id registration

still not a perfect framework but better than before

[BVS-486] getPortType will default to UPLINK ports for BigSwitchOVS.  The autoportfast detection is performed only in BSNOVS and Xenon switches.

BVS-515 (https://github.com/bigswitch/bigswitchcontroller/pull/3608)

mostly cleanup (https://github.com/bigswitch/bigswitchcontroller/pull/3633)

* alwasy log stack trace at debug level (even if we just log the message
  at error)
* don't wrap other exceptions in SwitchStateException. We don't log that
  with stack trace
* IOFSwitch.setDebugCounterService now throws the correct exception

BSC-3818 BSC-3817 Send SNMP traps for switch connect and port changes (https://github.com/bigswitch/bigswitchcontroller/pull/3607)

add fields to forwardingconfig model in bigcon to hold idle/hard timeouts
add storage listeners for these 2 columns in bigfloodlight's forwarding

Fixes: non IP wildcards (BVS-525), ForwardingTest  (https://github.com/bigswitch/bigswitchcontroller/pull/3572)

Enable switch read/write throttling by default (https://github.com/bigswitch/bigswitchcontroller/pull/3578)

Flattened debug counters REST output a little bit.

Conflicts:
	bigfloodlight/src/main/java/com/bigswitch/floodlight/forwarding/Forwarding.java

moving counter registration to init() methods, and use of metadata constants

DebugEvents beta  (https://github.com/bigswitch/bigswitchcontroller/pull/3545)

Register Vendor Actions on Floodlight startup (https://github.com/bigswitch/bigswitchcontroller/pull/3538)

Don't allow 0 MAC address. More comprehensive DevMgr unit tests. BVS-516  (https://github.com/bigswitch/bigswitchcontroller/pull/3511)