feat: do nothing on conflict for save pull request

1024pix · Dec 23, 2024 · 4181b95 · 4181b95
1 parent 22a9f7c
commit 4181b95
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/build/repositories/pull-request-repository.js b/build/repositories/pull-request-repository.js
@@ -1,7 +1,7 @@
 import { knex } from '../../db/knex-database-connection.js';
 
 async function save({ number, repositoryName }) {
-  return knex('pull_requests').insert({ number, repositoryName });
+  return knex('pull_requests').insert({ number, repositoryName }).onConflict().ignore();
 }
 
 async function isAtLeastOneMergeInProgress(repositoryName) {

diff --git a/test/integration/build/repositories/pull-request-repository_test.js b/test/integration/build/repositories/pull-request-repository_test.js
@@ -23,6 +23,14 @@ describe('PullRequestRepository', function () {
         isMerging: false,
       });
     });
+
+    it('should ignore on conflict', async function () {
+      await pullRequestRepository.save({ number: 123, repositoryName: 'pix-sample-repo' });
+      await pullRequestRepository.save({ number: 123, repositoryName: 'pix-sample-repo' });
+
+      const result = await knex('pull_requests').select().where({ number: 123, repositoryName: 'pix-sample-repo' });
+      expect(result).to.be.lengthOf(1);
+    });
   });
 
   describe('#isAtLeastOneMergeInProgress', function () {