refactoring

intlify · Apr 29, 2020 · 7b1211a · 7b1211a
1 parent 0e0b2bb
commit 7b1211a
Show file tree

Hide file tree

Showing 5 changed files with 54 additions and 34 deletions.
diff --git a/test/composer.test.ts b/test/composer.test.ts
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-empty-function */
+
 // utils
 jest.mock('../src/utils', () => ({
   ...jest.requireActual('../src/utils'),
@@ -287,7 +289,7 @@ describe('fallbackFormat', () => {
 
   test('interpolation', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const { t } = createComposer({
       locale: 'en',
@@ -344,13 +346,13 @@ describe('getMissingHandler / setMissingHandler', () => {
     const { getMissingHandler, setMissingHandler } = createComposer({})
     expect(getMissingHandler()).toEqual(null)
 
-    const missing = () => {} // eslint-disable-line @typescript-eslint/no-empty-function
+    const missing = () => {}
     setMissingHandler(missing as MissingHandler)
     expect(getMissingHandler()).toEqual(missing)
   })
 
   test('initialize at composer creating', () => {
-    const missing = () => {} // eslint-disable-line @typescript-eslint/no-empty-function
+    const missing = () => {}
     const { getMissingHandler } = createComposer({ missing })
     expect(getMissingHandler()).toEqual(missing)
   })
@@ -713,3 +715,5 @@ describe('__i18n', () => {
     })
   })
 })
+
+/* eslint-enable @typescript-eslint/no-empty-function */
diff --git a/test/legacy.test.ts b/test/legacy.test.ts
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-empty-function */
+
 // utils
 jest.mock('../src/utils', () => ({
   ...jest.requireActual('../src/utils'),
@@ -35,7 +37,7 @@ test('availableLocales', () => {
 
 test('formatter', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
 
   const i18n = createVueI18n({
     formatter: {
@@ -407,7 +409,7 @@ test('getNumberFormat / setNumberFormat / mergeNumberFormat', () => {
 
 test('getChoiceIndex', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
 
   const i18n = createVueI18n({})
   i18n.getChoiceIndex(1, 2)
@@ -418,7 +420,7 @@ test('getChoiceIndex', () => {
 
 test('warnHtmlInMessage', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
 
   const i18n = createVueI18n({
     locale: 'en',
@@ -438,3 +440,5 @@ test('warnHtmlInMessage', () => {
   expect(i18n.t('hello')).toEqual('<p>hello</p>')
   expect(mockWarn).toHaveBeenCalledTimes(2)
 })
+
+/* eslint-enable @typescript-eslint/no-empty-function */
diff --git a/test/runtime/datetime.test.ts b/test/runtime/datetime.test.ts
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-empty-function */
+
 // utils
 jest.mock('../../src/utils', () => ({
   ...jest.requireActual('../../src/utils'),
@@ -157,7 +159,7 @@ test('override format options with number function options', () => {
 
 test('fallback', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -182,7 +184,7 @@ test('fallback', () => {
 
 test(`context fallbackWarn 'false' option`, () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -202,7 +204,7 @@ test(`context fallbackWarn 'false' option`, () => {
 
 test(`datetime function fallbackWarn 'false' option`, () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -224,7 +226,7 @@ test(`datetime function fallbackWarn 'false' option`, () => {
 describe('context unresolving option', () => {
   test('not specify fallbackLocales', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
     const mockAvailabilities = Availabilities as jest.Mocked<
       typeof Availabilities
     >
@@ -244,7 +246,7 @@ describe('context unresolving option', () => {
 
   test('not found key in fallbackLocales', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
     const mockAvailabilities = Availabilities as jest.Mocked<
       typeof Availabilities
     >
@@ -292,7 +294,7 @@ test('part', () => {
 
 test('not available Intl API', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -309,3 +311,5 @@ test('not available Intl API', () => {
     `Cannot format a Date value due to not supported Intl.DateTimeFormat.`
   )
 })
+
+/* eslint-eable @typescript-eslint/no-empty-function */
diff --git a/test/runtime/number.test.ts b/test/runtime/number.test.ts
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-empty-function */
+
 // utils
 jest.mock('../../src/utils', () => ({
   ...jest.requireActual('../../src/utils'),
@@ -138,7 +140,7 @@ test('override format options with number function options', () => {
 
 test('fallback', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -163,7 +165,7 @@ test('fallback', () => {
 
 test(`context fallbackWarn 'false' option`, () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -183,7 +185,7 @@ test(`context fallbackWarn 'false' option`, () => {
 
 test(`number function fallbackWarn 'false' option`, () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -205,7 +207,7 @@ test(`number function fallbackWarn 'false' option`, () => {
 describe('context unresolving option', () => {
   test('not specify fallbackLocales', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
     const mockAvailabilities = Availabilities as jest.Mocked<
       typeof Availabilities
     >
@@ -225,7 +227,7 @@ describe('context unresolving option', () => {
 
   test('not found key in fallbackLocales', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
     const mockAvailabilities = Availabilities as jest.Mocked<
       typeof Availabilities
     >
@@ -268,7 +270,7 @@ test('part', () => {
 
 test('not available Intl API', () => {
   const mockWarn = warn as jest.MockedFunction<typeof warn>
-  mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+  mockWarn.mockImplementation(() => {})
   const mockAvailabilities = Availabilities as jest.Mocked<
     typeof Availabilities
   >
@@ -285,3 +287,5 @@ test('not available Intl API', () => {
     `Cannot format a Date value due to not supported Intl.NumberFormat.`
   )
 })
+
+/* eslint-enable @typescript-eslint/no-empty-function */
diff --git a/test/runtime/translate.test.ts b/test/runtime/translate.test.ts
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-empty-function */
+
 // utils
 jest.mock('../../src/utils', () => ({
   ...jest.requireActual('../../src/utils'),
@@ -113,7 +115,7 @@ describe('default option', () => {
 describe('context missing option', () => {
   test('not specified missing handler', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -130,7 +132,7 @@ describe('context missing option', () => {
 
   test('specified missing handler', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -151,7 +153,7 @@ describe('context missing option', () => {
 describe('context missingWarn option', () => {
   test('false', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -168,7 +170,7 @@ describe('context missingWarn option', () => {
 
   test('regex', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -189,7 +191,7 @@ describe('context missingWarn option', () => {
 
   test('missingWarn option', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -207,7 +209,7 @@ describe('context missingWarn option', () => {
 describe('context fallbackWarn option', () => {
   test('not specify fallbackLocale', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -223,7 +225,7 @@ describe('context fallbackWarn option', () => {
 
   test('specify fallbackLocale', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -246,7 +248,7 @@ describe('context fallbackWarn option', () => {
 
   test('not found fallback message', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -270,7 +272,7 @@ describe('context fallbackWarn option', () => {
 
   test('context option: false', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -289,7 +291,7 @@ describe('context fallbackWarn option', () => {
 
   test('context option: regex', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -308,7 +310,7 @@ describe('context fallbackWarn option', () => {
 
   test('specify fallbackWarn option to translate function', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -331,7 +333,7 @@ describe('context fallbackWarn option', () => {
 describe('context fallbackFormat option', () => {
   test('specify true', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -367,7 +369,7 @@ describe('context fallbackFormat option', () => {
 
   test('overrided with default option', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -403,7 +405,7 @@ describe('context fallbackFormat option', () => {
 
   test('fallbackLocales is nothing', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -456,7 +458,7 @@ describe('context unresolving option', () => {
 
   test('fallbackFormat is true', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -530,7 +532,7 @@ describe('context postTranslation option', () => {
 describe('warnHtmlMessage', () => {
   test('default', () => {
     const mockWarn = warn as jest.MockedFunction<typeof warn>
-    mockWarn.mockImplementation(() => {}) // eslint-disable-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
     const ctx = context({
       locale: 'en',
@@ -571,3 +573,5 @@ describe('edge cases', () => {
     expect(translate(ctx, 'side.left')).toEqual('Left')
   })
 })
+
+/* eslint-enable @typescript-eslint/no-empty-function */