Skip to content

fix(library-update): fallback להורדה מלאה על כל כשל apply — הקשחה לקראת סכמה 4 #1043

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

Closed
palmoni5 wants to merge 1 commit into from
Closed

fix(library-update): fallback להורדה מלאה על כל כשל apply — הקשחה לקראת סכמה 4 #1043

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 13 additions & 6 deletions lib/library_update/bloc/library_update_bloc.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -246,12 +246,19 @@ class LibraryUpdateBloc extends Bloc<LibraryUpdateEvent, LibraryUpdateState> {
} catch (e, st) {
if (_isStale(opId)) return;
_logUpdateError('applyDeltaPlan', e, st);
// אי-התאמת hash הופכת את מסלול הדלתא ללא בטוח; הורדה מלאה עוקפת אותו.
if (e is PatchApplyException && e.isContentMismatch) {
final mismatchReason =
e.hashMismatchStage == PatchHashMismatchStage.toContentHash
? 'תוצאת עדכון הדלתא אינה תואמת לגרסה הצפויה'
: 'תוכן הספרייה המקומית שונה מהצפוי';
// כל כשל apply (אי-התאמת hash, גרסה/סכמה לא תואמת, patch פגום) הופך
// את מסלול הדלתא ללא בטוח; הורדה מלאה עוקפת אותו. בלי זה, כשל שאינו
// אי-התאמת תוכן — למשל patch בסכמה חדשה מהנתמכת — משאיר את המשתמש
// בלולאת שגיאה ללא מוצא עד עדכון אפליקציה.
if (e is PatchApplyException) {
final String mismatchReason;
if (!e.isContentMismatch) {
mismatchReason = 'החלת עדכון הדלתא נכשלה';
} else if (e.hashMismatchStage == PatchHashMismatchStage.toContentHash) {
mismatchReason = 'תוצאת עדכון הדלתא אינה תואמת לגרסה הצפויה';
} else {
mismatchReason = 'תוכן הספרייה המקומית שונה מהצפוי';
}
final fallback = plan.toFullDownloadFallback(
reason: mismatchReason,
);
Expand Down
36 changes: 36 additions & 0 deletions test/library_update/library_update_bloc_test.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1091,6 +1091,42 @@ void main() {
],
);

blocTest<LibraryUpdateBloc, LibraryUpdateState>(
'כשל apply שאינו סטיית תוכן (למשל סכמה לא נתמכת) → fallback להורדה מלאה',
build: () => _bloc(
_FakeService(
deltaWithFallbackPlan,
applyError: const PatchApplyException(
'גרסת סכמת ה-patch (5) חדשה מהנתמך (4) — נדרש עדכון תוכנה',
),
),
),
act: (b) => b.add(const StartLibraryUpdate()),
expect: () => [
isA<LibraryUpdateState>().having(
(s) => s.status,
'status',
LibraryUpdateStatus.checking,
),
isA<LibraryUpdateState>()
.having(
(s) => s.status,
'status',
LibraryUpdateStatus.needsFullConfirmation,
)
.having(
(s) => s.plan?.kind,
'plan.kind',
LibraryUpdatePlanKind.fullDownload,
)
.having(
(s) => s.message,
'message',
contains('החלת עדכון הדלתא נכשלה'),
),
],
);

blocTest<LibraryUpdateBloc, LibraryUpdateState>(
'סטיית תוכן בלי DB מלא בתוכנית → error',
build: () => _bloc(
Expand Down
Loading