Skip to content

Fix android detox build#74

Merged
f2prateek merged 1 commit intosegmentio:masterfrom
FormidableLabs:bugfix/fix-android-detox
Jun 6, 2019
Merged

Fix android detox build#74
f2prateek merged 1 commit intosegmentio:masterfrom
FormidableLabs:bugfix/fix-android-detox

Conversation

@kadikraman
Copy link
Copy Markdown
Contributor

@kadikraman kadikraman commented Jun 6, 2019

The end to end tests for detox aren't actually being run during the test-android step, just the detox app is built.

The build fails with the following error, because yarn cannot find the detox binary:

Screenshot 2019-06-06 at 15 17 03

This adds the detox command to package.json in the test app.

@kadikraman kadikraman force-pushed the bugfix/fix-android-detox branch from 79677d6 to dd9cef7 Compare June 6, 2019 14:15
@kadikraman kadikraman changed the title WIP add detox command to package.json Fix android detox build Jun 6, 2019
@kadikraman kadikraman force-pushed the bugfix/fix-android-detox branch from dd9cef7 to 7144474 Compare June 6, 2019 14:19
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   86.09%   86.09%           
=======================================
  Files           7        7           
  Lines         151      151           
  Branches       28       21    -7     
=======================================
  Hits          130      130           
  Misses         20       20           
  Partials        1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1dd1396...7144474. Read the comment docs.

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.

3 participants