Ran
|
Jobs
1
|
Files
48
|
Run time
1s
|
Badge
Embed ▾
README BADGES
|
push
github
fix crash when calling reloadObjects during an update Summary: Follow up to D47263265 ## What's happening We're calling `listAdapter.reloadObjects(...)` in the middle of the IGListKit's internal data update, so it's using the new section index path, instead of the old one. So we're going from a list of 2, insert at 0, and try to reload index 1, then we crash (it'll try to update index 2, which doesn't exist in the old array) ## Fix Just like for `[IGListAdapter performBatchAnimated:updates:completion:]`, lets use the old index. To get this working, lets ask the updater if we're in an update, rather than `IGListAdapter` try to keep track of it. Sorry the diff is a bit long, but in case this feels reasonable to pick, I might Differential Revision: D47281472 fbshipit-source-id: 193153cfc
29 of 29 new or added lines in 4 files covered. (100.0%)
4119 of 4125 relevant lines covered (99.85%)
706.39 hits per line
Lines | Coverage | ∆ | File |
---|---|---|---|
3 |
94.74 |
-5.26% | Source/IGListKit/IGListReloadDataUpdater.m |
ID | Job ID | Ran | Files | Coverage | |
---|---|---|---|---|---|
1 | iOS - 5489555151.1 | 52 |
99.83 |
GitHub Action Run |
Coverage | ∆ | File | Lines | Relevant | Covered | Missed | Hits/Line |
---|