From fa3500e3bbd11caa0e7ad1633b85f95600cf962f Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Sat, 25 Sep 2021 16:10:08 +0900
Subject: Sort frame IDs

---
 src/background/repositories/ReadyFrameRepository.ts       | 2 +-
 test/background/repositories/ReadyFrameRepository.test.ts | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/background/repositories/ReadyFrameRepository.ts b/src/background/repositories/ReadyFrameRepository.ts
index 72ae5a4..aa5d986 100644
--- a/src/background/repositories/ReadyFrameRepository.ts
+++ b/src/background/repositories/ReadyFrameRepository.ts
@@ -36,7 +36,7 @@ export class ReadyFrameRepositoryImpl implements ReadyFrameRepository {
     if (typeof state === "undefined") {
       state = {};
     }
-    state[tabId] = (state[tabId] || []).concat(frameId);
+    state[tabId] = (state[tabId] || []).concat(frameId).sort();
     this.cache.set(REPOSITORY_KEY, state);
     return Promise.resolve();
   }
diff --git a/test/background/repositories/ReadyFrameRepository.test.ts b/test/background/repositories/ReadyFrameRepository.test.ts
index d952a9b..888f3f5 100644
--- a/test/background/repositories/ReadyFrameRepository.test.ts
+++ b/test/background/repositories/ReadyFrameRepository.test.ts
@@ -12,10 +12,11 @@ describe("background/repositories/ReadyFrameRepositoryImpl", () => {
     expect(await sut.getFrameIds(1)).to.be.undefined;
 
     await sut.addFrameId(1, 10);
+    await sut.addFrameId(1, 12);
     await sut.addFrameId(1, 11);
     await sut.addFrameId(2, 20);
 
-    expect(await sut.getFrameIds(1)).to.deep.equal([10, 11]);
+    expect(await sut.getFrameIds(1)).to.deep.equal([10, 11, 12]);
     expect(await sut.getFrameIds(2)).to.deep.equal([20]);
 
     await sut.clearFrameIds(1);
-- 
cgit v1.2.3