[mod_opus] Fix status returned by switch_opus_decode(). Should be SWITCH_STATUS_FALSE instead of SWITCH_STATUS_NOOP. Add a unit-test.

[andywolk]

This PR reverts the change made by https://github.com/signalwire/freeswitch/pull/982

[signalwire-ci[bot]]

Unit-tests compilation failed: https://public-artifacts.signalwire.cloud/drone/signalwire/freeswitch/1677/unit-tests-build-result.txt

[signalwire-ci[bot]]

Scan-build compilation failed: https://public-artifacts.signalwire.cloud/drone/signalwire/freeswitch/1677/scan-build-result.txt

[signalwire-ci[bot]]

Unit-tests failed: https://public-artifacts.signalwire.cloud/drone/signalwire/freeswitch/1678/artifacts.html

[a-rose]

Hello,

I agree with the intent in #982 , hanging up on any decoding error is not great. Opus is able to keep decoding the stream if one packet is missing. I think it's better to return SWITCH_STATUS_BREAK as I mentioned in the original PR.