LPS-32232

@BeforeClass was intentional. It is needed because we need to create the user outside of the transactional context

Because of the workflow logic in ULSI, addUser changes the status of the user in a callback (UserLocalServiceImpl.startWorkflowInstance().. which we'll never receive.

By running it in @BeforeClass, it passes the callbackListList.isEmpty() check in TransactionCommitCallbackUtil and invokes it right away.

The alternative was to add a user, change the status, and update the user as jozsefilles does here: 1bc3dcd.

I ran this by Shuyang and this is what he advised me to do.