Browse Source

remove eslint-disable indent lines

Yuki Takei 2 months ago
parent
commit
73f6c1fc26

+ 0 - 1
apps/app/src/server/service/acl.integ.ts

@@ -176,7 +176,6 @@ describe('AclService test', () => {
       expect(result).toBe(true);
     });
 
-    /* eslint-disable indent */
     describe.each`
       restrictGuestMode | expected
       ${undefined}      | ${false}

+ 0 - 1
apps/app/src/server/service/passport.spec.ts

@@ -40,7 +40,6 @@ describe('PassportService test', () => {
       );
     });
 
-    /* eslint-disable indent */
     let i = 0;
     describe.each`
       conditionId | departments   | positions     | ruleStr                                                   | expected

+ 0 - 3
apps/app/src/services/renderer/rehype-plugins/relative-links-by-pukiwiki-like-linker.spec.ts

@@ -8,7 +8,6 @@ import { pukiwikiLikeLinker } from '../remark-plugins/pukiwiki-like-linker';
 import { relativeLinksByPukiwikiLikeLinker } from './relative-links-by-pukiwiki-like-linker';
 
 describe('relativeLinksByPukiwikiLikeLinker', () => {
-  /* eslint-disable indent */
   describe.each`
     input                              | expectedHref                       | expectedValue
     ${'[[/page]]'}                     | ${'/page'}                         | ${'/page'}
@@ -22,8 +21,6 @@ describe('relativeLinksByPukiwikiLikeLinker', () => {
   `(
     'should convert relative links correctly',
     ({ input, expectedHref, expectedValue }) => {
-      /* eslint-enable indent */
-
       test(`when the input is '${input}'`, () => {
         // setup:
         const processor = unified()

+ 0 - 6
apps/app/test/integration/middlewares/login-required.test.js

@@ -49,7 +49,6 @@ describe('loginRequired', () => {
           .mockImplementation(() => false);
       });
 
-      /* eslint-disable indent */
       test.each`
         userStatus | expectedPath
         ${1}       | ${'/login/error/registered'}
@@ -75,7 +74,6 @@ describe('loginRequired', () => {
           expect(req.session.redirectTo).toBe(undefined);
         },
       );
-      /* eslint-disable indent */
 
       test("redirect to '/login' when the user does not loggedin", () => {
         req.baseUrl = '/path/that/requires/loggedin';
@@ -125,7 +123,6 @@ describe('loginRequired', () => {
           .mockImplementation(() => true);
       });
 
-      /* eslint-disable indent */
       test.each`
         userStatus | expectedPath
         ${1}       | ${'/login/error/registered'}
@@ -151,7 +148,6 @@ describe('loginRequired', () => {
           expect(req.session.redirectTo).toBe(undefined);
         },
       );
-      /* eslint-disable indent */
 
       test('pass guest user', () => {
         const result = loginRequired(req, res, next);
@@ -251,7 +247,6 @@ describe('loginRequired', () => {
       expect(req.session.redirectTo).toBe(undefined);
     });
 
-    /* eslint-disable indent */
     test.each`
       userStatus | expectedPath
       ${1}       | ${'/login/error/registered'}
@@ -277,7 +272,6 @@ describe('loginRequired', () => {
         expect(req.session.redirectTo).toBe(undefined);
       },
     );
-    /* eslint-disable indent */
 
     test("redirect to '/login' when user.status is 'STATUS_DELETED'", () => {
       const User = crowi.model('User');

+ 0 - 16
packages/core/src/interfaces/primitive/string.spec.ts

@@ -10,7 +10,6 @@ import {
 } from './string';
 
 describe('isNonEmptyString', () => {
-  /* eslint-disable indent */
   it.each`
     input        | expected | description
     ${'hello'}   | ${true}  | ${'non-empty string'}
@@ -25,14 +24,12 @@ describe('isNonEmptyString', () => {
   `(
     'should return $expected for $description: $input',
     ({ input, expected }) => {
-      /* eslint-enable indent */
       expect(isNonEmptyString(input)).toBe(expected);
     },
   );
 });
 
 describe('isNonBlankString', () => {
-  /* eslint-disable indent */
   it.each`
     input        | expected | description
     ${'hello'}   | ${true}  | ${'non-blank string'}
@@ -49,14 +46,12 @@ describe('isNonBlankString', () => {
   `(
     'should return $expected for $description: $input',
     ({ input, expected }) => {
-      /* eslint-enable indent */
       expect(isNonBlankString(input)).toBe(expected);
     },
   );
 });
 
 describe('toNonEmptyStringOrUndefined', () => {
-  /* eslint-disable indent */
   it.each`
     input        | expected     | description
     ${'hello'}   | ${'hello'}   | ${'non-empty string'}
@@ -71,14 +66,12 @@ describe('toNonEmptyStringOrUndefined', () => {
   `(
     'should return $expected for $description: $input',
     ({ input, expected }) => {
-      /* eslint-enable indent */
       expect(toNonEmptyStringOrUndefined(input)).toBe(expected);
     },
   );
 });
 
 describe('toNonBlankStringOrUndefined', () => {
-  /* eslint-disable indent */
   it.each`
     input        | expected     | description
     ${'hello'}   | ${'hello'}   | ${'non-blank string'}
@@ -95,14 +88,12 @@ describe('toNonBlankStringOrUndefined', () => {
   `(
     'should return $expected for $description: $input',
     ({ input, expected }) => {
-      /* eslint-enable indent */
       expect(toNonBlankStringOrUndefined(input)).toBe(expected);
     },
   );
 });
 
 describe('toNonEmptyString', () => {
-  /* eslint-disable indent */
   it.each`
     input      | expected   | description
     ${'hello'} | ${'hello'} | ${'non-empty string'}
@@ -114,19 +105,16 @@ describe('toNonEmptyString', () => {
   `(
     'should return $expected for valid $description: $input',
     ({ input, expected }) => {
-      /* eslint-enable indent */
       expect(toNonEmptyString(input)).toBe(expected);
     },
   );
 
-  /* eslint-disable indent */
   it.each`
     input        | description
     ${''}        | ${'empty string'}
     ${null}      | ${'null'}
     ${undefined} | ${'undefined'}
   `('should throw error for invalid $description: $input', ({ input }) => {
-    /* eslint-enable indent */
     expect(() => toNonEmptyString(input)).toThrow(
       'Expected a non-empty string, but received:',
     );
@@ -134,7 +122,6 @@ describe('toNonEmptyString', () => {
 });
 
 describe('toNonBlankString', () => {
-  /* eslint-disable indent */
   it.each`
     input      | expected   | description
     ${'hello'} | ${'hello'} | ${'non-blank string'}
@@ -144,12 +131,10 @@ describe('toNonBlankString', () => {
   `(
     'should return $expected for valid $description: $input',
     ({ input, expected }) => {
-      /* eslint-enable indent */
       expect(toNonBlankString(input)).toBe(expected);
     },
   );
 
-  /* eslint-disable indent */
   it.each`
     input        | description
     ${' '}       | ${'space character'}
@@ -160,7 +145,6 @@ describe('toNonBlankString', () => {
     ${null}      | ${'null'}
     ${undefined} | ${'undefined'}
   `('should throw error for invalid $description: $input', ({ input }) => {
-    /* eslint-enable indent */
     expect(() => toNonBlankString(input)).toThrow(
       'Expected a non-blank string, but received:',
     );

+ 0 - 1
packages/core/src/utils/objectid-utils.spec.ts

@@ -3,7 +3,6 @@ import ObjectId from 'bson-objectid';
 import { isValidObjectId } from './objectid-utils';
 
 describe('isValidObjectId', () => {
-  /* eslint-disable indent */
   describe.concurrent.each`
     arg                                           | expected
     ${undefined}                                  | ${false}