Remove unused instance variable in checkout spec
Check for unprocessed payments only before attempting to transition to complete state

Fixes #2694
Payment factory should create payments in the default state of 'checkout', rather than 'pending'
Mention that there's a regression test for #2694 in checkout_controller_spec for posterity
Fix state in payments_controller_spec to reference correct state