From a1e9238bf5bca418f02b6daa14e6d9b4f4c5fb80 Mon Sep 17 00:00:00 2001
From: Gauthier Roebroeck <gauthier.roebroeck@gmail.com>
Date: Tue, 14 Dec 2021 13:58:58 +0800
Subject: [PATCH] test: fix tests

---
 komga-webui/src/functions/book-spreads.ts     |  1 +
 .../tests/unit/functions/book-spreads.spec.ts | 26 ++++++++++++-------
 2 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/komga-webui/src/functions/book-spreads.ts b/komga-webui/src/functions/book-spreads.ts
index fba8b3fb2..98a92afb0 100644
--- a/komga-webui/src/functions/book-spreads.ts
+++ b/komga-webui/src/functions/book-spreads.ts
@@ -46,6 +46,7 @@ export function buildSpreads(pages: PageDtoWithUrl[], pageLayout: PagedReaderLay
 function createEmptyPage(page: PageDtoWithUrl): PageDtoWithUrl {
   return {
     url: createTransparentDataUrl(page?.width || 20, page?.height || 30),
+    number: 0,
   } as PageDtoWithUrl
 }
 
diff --git a/komga-webui/tests/unit/functions/book-spreads.spec.ts b/komga-webui/tests/unit/functions/book-spreads.spec.ts
index 6dd0dc668..df382912f 100644
--- a/komga-webui/tests/unit/functions/book-spreads.spec.ts
+++ b/komga-webui/tests/unit/functions/book-spreads.spec.ts
@@ -49,8 +49,9 @@ describe('Double Pages', () => {
     const spreads = buildSpreads(pages, pageLayout)
 
     expect(spreads.length).toEqual(1)
-    expect(spreads[0].length).toEqual(1)
-    expect(spreads[0][0].number).toEqual(1)
+    expect(spreads[0].length).toEqual(2)
+    expect(spreads[0][1].number).toEqual(1)
+    expect(spreads[0][0].number).toEqual(0) // empty page
   })
 
   test('given even pages then it should return correct spreads', () => {
@@ -67,8 +68,9 @@ describe('Double Pages', () => {
 
     expect(spreads.length).toEqual(4)
 
-    expect(spreads[0].length).toEqual(1)
-    expect(spreads[0][0].number).toEqual(1)
+    expect(spreads[0].length).toEqual(2)
+    expect(spreads[0][1].number).toEqual(1)
+    expect(spreads[0][0].number).toEqual(0)  // empty page
 
     expect(spreads[1].length).toEqual(2)
     expect(spreads[1][0].number).toEqual(2)
@@ -78,8 +80,9 @@ describe('Double Pages', () => {
     expect(spreads[2][0].number).toEqual(4)
     expect(spreads[2][1].number).toEqual(5)
 
-    expect(spreads[3].length).toEqual(1)
+    expect(spreads[3].length).toEqual(2)
     expect(spreads[3][0].number).toEqual(6)
+    expect(spreads[3][1].number).toEqual(0) // empty page
   })
 
   test('given odd pages then it should return correct spreads', () => {
@@ -95,18 +98,20 @@ describe('Double Pages', () => {
 
     expect(spreads.length).toEqual(4)
 
-    expect(spreads[0].length).toEqual(1)
-    expect(spreads[0][0].number).toEqual(1)
+    expect(spreads[0].length).toEqual(2)
+    expect(spreads[0][1].number).toEqual(1)
+    expect(spreads[0][0].number).toEqual(0) // empty page
 
     expect(spreads[1].length).toEqual(2)
     expect(spreads[1][0].number).toEqual(2)
     expect(spreads[1][1].number).toEqual(3)
 
-    expect(spreads[2].length).toEqual(1)
+    expect(spreads[2].length).toEqual(2)
     expect(spreads[2][0].number).toEqual(4)
 
-    expect(spreads[3].length).toEqual(1)
+    expect(spreads[3].length).toEqual(2)
     expect(spreads[3][0].number).toEqual(5)
+    expect(spreads[3][1].number).toEqual(0) // empty page
   })
 })
 
@@ -131,7 +136,8 @@ describe('Double Pages No Cover', () => {
     const spreads = buildSpreads(pages, pageLayout)
 
     expect(spreads.length).toEqual(1)
-    expect(spreads[0].length).toEqual(1)
+    expect(spreads[0].length).toEqual(2)
     expect(spreads[0][0].number).toEqual(1)
+    expect(spreads[0][1].number).toEqual(0) // empty page
   })
 })