tests: Update string test suite

package.json

@@ -22,6 +22,7 @@
   },
   "devDependencies": {
     "@biomejs/biome": "^1.5.3",
+    "@jest/globals": "^29.7.0",
     "@types/bun": "latest"
   },
   "peerDependencies": {

test/commands/compose.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, afterAll, describe, expect, test } from 'bun:test';
+import { afterAll, beforeEach, describe, expect, test } from 'bun:test';
 import { projectConfig } from '../../src/config/project';
 import { composeExecParams, composeFiles } from '../../src/utils/compose';
 import { testSetup, testTeardown } from '../lifecycle';

test/lifecycle.ts

@@ -2,7 +2,6 @@ import { setupCommand } from '../src/commands/setup';
 import cliConfig from '../src/config/cli';
 import { projectConfig } from '../src/config/project';
 import { deletePath } from '../src/utils/file-system';
-import { omitConsoleLogs } from './helpers';
 
 const PROJECT = 'test';
 

test/utils/file-system.test.ts

@@ -10,22 +10,22 @@ import {
 describe('Utils: file-system', () => {
   test('isFile returns true if file exists', () => {
     const exists = isFile('./cli.ts');
-    expect(exists).toBeTrue();
+    expect(exists).toBe(true);
   });
 
   test("isFile returns false if file doesn't exist", () => {
     const exists = isFile('./cli.not.exists.ts');
-    expect(exists).toBeFalse();
+    expect(exists).toBe(false);
   });
 
   test('isDirectory returns true if dir exists', () => {
     const exists = isDirectory('./src');
-    expect(exists).toBeTrue();
+    expect(exists).toBe(true);
   });
 
   test("isDirectory returns false if dir doesn't exist", () => {
     const exists = isDirectory('./src-that-do-not-exist');
-    expect(exists).toBeFalse();
+    expect(exists).toBe(false);
   });
 
   test('createFile & copyFile create and copy a file', () => {
@@ -35,14 +35,14 @@ describe('Utils: file-system', () => {
 
     // Create and copy file with createFile() and copyFile()
     createFile({ file: file, content: content, verbose: false });
-    expect(isFile(file)).toBeTrue();
+    expect(isFile(file)).toBe(true);
     copyFile({ source: file, target: copiedFile, verbose: false });
-    expect(isFile(copiedFile)).toBeTrue();
+    expect(isFile(copiedFile)).toBe(true);
 
     // Ensure files are deleted
     fs.rmSync(file);
     fs.rmSync(copiedFile);
-    expect(isFile(file)).toBeFalse();
-    expect(isFile(copiedFile)).toBeFalse();
+    expect(isFile(file)).toBe(false);
+    expect(isFile(copiedFile)).toBe(false);
   });
 });

test/utils/object.test.ts

@@ -18,7 +18,7 @@ describe('Utils: object', () => {
 
   test('Replaces a key value in a file', () => {
     createFile({ file: testFile, content: testFileContent });
-    expect(isFile(testFile)).toBeTrue();
+    expect(isFile(testFile)).toBe(true);
 
     replaceKeyValue(testFile, 'KEY1', 'VALUE10');
     const replacedContent = fs.readFileSync(testFile, 'utf8');
@@ -27,7 +27,7 @@ describe('Utils: object', () => {
 
   test("Replaces various keys' values in a file", () => {
     createFile({ file: testFile, content: testFileContent });
-    expect(isFile(testFile)).toBeTrue();
+    expect(isFile(testFile)).toBe(true);
 
     replaceKeysInFile(testFile, {
       KEY1: 'VALUE_KEY1',

test/utils/string.test.ts

@@ -6,32 +6,47 @@ import {
   removeExtraSpaces,
 } from '../../src/utils/string';
 
-describe('Utils: string', () => {
-  test('first letter of a word is capitalized', () => {
-    const lowerWord = 'lowercase';
-    const capWord = capitalizeFirstLetter(lowerWord);
-    expect(capWord).toBeString();
-    expect(capWord).toEqual('Lowercase');
+describe('Utils (string): hyphenToUnderscore', () => {
+  test('replaces hyphens with underscores', () => {
+    expect(hyphenToUnderscore('hello-world')).toBe('hello_world');
   });
 
-  test('extra spaces are removed from a string', () => {
-    const sentence = '  this    is   a    test   ';
-    const noSpaceSentence = removeExtraSpaces(sentence);
-    expect(noSpaceSentence).toBeString();
-    expect(noSpaceSentence).toEqual('this is a test');
+  test('handles multiple hyphens', () => {
+    expect(hyphenToUnderscore('foo-bar-baz')).toBe('foo_bar_baz');
   });
 
-  test('all break lines are removed from string', () => {
-    const sentence = '\nthis \nis \na \nanother \ntest\n';
-    const noSpaceSentence = removeBreakLines(sentence);
-    expect(noSpaceSentence).toBeString();
-    expect(noSpaceSentence).toEqual('this is a another test');
+  test('does not change words without hyphens', () => {
+    expect(hyphenToUnderscore('hello')).toBe('hello');
+  });
+});
+
+describe('Utils (string): capitalizeFirstLetter', () => {
+  test('capitalizes the first letter of a word', () => {
+    expect(capitalizeFirstLetter('hello')).toBe('Hello');
+  });
+
+  test('does not change an already capitalized word', () => {
+    expect(capitalizeFirstLetter('World')).toBe('World');
+  });
+});
+
+describe('Utils (string): removeExtraSpaces', () => {
+  test('removes extra spaces from the beginning and end', () => {
+    expect(removeExtraSpaces('  hello   ')).toBe('hello');
+  });
+
+  test('replaces multiple spaces with a single space', () => {
+    expect(removeExtraSpaces('hello    world')).toBe('hello world');
+  });
+});
+
+describe('Utils (string): removeBreakLines', () => {
+  test('removes break lines from text', () => {
+    expect(removeBreakLines('hello\nworld')).toBe('helloworld');
   });
 
-  test('converts hypen to underscore', () => {
-    const string1 = 'my-word-with-multiple-hyphens-';
-    const stringUnderscore = hyphenToUnderscore(string1);
-    expect(stringUnderscore).toContain('_');
-    expect(stringUnderscore).toEqual('my_word_with_multiple_hyphens_');
+  test('handles different types of break lines', () => {
+    expect(removeBreakLines('hello\r\nworld')).toBe('helloworld');
+    expect(removeBreakLines('hello\rworld')).toBe('helloworld');
   });
 });

tsconfig.json

@@ -1,22 +1,27 @@
 {
   "compilerOptions": {
+    // Enable latest features
     "lib": ["ESNext"],
     "target": "ESNext",
     "module": "ESNext",
     "moduleDetection": "force",
     "jsx": "react-jsx",
     "allowJs": true,
 
-    /* Bundler mode */
-    "moduleResolution": "Bundler",
+    // Bundler mode
+    "moduleResolution": "bundler",
     "allowImportingTsExtensions": true,
     "verbatimModuleSyntax": true,
     "noEmit": true,
 
-    /* Linting */
-    "skipLibCheck": true,
+    // Best practices
     "strict": true,
+    "skipLibCheck": true,
     "noFallthroughCasesInSwitch": true,
-    "forceConsistentCasingInFileNames": true
+
+    // Some stricter flags (disabled by default)
+    "noUnusedLocals": false,
+    "noUnusedParameters": false,
+    "noPropertyAccessFromIndexSignature": false
   }
 }