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

[WIP] E2E Test: Add tests for running real RN app #312

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

jhen0409
Copy link
Owner

@jhen0409 jhen0409 commented Feb 3, 2019

TODO

  • Context Menu: should show AsyncStorage content as expected
  • Context Menu: should clear AsyncStorage as expected
  • Context Menu: should send Network request as expected if Network Inspsect enabled
  • Test with Redux DevTools
  • Test with React DevTools
  • Test with Apollo Client DevTools
  • Run tests on CI
  • TestApp: Fix install errors on Node 9

Test plan

  • Use Detox to control the simulator
  • Use NativeModules.DevSettings.setIsDebuggingRemotely(true) to enforce app use debug mode
  • Use Spectron to launch RNDebugger
  • Test context menu functions (AsyncStorage / Networks Inspect)
  • Test with Redux / MobX app and Redux DevTools
  • Test React DevTools behavior
  • Test with Apollo Client DevTools

CI plan

  • It might running on macOS only
  • Followed the guide for running on CI

@jhen0409 jhen0409 force-pushed the e2e-test-real-project branch from 3d1a852 to 6645e96 Compare February 7, 2019 05:30
@jhen0409 jhen0409 force-pushed the e2e-test-real-project branch from ca9a795 to ecac16a Compare February 13, 2019 06:16
@Gongreg
Copy link
Collaborator

Gongreg commented Feb 14, 2019

Hey @jhen0409, do you need any help regarding detox? If you have any questions feel free to ask.

I see that you are trying to use jest for detox, from my experience mocha works a lot more reliably.

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.

2 participants