Yeah, I used the "never" expiration.

I wanted to look at the Workflow-ng part first because it seemed like it would be more likely to be the part that failed. The hook_order() in uc_roles looks like it should work because the workflow action would cause it to happen.