Refactor unlock method

[arcturas1]

I'd like to refactor the unlock method to remove magic numbers and enforce a single point of return.

[arcturas1]

The unlock method has been refactored, changes are as follows:

1. renamed several variables
2. Modified the main heuristic loop to enforce a single point of return

• Created constants for Max cycles, max spins, default pattern, and alternative pattern
• Added a nested for loop to run j "cycles" of i "spins"
• Added code to alternate between peek patterns between cycles
• Labelled the "unlockHeuristic" loop
• removed return statement in heuristic loop; replaced with a break referring to the above label
• modified the method to use the boolean value obtained from the most recent spin

3. Changed the logging format to report success/failure as: "Unlock Successful: " + device lock state

• removed logging of final success from within the loop. The unlock success message is now unconditionally performed prior to returning values
• modified unitTests to search for "true" and "false" within the trace instead of "SUCCESS" and" FAILED"

[arcturas1]

Please review changes:
It seems I exceeded the scope of the initial issue I set out to resolve and stuck myself out into messing with the way that we log events. Just the same, the code passes all tests quite nicely, and it does address the initial issue. Just the same, please review my changes thoroughly and let me know if I've gone too far.