Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add authentication test helpers #1129

Closed
wants to merge 59 commits into from
Closed

Conversation

qroac
Copy link
Contributor

@qroac qroac commented Jul 22, 2019

Exports TestHelper from @gqlapp/authentication-server-ts, containing:

  • login(usernameOrEmail, password)
    (defaults to admin user if not provided)
  • logout()
    to be imported and used in tests for any module that relies on authenticated users

Solves #1126

Because jest normally runs multiple tests simultaneously, --runInBand forces to run them one after another. Authentication is shared between simultaneously running tests, thus login and logout have side effects to other tests running at the same time.

Michael Fürmann and others added 30 commits October 5, 2018 16:24
Check for items[name].skip results in error on app start.
Property skip is checked from original drawerItem object instead
larixer and others added 27 commits June 17, 2019 22:34
* Switch to webpack.config.js for React frontend builds

* Use babel to transpile web frontend

* Add storybook support

* Switch to webpack.config.js for server

* Use default Heroku caching

* Use webpack.config.js for Vue

* Switch to babel-loader

* Add webpack.config.js for Angular

* Fix storybook support
Add a convenience script
* Add minimal working vanilla Expo build scripts

* Add good Expo build infra

* Remove zen usage

* Remove yarn build call

* Add expo-cli dep

* Switch from @babel/polyfill to core-js/stable

* Remove custom cacheDirectories

* Remove heroku-postbuild for now
# Conflicts:
#	modules/look/client-react-native/DrawerComponent.jsx
@larixer
Copy link
Member

larixer commented Oct 15, 2019

Closing this PR, because --runInBand is a constraint that we don't want to have in our tests

@larixer larixer closed this Oct 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants