fix(git tests): Reset server repository manually

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
floccusaddon · Jan 14, 2024 · e837197 · e837197
1 parent c0376e3
commit e837197
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 16 deletions.
diff --git a/src/lib/adapters/Git.ts b/src/lib/adapters/Git.ts
@@ -346,6 +346,21 @@ export default class GitAdapter extends CachingAdapter {
       force: true,
       onAuth: () => this.onAuth()
     })
+    await git.fetch({
+      http,
+      fs,
+      dir: this.dir,
+      tags: true,
+      pruneTags: true,
+      remote: 'origin',
+      depth: 10,
+      onAuth: () => this.onAuth()
+    })
+    const tags = await git.listTags({ fs, dir: this.dir })
+    const lockTags = tags.filter(tag => tag.startsWith('floccus-lock-'))
+    for (const tag of lockTags) {
+      await git.push({ fs, http, dir: this.dir, ref: tag, delete: true, onAuth: () => this.onAuth() })
+    }
   }
 }
 

diff --git a/src/test/test.js b/src/test/test.js
@@ -258,7 +258,11 @@ describe('Floccus', function() {
             } catch (e) {
               console.error(e)
             }
-            if (ACCOUNT_DATA.type !== 'fake') {
+            if (ACCOUNT_DATA.type === 'git') {
+              await account.server.clearServer()
+              GitAdapter.resetFilesystem()
+            }
+            else if (ACCOUNT_DATA.type !== 'fake') {
               await account.setData({ ...account.getData(), serverRoot: null })
               const tree = await getAllBookmarks(account)
               await withSyncConnection(account, async() => {
@@ -278,9 +282,6 @@ describe('Floccus', function() {
                 await account.server.deleteFile(file.id)
               }
             }
-            if (ACCOUNT_DATA.type === 'git') {
-              GitAdapter.resetFilesystem()
-            }
             await account.delete()
           })
           it('should create local bookmarks on the server', async function() {
@@ -3162,7 +3163,10 @@ describe('Floccus', function() {
           })
           afterEach('clean up accounts', async function() {
             await browser.bookmarks.removeTree(account1.getData().localRoot)
-            if (ACCOUNT_DATA.type !== 'fake') {
+            if (ACCOUNT_DATA.type === 'git') {
+              await account1.server.clearServer()
+              GitAdapter.resetFilesystem()
+            }else if (ACCOUNT_DATA.type !== 'fake') {
               await account1.setData({
                 ...account1.getData(),
                 serverRoot: null
@@ -3185,9 +3189,6 @@ describe('Floccus', function() {
                 await account1.server.deleteFile(file.id)
               }
             }
-            if (ACCOUNT_DATA.type === 'git') {
-              GitAdapter.resetFilesystem()
-            }
             await account1.delete()
             await browser.bookmarks.removeTree(account2.getData().localRoot)
             await account2.delete()
@@ -4556,7 +4557,10 @@ describe('Floccus', function() {
             } catch (e) {
               console.error(e)
             }
-            if (ACCOUNT_DATA.type !== 'fake') {
+            if (ACCOUNT_DATA.type === 'git') {
+              await account.server.clearServer()
+              GitAdapter.resetFilesystem()
+            }else if (ACCOUNT_DATA.type !== 'fake') {
               await account.setData({ ...account.getData(), serverRoot: null })
               const tree = await getAllBookmarks(account)
               await withSyncConnection(account, async() => {
@@ -4576,9 +4580,6 @@ describe('Floccus', function() {
                 await account.server.deleteFile(file.id)
               }
             }
-            if (ACCOUNT_DATA.type === 'git') {
-              GitAdapter.resetFilesystem()
-            }
             await account.delete()
           })
           it('should create local tabs on the server', async function() {
@@ -4876,7 +4877,10 @@ describe('Floccus', function() {
         afterEach('clean up accounts', async function() {
           RUN_INTERRUPTS = false
           await browser.bookmarks.removeTree(account1.getData().localRoot)
-          if (ACCOUNT_DATA.type !== 'fake') {
+          if (ACCOUNT_DATA.type === 'git') {
+            await account1.server.clearServer()
+            GitAdapter.resetFilesystem()
+          } else if (ACCOUNT_DATA.type !== 'fake') {
             await account1.setData({
               ...account1.getData(),
               serverRoot: null
@@ -4899,9 +4903,6 @@ describe('Floccus', function() {
               await account1.server.deleteFile(file.id)
             }
           }
-          if (ACCOUNT_DATA.type === 'git') {
-            GitAdapter.resetFilesystem()
-          }
           await account1.delete()
           await browser.bookmarks.removeTree(account2.getData().localRoot)
           await account2.delete()