So no need for connect window close test here

Also moved log file settings save check to mavlink protocol

Also added/converted some logging to debug the problem caused by the
two parameter request apis.

This removes confusion as to whether you call UAS::requestParameters
(incorrect) or QGCUASParamManager::requestParameterList (correct) from
client code.

Also only update the full tree one time once the full parameter list is
ready.

Previously link were just destroyed

This makes unit test output easier to read

This allows unit tests to simulate user selections in a combo box

This allows use of compile time Object::connect syntax

This is done by tweaking the application name when running unit test

I initially change version numbers incorrectly. Should have left stable
at 2.1 and only bumped Developer to 2.2. This leaves Stable at 2.2 and
Developer at 2.3.

Previously link were just destroyed

This makes unit test output easier to read

Also restructured code to remove duplication

Also restructured code to remove duplication

There appears to be a threading issue in qRegisterMetaType which can
cause it to throw a qWarning about duplicate type converters. This is
caused by a race condition in the Qt code. Still working with them on
tracking down the bug. For now we register the type which is giving us
problems here while we only have the main thread. That should prevent
it from hitting the race condition later on in the code.