Includes removing an if-statement that should've been dead but was erronously applied to the following statement. Dead code = bad, if-statements without braces = bad.

Without this, when you go to airframe config and the current config is
Simulation it will not be selected. Hence you won’t know what the
current config is.

Specifically much better testing for the handling of:
- Ack responses not being sent by server
- Getting Nak’s back from commands
- CRC failures

Also changed file validation to take into account Mock Server no longer
sending file length in first byte of file.

Also changed the downloaded file contents to not include the file
length in the first byte. This makes the code a bit less confusing.

Changed to get the current voice properties, then modify then and set new ones.

Changed to get the current voice properties, then modify then and set new ones.

When unit tests are being run in TeamCity the host box can end up being
very bogged down with other builds. So need to up wait time, to take
into account CPUs grinding.

Need to disambiguate from video stream messages

Found by valgrind.

Found via valgrind.

I don't think it's needed anywhere in QGC since this has been a bug for a long time.

SLUGS has maintained their patches out-of-tree for a long while now.
These will eventually get merged back in, but it'll be easier to evaluate them as fresh commits instead of modifications to this old & dead code, so we'll delete this for now.