Update TypeScript version and add tests for task management

- Downgraded TypeScript dependency from 5.8.3 to 5.6.2 in the ts-uploader package for compatibility.
- Added new tests for the chainActions function to ensure proper execution order of tasks.
- Enhanced TaskManager tests to verify task creation and promise resolution, improving overall test coverage.

Author: Oksamies

packages/ts-uploader/package.json

@@ -20,7 +20,7 @@
     "@thunderstore/thunderstore-api": "^0.1.0",
     "@thunderstore/typed-event-emitter": "*",
     "crypto-js": "^4.2.0",
-    "typescript": "^5.8.3"
+    "typescript": "^5.6.2"
   },
   "devDependencies": {
     "@types/crypto-js": "^4.1.3",

packages/ts-uploader/src/tasks/__tests__/TaskManager.test.ts

@@ -13,6 +13,33 @@ describe("TaskManager", () => {
     vi.clearAllMocks();
   });
 
+  describe("tasks", () => {
+    it("should get tasks", async () => {
+      expect(taskManager.tasks.length).toBe(0);
+      taskManager.createTask(async () => {}, {});
+      taskManager.createTask(async () => {}, {});
+      expect(taskManager.createdTasks.length).toBe(2);
+    });
+  });
+
+  describe("taskPromises", () => {
+    it("should get taskPromises", async () => {
+      expect(taskManager.tasks.length).toBe(0);
+      expect(taskManager.taskPromises.length).toBe(0);
+      taskManager.waitTask(
+        taskManager.startTask(taskManager.createTask(async () => {}, {}))
+      );
+      taskManager.waitTask(
+        taskManager.startTask(taskManager.createTask(async () => {}, {}))
+      );
+      expect(taskManager.tasks.length).toBe(4);
+      expect(taskManager.taskPromises.length).toBe(2);
+      await taskManager.resolveTaskPromises();
+      expect(taskManager.tasks.length).toBe(6);
+      expect(taskManager.taskPromises.length).toBe(2);
+    });
+  });
+
   describe("createdTasks", () => {
     it("should get tasks with PENDING status", async () => {
       expect(taskManager.createdTasks.length).toBe(0);
@@ -135,6 +162,28 @@ describe("TaskManager", () => {
     });
   });
 
+  describe("resolveTaskPromises", () => {
+    it("should resolve promises in _tasks and add the resolved tasks to _tasks", async () => {
+      expect(taskManager.tasks.length).toBe(0);
+      expect(taskManager.taskPromises.length).toBe(0);
+      const task1 = taskManager.waitTask(
+        taskManager.startTask(taskManager.createTask(async () => {}, {}))
+      );
+      const task2 = taskManager.waitTask(
+        taskManager.startTask(taskManager.createTask(async () => {}, {}))
+      );
+      expect(taskManager.tasks.length).toBe(4);
+      expect(taskManager.taskPromises.length).toBe(2);
+      expect(taskManager.taskPromises[0]).toBe(task1);
+      expect(taskManager.taskPromises[1]).toBe(task2);
+      await taskManager.resolveTaskPromises();
+      expect(taskManager.tasks[4]).toBe(await task1);
+      expect(taskManager.tasks[5]).toBe(await task2);
+      expect(taskManager.tasks.length).toBe(6);
+      expect(taskManager.taskPromises.length).toBe(2);
+    });
+  });
+
   describe("createTask", () => {
     it("should create a task and add it to the task manager tasks array", () => {
       const task = taskManager.createTask(async () => {}, {});

packages/ts-uploader/src/tasks/__tests__/chain.test.ts

@@ -0,0 +1,28 @@
+import { describe, it, expect } from "vitest";
+import { createTask, startTask, waitTask } from "../task";
+import { TaskStatus, TaskFinishReason } from "../types";
+import { chainActions } from "../chain";
+
+describe("Chain Module", () => {
+  describe("chainActions", () => {
+    it("should call and await actions in order", async () => {
+      const args = 5;
+      const action1 = async (args: number) => args * 2;
+      const action2 = async (args: number) => args * 2;
+
+      const chainedAction = chainActions(action1, action2);
+
+      const createdTask = createTask(chainedAction, args);
+      const startedTask = startTask(createdTask);
+      const taskPromise = waitTask(startedTask);
+      const finishedTask = await taskPromise;
+      expect(finishedTask.status).toBe(TaskStatus.FINISHED);
+      expect(finishedTask.finishReason).toBe(TaskFinishReason.SUCCESS);
+      if (finishedTask.finishReason === TaskFinishReason.SUCCESS) {
+        expect(finishedTask.result).toBe(20);
+      } else {
+        throw new Error("Task didn't finnish successfully");
+      }
+    });
+  });
+});

yarn.lock

@@ -15631,11 +15631,6 @@ typescript@^5.6.2:
   resolved "https://registry.yarnpkg.com/typescript/-/typescript-5.6.2.tgz#d1de67b6bef77c41823f822df8f0b3bcff60a5a0"
   integrity sha512-NW8ByodCSNCwZeghjN3o+JX5OFH0Ojg6sadjEKY4huZ52TqbJTJnDo5+Tw98lSy63NZvi4n+ez5m2u5d4PkZyw==
 
-typescript@^5.8.3:
-  version "5.8.3"
-  resolved "https://registry.npmjs.org/typescript/-/typescript-5.8.3.tgz#92f8a3e5e3cf497356f4178c34cd65a7f5e8440e"
-  integrity sha512-p1diW6TqL9L07nNxvRMM7hMMw4c5XOo/1ibL4aAIGmSAt9slTE1Xgw5KWuof2uTOvCg9BY7ZRi+GaF+7sfgPeQ==
-
 ufo@^1.5.3:
   version "1.5.4"
   resolved "https://registry.yarnpkg.com/ufo/-/ufo-1.5.4.tgz#16d6949674ca0c9e0fbbae1fa20a71d7b1ded754"