Auto-boxing null Integer to long causing NPE

[kmbotts]

Issue #4, NPE when short-polling a virtual queue

Description of changes:
Appears that the ReceiveMessageRequest.waitTimeSeconds is type Integer, so it will be null if not provided. The Completable future get timeout seconds is long type. So the NPE is a result of the auto-boxing of Integer to long which isn't clear as it's happening under the hood in the JVM.

Added check for null and removed implicit auto-boxing by calling longValue() method on Integer object.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[robin-aws]

Thanks for the fix! Would you be able to add a quick unit test for this case?

[kmbotts]

Thanks for the fix! Would you be able to add a quick unit test for this case?

Things are a little hectic at work and the holidays are draining. So maybe after all that I can put a little something together.

[kmbotts]

@robin-aws , I have added the unit test. I just copied the existing test case and modified it to send a null WaitTimeoutSeconds on ReceiveMessageRequest and asserted that it worked. I catch NPE if it occurs. As a sanity check, I tested the negative test case by reverting my fix to ensure it failed as expected. So this should be adequate.

[robin-aws]

Great, appreciate it! Thanks for the contribution!