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

bug: mobile: e2e: sign-in tests fail on Android #111

Open
ericswpark opened this issue Mar 24, 2024 · 1 comment
Open

bug: mobile: e2e: sign-in tests fail on Android #111

ericswpark opened this issue Mar 24, 2024 · 1 comment
Assignees
Labels
blocked This is waiting on upstream to fix bug Something isn't working mobile Mobile-related

Comments

@ericswpark
Copy link
Member

What happened?

On Android, signing in fails due to a synchronization issue with Detox.

What can we do to reproduce the problem?

Run the sign-in E2E test with Detox, on Android, using the command:

npx detox test --configuration android.att.debug signin-screen.test.js

Relevant log output

● Sign in screen › try sign in with demo credentials                             
                                                                                   
    Test Failed: '(view has effective visibility <VISIBLE> and view.getGlobalVisibleRect() covers at least <75> percent of the view's area)' doesn't match the selected view.
    Expected: (view has effective visibility <VISIBLE> and view.getGlobalVisibleRect() covers at least <75> percent of the view's area)
         Got: was null

      33 |     await waitAndTap("sign-in-signin-arrow", 1500);
      34 |
    > 35 |     await expect(element(by.id("library-screen-view"))).toBeVisible();  
         |                                                         ^
      36 |   });
      37 |
      38 |   it("try sign in with bad credentials", async () => {

      at createErrorWithUserStack (node_modules/detox/src/utils/errorUtils.js:36:27)
      at Client._doSendAction (node_modules/detox/src/client/Client.js:168:32)     
      at Client._sendMonitoredAction (node_modules/detox/src/client/Client.js:161:25)
      at Client.sendAction (node_modules/detox/src/client/Client.js:147:14)        
      at Client.execute (node_modules/detox/src/client/Client.js:132:25)
      at InvocationManager.execute (node_modules/detox/src/invoke.js:11:39)        
      at MatcherAssertionInteraction.execute (node_modules/detox/src/android/interactions/native.js:23:31)
      at NativeExpectElement.toBeVisible (node_modules/detox/src/android/core/NativeExpect.js:25:137)
      at Object.toBeVisible (e2e/signin-screen.test.js:35:57)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
      at _next (node_modules/@babel/runtime/helpers/asyncToGenerator.js:22:9)

Anything else?

IMG_6552.mov
@ericswpark ericswpark added the bug Something isn't working label Mar 24, 2024
@ericswpark ericswpark self-assigned this Mar 24, 2024
@ericswpark ericswpark added the blocked This is waiting on upstream to fix label Mar 24, 2024
ericswpark added a commit that referenced this issue Mar 24, 2024
@ericswpark
Copy link
Member Author

This has been "fixed" with a workaround for now that adds a delay, but I'm keeping this issue open until there's an official patch from upstream.

@ericswpark ericswpark added the mobile Mobile-related label Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This is waiting on upstream to fix bug Something isn't working mobile Mobile-related
Projects
None yet
Development

No branches or pull requests

1 participant