Skip to content
This repository has been archived by the owner on Jan 26, 2019. It is now read-only.

Travis build/e2e-kitchensink: Outstanding behavior fixes... #317

Merged
merged 1 commit into from
May 15, 2018

Conversation

DorianGrey
Copy link
Collaborator

@DorianGrey DorianGrey commented May 2, 2018

Just another attempt...

Current problem: the e2e-kitchensink task fails on testing the eject command, since babel-loader does not seem to be installed after ejection, even though the log mentions that it was added properly.

This might have something to do with a mix of npm run and yarn... but I'm still inspecting this further.

€dit:

  • Workaround seems to be to force a yarn install before building ...yay.
  • Next error to solve: Module not found: Error: Can't resolve 'absoluteLoad' in '/tmp/tmp.ji4bCBWZ7D/test-kitchensink/src/features/env'.

€dit#2:

  • Next error to fix: TypeError: (0 , _testIntegrity.version) is not a function raised in src/features/webpack/LinkedModules.test.js
  • There is a module named test-integrity which gets linked during the test. However, it does not seem to be in the expected format. Attempting to use yarn instead of npm to link it results in a module not found error.

€dit#3:
Seems that the dependency installation for the ejected projected has to happend before the npm link for test-integrity - otherwise, an outdated version is installed which does not match the one expected in the test. After having this pointed out, the updated suite seems to work fine - finally!

@DorianGrey DorianGrey force-pushed the another-travis-fix-attempt branch 2 times, most recently from 3d7e9cd to 5207cfc Compare May 3, 2018 06:31
@DorianGrey DorianGrey changed the title [W.I.P.]Travis build/e2e-kitchensink: 'babel-loader' is missing after eject [W.I.P.]Travis build/e2e-kitchensink: Outstanding behavior fixes... May 3, 2018
@DorianGrey DorianGrey force-pushed the another-travis-fix-attempt branch 2 times, most recently from 0346af7 to 85270a9 Compare May 15, 2018 07:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant