Now that we're using stubbing in the tests instead of this users.json fixture file, the only remaining uses of it were for the "bypass OAuth" mode. Also, since we're now managing user roles within editor-api (instead of in the user info API), it's a lot less obvious the users in the fixture file are actually useful even for the "bypass OAuth" mode. I think it makes the code clearer to remove the fixture file and hard-code some user data for the "bypass OAuth" mode.
Now that we're using stubbing in the tests instead of this
users.jsonfixture file, the only remaining uses of it were for the "bypass OAuth" mode. Also, since we're now managing user roles within editor-api (instead of in the user info API), it's a lot less obvious the users in the fixture file are actually useful even for the "bypass OAuth" mode. I think it makes the code clearer to remove the fixture file and hard-code some user data for the "bypass OAuth" mode.Closes #308