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.

Sign up for GitHub

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

Jump to bottom

Crash when tapping on People menu item #20112

Closed
momo-ozawa opened this issue Feb 9, 2023 · 0 comments
Closed

Crash when tapping on People menu item #20112

momo-ozawa opened this issue Feb 9, 2023 · 0 comments
Labels
Core Data Issues related to Core Data [Type] Crash

Comments

@momo-ozawa
Copy link
Contributor

Expected behavior

Tapping on the People menu item should open the People screen

Actual behavior

The app crashes

Steps to reproduce the behavior

  1. Make sure you're testing WPiOS 21.7
  2. Go to My Site
  3. Switch to the Menu tab
  4. Tap on People
  5. 💥 Crash

Notes

This crash was introduced in 21.7. No crash on 21.6 (the most recent app store release).

Likely a regression caused by Core Data changes:

*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Illegal attempt to establish a relationship 'blog' between objects in different contexts
Tested on WPiOS 21.7
@momo-ozawa momo-ozawa added [Type] Crash Core Data Issues related to Core Data labels Feb 9, 2023
crazytonyli added a commit that referenced this issue Feb 9, 2023
@crazytonyli
Query the blog instance before using it to update a Role
eb30c07 
Fix #20112
@crazytonyli crazytonyli mentioned this issue Feb 9, 2023
Merged
4 tasks
@guarani guarani mentioned this issue Feb 10, 2023
Closed
mokagio added a commit that referenced this issue Feb 10, 2023
@mokagio
Add simple test to load People screen
74a8370 
This relates to the issue reported in
#20112.

The test _doesn't actually catch the crash_.
When the People screen loads from the UI tests, it gets stuck in some
kind of loading loop where the animation continually restarts.

I decided to not investigate this further because the issue with the
loading loop is likely due to outdated or incomplete API request JSON
stubs and I don't have the time at this point to chase that down

I decide to keep the test anyway because it's a valuable starting point
to ensure that the people screen is available and interactive.
@mokagio mokagio mentioned this issue Feb 10, 2023
Merged
4 tasks
mokagio added a commit that referenced this issue Feb 10, 2023
@mokagio
Add simple test to load People screen
4dd2387 
This relates to the issue reported in
#20112.

The test _doesn't actually catch the crash_.
When the People screen loads from the UI tests, it gets stuck in some
kind of loading loop where the animation continually restarts.

I decided to not investigate this further because the issue with the
loading loop is likely due to outdated or incomplete API request JSON
stubs and I don't have the time at this point to chase that down

I decide to keep the test anyway because it's a valuable starting point
to ensure that the people screen is available and interactive.
mokagio added a commit that referenced this issue Feb 14, 2023
@mokagio
Add simple test to load People screen
7832b0c 
This relates to the issue reported in
#20112.

The test _doesn't actually catch the crash_.
When the People screen loads from the UI tests, it gets stuck in some
kind of loading loop where the animation continually restarts.

I decided to not investigate this further because the issue with the
loading loop is likely due to outdated or incomplete API request JSON
stubs and I don't have the time at this point to chase that down

I decide to keep the test anyway because it's a valuable starting point
to ensure that the people screen is available and interactive.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Core Data Issues related to Core Data [Type] Crash
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@momo-ozawa