From 0ff53cccd47347437745d1e5d350cf260436b9a3 Mon Sep 17 00:00:00 2001
From: "CORP\\kotov.alexander" <alexander.kotov@devexpress.com>
Date: Thu, 14 Apr 2022 12:54:21 +0400
Subject: [PATCH] tests refactoring

---
 test/unit/modules/uploader.js | 76 ++++++++++++-----------------------
 1 file changed, 25 insertions(+), 51 deletions(-)

diff --git a/test/unit/modules/uploader.js b/test/unit/modules/uploader.js
index 071e19e74e..fa44084e3f 100644
--- a/test/unit/modules/uploader.js
+++ b/test/unit/modules/uploader.js
@@ -109,63 +109,37 @@ describe('Uploader', function() {
       expect(dropEvent.defaultPrevented).toBeTrue();
     });
 
-    it('check preventImageUploading on', function() {
-      const testRange = new Range(0);
-      const file = {
-        name: 'test.png',
-        type: 'image/png',
-      };
-      let uploads = [];
-
-      const quillMock = {
-        root: {
-          addEventListener: () => {},
-        },
-      };
-
-      const uploaderInstance = new Uploader(quillMock, {
-        mimetypes: Uploader.DEFAULTS.mimetypes,
-        handler: (range, files) => {
-          uploads = files;
-        },
-      });
-
-      uploaderInstance.preventImageUploading(true);
+    [true, false].forEach(preventValue => {
+      it(`check preventImageUploading ${preventValue}`, function() {
+        const testRange = new Range(0);
+        const file = {
+          name: 'test.png',
+          type: 'image/png',
+        };
+        const expectedUploadsCount = preventValue ? 0 : 1;
+        let uploads = [];
 
-      uploaderInstance.upload(testRange, [file]);
+        const quillMock = {
+          root: {
+            addEventListener: () => {},
+          },
+        };
 
-      expect(uploaderInstance.preventImageUploading()).toBeTrue();
-      expect(uploads.length).toEqual(0);
-    });
+        const uploaderInstance = new Uploader(quillMock, {
+          mimetypes: Uploader.DEFAULTS.mimetypes,
+          handler: (range, files) => {
+            uploads = files;
+          },
+        });
 
-    it('check preventImageUploading off', function() {
-      const testRange = new Range(0);
-      const file = {
-        name: 'test.png',
-        type: 'image/png',
-      };
-      let uploads = [];
+        uploaderInstance.preventImageUploading(!preventValue);
+        uploaderInstance.preventImageUploading(preventValue);
 
-      const quillMock = {
-        root: {
-          addEventListener: () => {},
-        },
-      };
+        uploaderInstance.upload(testRange, [file]);
 
-      const uploaderInstance = new Uploader(quillMock, {
-        mimetypes: Uploader.DEFAULTS.mimetypes,
-        handler: (range, files) => {
-          uploads = files;
-        },
+        expect(uploaderInstance.preventImageUploading()).toEqual(preventValue);
+        expect(uploads.length).toEqual(expectedUploadsCount);
       });
-
-      uploaderInstance.preventImageUploading(true);
-      uploaderInstance.preventImageUploading(false);
-
-      uploaderInstance.upload(testRange, [file]);
-
-      expect(uploaderInstance.preventImageUploading()).toBeFalse();
-      expect(uploads.length).toEqual(1);
     });
   });
 });