From 0de1358924e92a863f70a8b9186203e7ac8c7743 Mon Sep 17 00:00:00 2001
From: afc163 <afc163@gmail.com>
Date: Tue, 15 Sep 2020 11:54:47 +0800
Subject: [PATCH] =?UTF-8?q?test:=20=E2=9C=85=20fix=20eslint=20jest=20rules?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .eslintrc.js                                | 3 +++
 components/affix/__tests__/Affix.test.tsx   | 2 +-
 components/calendar/__tests__/index.test.js | 4 ++--
 components/carousel/__tests__/index.test.js | 8 ++------
 components/upload/__tests__/upload.test.js  | 2 +-
 package.json                                | 4 ++--
 6 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 71210e8668..47d116601c 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -113,6 +113,9 @@ module.exports = {
 
     'jest/no-test-callback': 0,
     'jest/expect-expect': 0,
+    'jest/no-done-callback': 0,
+    'jest/valid-title': 0,
+    'jest/no-conditional-expect': 0,
 
     'unicorn/better-regex': 2,
     'unicorn/prefer-trim-start-end': 2,
diff --git a/components/affix/__tests__/Affix.test.tsx b/components/affix/__tests__/Affix.test.tsx
index b215a9e7b2..9281f2a9c7 100644
--- a/components/affix/__tests__/Affix.test.tsx
+++ b/components/affix/__tests__/Affix.test.tsx
@@ -186,7 +186,7 @@ describe('Affix Render', () => {
     it.each([
       { name: 'inner', index: 0 },
       { name: 'outer', index: 1 },
-    ])(name, async ({ index }) => {
+    ])('inner or outer', async ({ index }) => {
       document.body.innerHTML = '<div id="mounter" />';
 
       const updateCalled = jest.fn();
diff --git a/components/calendar/__tests__/index.test.js b/components/calendar/__tests__/index.test.js
index ad603131f3..c4204c3130 100644
--- a/components/calendar/__tests__/index.test.js
+++ b/components/calendar/__tests__/index.test.js
@@ -214,7 +214,7 @@ describe('Calendar', () => {
     expect(onValueChange).toHaveBeenCalledWith(value.year('2019').month('3'));
   });
 
-  it('if start.month > value.month, set value.month to start.month ', () => {
+  it('if start.month > value.month, set value.month to start.month', () => {
     const value = new Moment('1990-01-03');
     const start = new Moment('2019-11-01');
     const end = new Moment('2019-03-01');
@@ -223,7 +223,7 @@ describe('Calendar', () => {
     expect(onValueChange).toHaveBeenCalledWith(value.year('2019').month('10'));
   });
 
-  it('if change year and new month > end month, set value.month to end.month ', () => {
+  it('if change year and new month > end month, set value.month to end.month', () => {
     const value = new Moment('2018-11-03');
     const start = new Moment('2000-01-01');
     const end = new Moment('2019-03-01');
diff --git a/components/carousel/__tests__/index.test.js b/components/carousel/__tests__/index.test.js
index 8927384cc3..88d91f21be 100644
--- a/components/carousel/__tests__/index.test.js
+++ b/components/carousel/__tests__/index.test.js
@@ -86,6 +86,7 @@ describe('Carousel', () => {
 
   describe('should works for dotPosition', () => {
     ['left', 'right', 'top', 'bottom'].forEach(dotPosition => {
+      // eslint-disable-next-line jest/valid-title
       it(dotPosition, () => {
         const wrapper = mount(
           <Carousel dotPosition={dotPosition}>
@@ -118,12 +119,7 @@ describe('Carousel', () => {
         children: [<div key="1" />, <div key="2" />, <div key="3" />],
       });
       wrapper.update();
-      expect(
-        wrapper
-          .find('.slick-dots li')
-          .at(1)
-          .hasClass('slick-active'),
-      ).toBeTruthy();
+      expect(wrapper.find('.slick-dots li').at(1).hasClass('slick-active')).toBeTruthy();
     });
   });
 
diff --git a/components/upload/__tests__/upload.test.js b/components/upload/__tests__/upload.test.js
index 407adb6ed8..d159e0968d 100644
--- a/components/upload/__tests__/upload.test.js
+++ b/components/upload/__tests__/upload.test.js
@@ -539,7 +539,7 @@ describe('Upload', () => {
     errorSpy.mockRestore();
   });
 
-  it('it should be treated as file but not an image', () => {
+  it('should be treated as file but not an image', () => {
     const file = {
       status: 'done',
       uid: '-1',
diff --git a/package.json b/package.json
index 0be9a1ec77..13705968b8 100644
--- a/package.json
+++ b/package.json
@@ -162,7 +162,7 @@
     "@types/enzyme": "^3.10.5",
     "@types/gtag.js": "^0.0.3",
     "@types/jest": "^26.0.0",
-    "@types/jest-image-snapshot": "^3.1.0",
+    "@types/jest-image-snapshot": "^4.1.0",
     "@types/lodash": "^4.14.139",
     "@types/puppeteer": "^3.0.0",
     "@types/raf": "^3.4.0",
@@ -202,7 +202,7 @@
     "eslint-config-prettier": "^6.0.0",
     "eslint-plugin-babel": "^5.3.0",
     "eslint-plugin-import": "^2.21.1",
-    "eslint-plugin-jest": "^23.0.2",
+    "eslint-plugin-jest": "^24.0.1",
     "eslint-plugin-jsx-a11y": "^6.2.1",
     "eslint-plugin-markdown": "^1.0.0",
     "eslint-plugin-react": "^7.14.2",
-- 
GitLab