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

Fix issues related to single-task activities #1128

Merged
merged 5 commits into from
Jan 27, 2019
Merged

Conversation

d4vidi
Copy link
Collaborator

@d4vidi d4vidi commented Jan 23, 2019

Description:

Address the problem specified in #1125.
The root cause resides in what appears to be an Espresso bug (still need to validate this). This PR introduces a workaround, along with associated improvements around the current confusion the android driver currently has around activities launching. I suppose that can be further improved, but one step at a time.

@d4vidi d4vidi requested a review from rotemmiz as a code owner January 23, 2019 10:45
@d4vidi d4vidi requested review from rotemmiz and LeoNatan January 23, 2019 10:58
@d4vidi d4vidi force-pushed the android-fix-single-task branch 2 times, most recently from 4c395b7 to 06f561f Compare January 23, 2019 11:02
@d4vidi d4vidi closed this Jan 23, 2019
@d4vidi
Copy link
Collaborator Author

d4vidi commented Jan 23, 2019

On 2nd thought, I could have an idea on how to get rid of forceSingleTask. I'll give it a go and we'll see.

@d4vidi d4vidi reopened this Jan 23, 2019
@d4vidi d4vidi closed this Jan 23, 2019
@d4vidi d4vidi reopened this Jan 23, 2019
@d4vidi d4vidi force-pushed the android-fix-single-task branch 2 times, most recently from 31e3132 to 9a39d61 Compare January 23, 2019 12:55
@d4vidi
Copy link
Collaborator Author

d4vidi commented Jan 23, 2019

@rotemmiz @LeoNatan CI looks ok (red due to ios flakiness). Waiting for your feedback.

detox/test/android/app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
detox/test/e2e/15.urls.test.js Outdated Show resolved Hide resolved
detox/test/e2e/15.urls.test.js Outdated Show resolved Hide resolved
detox/test/e2e/15.urls.test.js Outdated Show resolved Hide resolved
docs/Introduction.Android.md Show resolved Hide resolved
@d4vidi d4vidi force-pushed the android-fix-single-task branch 2 times, most recently from 11898c6 to d4eb1af Compare January 27, 2019 12:17
@d4vidi d4vidi force-pushed the android-fix-single-task branch from d4eb1af to d9e8a2e Compare January 27, 2019 12:24
@d4vidi d4vidi merged commit a90616a into master Jan 27, 2019
@rotemmiz
Copy link
Member

♨️

@lock lock bot locked as resolved and limited conversation to collaborators Jan 30, 2019
@LeoNatan LeoNatan deleted the android-fix-single-task branch July 22, 2019 02:10
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.

2 participants