fix(packages/i18n): remove unnecessary staff from tests and add question

Author: PatrykKuniczak

packages/i18n/src/__tests__/types.test.ts

@@ -4,6 +4,7 @@ import { browser } from '@wxt-dev/browser';
 
 vi.mock('@wxt-dev/browser', async () => {
   const { vi } = await import('vitest');
+
   return {
     browser: {
       i18n: {
@@ -14,7 +15,7 @@ vi.mock('@wxt-dev/browser', async () => {
 });
 const getMessageMock = vi.mocked(browser.i18n.getMessage);
 
-const n: number = 1;
+const n = 1;
 
 describe('I18n Types', () => {
   beforeEach(() => {
@@ -47,6 +48,7 @@ describe('I18n Types', () => {
     describe('t', () => {
       it('should only allow passing valid combinations of arguments', () => {
         i18n.t('simple');
+        // TODO: WHY THERE'S SO MUCH TS-EXPECT-ERRORS?
         // @ts-expect-error
         i18n.t('simple', []);
         // @ts-expect-error

packages/i18n/src/__tests__/utils.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, expect, it } from 'vitest';
 import { ChromeMessage } from '../build';
 import {
   applyChromeMessagePlaceholders,
@@ -43,7 +43,7 @@ describe('Utils', () => {
   });
 
   describe('getSubstitutionCount', () => {
-    it('should return the last substution present in the message', () => {
+    it('should return the last substitution present in the message', () => {
       expect(getSubstitutionCount('I like $1, but I like $2 better')).toBe(2);
     });