fix: don't call overloaded versions of find functions internally (seq…

…uelize#13951)
maramizo · Jun 2, 2022 · 618339d · 618339d
1 parent 9ccd1ee
commit 618339d
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/test/unit/model/find-one.test.js b/test/unit/model/find-one.test.js
@@ -118,6 +118,23 @@ describe(Support.getTestDialectTeaser('Model'), () => {
       });
       Model.findAll = sinon.stub();
 
+      await Model.findOne();
+      Model.findAll.should.not.have.been.called;
+      Sequelize.Model.findAll.should.have.been.called;
+    });
+    it('should call internal findAll() method if findOne() is overridden', async () => {
+      const Model = current.define('model', {
+        unique1: {
+          type: DataTypes.INTEGER,
+          unique: 'unique',
+        },
+        unique2: {
+          type: DataTypes.INTEGER,
+          unique: 'unique',
+        },
+      });
+      Model.findAll = sinon.stub();
+
       await Model.findOne();
       Model.findAll.should.not.have.been.called;
       Sequelize.Model.findAll.should.have.been.called;