refactor: code cleanup

__tests__/utils/FileProcessor.test.ts

@@ -15,17 +15,16 @@ jest.mock("simple-git");
 const mockedCore = core as jest.Mocked<typeof core>;
 const mockedSimpleGit = simpleGit as jest.MockedFunction<typeof simpleGit>;
 let gitMock: jest.Mocked<SimpleGit>;
-
+let params: FilesParams;
 describe("FileProcessor", () => {
  beforeEach(() => {
  jest.clearAllMocks();
- gitMock = {
- add: jest.fn().mockImplementation(() => Promise.resolve()),
- addConfig: jest.fn().mockImplementation(() => gitMock),
- commit: jest.fn().mockImplementation(() => Promise.resolve()),
- push: jest.fn().mockImplementation(() => Promise.resolve())
- } as unknown as jest.Mocked<SimpleGit>;
- mockedSimpleGit.mockReturnValue(gitMock);
+ params = {
+ files: ["test.txt"],
+ variables: new Map([["name", "Breno"]]),
+ encodings: "utf-8",
+ extension: ".txt"
+ }
  });
  describe("processFiles", () => {
  it("processes files correctly", () => {
@@ -34,31 +33,14 @@ describe("FileProcessor", () => {
  mockReadFileSync.mockReturnValue("Hello, {_name_}!");
  mockWriteFileSync.mockImplementation(() => {
  });
-
- const params: FilesParams = {
- files: ["test.txt"],
- variables: new Map([["name", "Breno"]]),
- encodings: "utf-8",
- extension: ".txt"
- };
-
  processFiles(params);
-
  expect(mockReadFileSync).toHaveBeenCalledWith("test.txt", { encoding: "utf-8" });
  expect(mockWriteFileSync).toHaveBeenCalledWith("test", "Hello, Breno!", { encoding: "utf-8" });
  });
 
  it("throws error when file cannot be read", async () => {
  const mockReadFileSync = jest.spyOn(fs, "readFileSync");
  mockReadFileSync.mockReturnValue("");
-
- const params: FilesParams = {
- files: ["test.txt"],
- variables: new Map([["name", "Breno"]]),
- encodings: "utf-8",
- extension: ".txt"
- };
-
  await expect(processFiles(params)).rejects.toThrow("Error reading file: test.txt");
  });
  });
@@ -67,40 +49,43 @@ describe("FileProcessor", () => {
  it("replaces variables correctly", () => {
  const variables = new Map([["name", "Breno"]]);
  const content = "Hello, {_name_}!";
-
  const result = replaceVariables(variables, content);
-
  expect(result).toBe("Hello, Breno!");
  });
 
  it("returns original content when no variables match", () => {
  const variables = new Map([["name", "Breno"]]);
  const content = "Hello, world!";
-
  const result = replaceVariables(variables, content);
-
  expect(result).toBe("Hello, world!");
  });
  });
 
  describe("gitAdd", () => {
- it("git add and commit", async () => {
- const mockReadFileSync = jest.spyOn(fs, "readFileSync");
- const mockWriteFileSync = jest.spyOn(fs, "writeFileSync");
- mockReadFileSync.mockReturnValue("Hello, {_name_}!");
- mockWriteFileSync.mockImplementation(() => {
- });
-
- const params: FilesParams = {
+ beforeEach(() => {
+ gitMock = {
+ add: jest.fn().mockImplementation(() => Promise.resolve()),
+ addConfig: jest.fn().mockImplementation(() => gitMock),
+ commit: jest.fn().mockImplementation(() => Promise.resolve()),
+ push: jest.fn().mockImplementation(() => Promise.resolve())
+ } as unknown as jest.Mocked<SimpleGit>;
+ mockedSimpleGit.mockReturnValue(gitMock);
+ params = {
  files: ["test.prp.txt"],
  variables: new Map([["name", "Breno"]]),
  encodings: "utf-8",
  extension: ".prp",
  git: gitMock
  };
+ });
 
+ it("git add and commit", async () => {
+ const mockReadFileSync = jest.spyOn(fs, "readFileSync");
+ const mockWriteFileSync = jest.spyOn(fs, "writeFileSync");
+ mockReadFileSync.mockReturnValue("Hello, {_name_}!");
+ mockWriteFileSync.mockImplementation(() => {
+ });
  await processFiles(params);
-
  expect(mockReadFileSync).toHaveBeenCalledWith("test.prp.txt", { encoding: "utf-8" });
  expect(mockWriteFileSync).toHaveBeenCalledWith("test.txt", "Hello, Breno!", { encoding: "utf-8" });
  expect(gitMock.add).toHaveBeenCalledWith("test.txt");
@@ -113,15 +98,6 @@ describe("FileProcessor", () => {
  });
 
  gitMock.add.mockRejectedValue(new GitError(undefined, "Error adding file"));
-
- const params: FilesParams = {
- files: ["test.prp.txt"],
- variables: new Map([["name", "Breno"]]),
- encodings: "utf-8",
- extension: ".prp",
- git: gitMock
- };
-
  await processFiles(params);
 
  expect(mockReadFileSync).toHaveBeenCalledWith("test.prp.txt", { encoding: "utf-8" });

src/main.ts

@@ -1,7 +1,6 @@
 import * as core from "@actions/core";
 import { getFilesByExtension } from "./utils/ExtensionFilter";
 import { InputParams } from "./utils/VariableManager";
-import { SEARCH_TEXT } from "./utils/texts";
 import { processFiles, replaceVariables } from "./utils/FileProcessor";
 import path from "path";
 import simpleGit, { GitError, SimpleGit } from "simple-git";
@@ -13,7 +12,9 @@ import simpleGit, { GitError, SimpleGit } from "simple-git";
  */
 export async function run(inputParams: InputParams): Promise<void> {
  const { rootDir, extension, ignoredDir, includeSubDir } = inputParams;
- core.debug(SEARCH_TEXT(extension, rootDir));
+ core.debug(
+ `Searching for files with extension: ${extension} in directory: ${rootDir}...`
+ );
  const files: string[] = getFilesByExtension({
  dir: path.join(rootDir),
  extension,