From 437c2ba2feb1ef07ff83ef66cbc61288204ff78b Mon Sep 17 00:00:00 2001 From: Lint Action Date: Thu, 24 Apr 2025 14:57:01 +0000 Subject: [PATCH] style: fix linting issues met Prettier --- .../assignments-controller.test.ts | 108 ++++++++--------- .../controllers/classes-controller.test.ts | 8 +- .../controllers/groups.controller.test.ts | 10 +- .../learning-paths-controller.test.ts | 6 +- .../controllers/student-controller.test.ts | 14 +-- .../submissions-controller.test.ts | 6 +- .../controllers/teacher-controller.test.ts | 4 +- frontend/tests/utils/assingment-rules.test.ts | 74 ++++++------ frontend/tests/utils/deep-equals.test.ts | 110 +++++++++--------- 9 files changed, 170 insertions(+), 170 deletions(-) diff --git a/frontend/tests/controllers/assignments-controller.test.ts b/frontend/tests/controllers/assignments-controller.test.ts index 4e765462..705b1953 100644 --- a/frontend/tests/controllers/assignments-controller.test.ts +++ b/frontend/tests/controllers/assignments-controller.test.ts @@ -1,65 +1,65 @@ -import { describe, it, expect, beforeEach } from 'vitest'; -import { AssignmentController } from '../../src/controllers/assignments'; -import { AssignmentDTO } from '@dwengo-1/common/interfaces/assignment'; +import { describe, it, expect, beforeEach } from "vitest"; +import { AssignmentController } from "../../src/controllers/assignments"; +import { AssignmentDTO } from "@dwengo-1/common/interfaces/assignment"; -describe('AssignmentController Tests', () => { - let controller: AssignmentController; +describe("AssignmentController Tests", () => { + let controller: AssignmentController; - beforeEach(() => { - controller = new AssignmentController('8764b861-90a6-42e5-9732-c0d9eb2f55f9'); // Example class ID - }); + beforeEach(() => { + controller = new AssignmentController("8764b861-90a6-42e5-9732-c0d9eb2f55f9"); // Example class ID + }); - it('should fetch all assignments', async () => { - const result = await controller.getAll(true); - expect(result).toHaveProperty('assignments'); - expect(Array.isArray(result.assignments)).toBe(true); - expect(result.assignments.length).toBeGreaterThan(0); - }); + it("should fetch all assignments", async () => { + const result = await controller.getAll(true); + expect(result).toHaveProperty("assignments"); + expect(Array.isArray(result.assignments)).toBe(true); + expect(result.assignments.length).toBeGreaterThan(0); + }); - it('should fetch an assignment by number', async () => { - const assignmentNumber = 21000; // Example assignment ID - const result = await controller.getByNumber(assignmentNumber); - expect(result).toHaveProperty('assignment'); - expect(result.assignment).toHaveProperty('id', assignmentNumber); - }); + it("should fetch an assignment by number", async () => { + const assignmentNumber = 21000; // Example assignment ID + const result = await controller.getByNumber(assignmentNumber); + expect(result).toHaveProperty("assignment"); + expect(result.assignment).toHaveProperty("id", assignmentNumber); + }); - it('should update an existing assignment', async () => { - const assignmentNumber = 21000; - const updatedData = { title: 'Updated Assignment Title' }; - const result = await controller.updateAssignment(assignmentNumber, updatedData); - expect(result).toHaveProperty('assignment'); - expect(result.assignment).toHaveProperty('id', assignmentNumber); - expect(result.assignment).toHaveProperty('title', updatedData.title); - }); + it("should update an existing assignment", async () => { + const assignmentNumber = 21000; + const updatedData = { title: "Updated Assignment Title" }; + const result = await controller.updateAssignment(assignmentNumber, updatedData); + expect(result).toHaveProperty("assignment"); + expect(result.assignment).toHaveProperty("id", assignmentNumber); + expect(result.assignment).toHaveProperty("title", updatedData.title); + }); - it('should fetch submissions for an assignment', async () => { - const assignmentNumber = 21000; - const result = await controller.getSubmissions(assignmentNumber, true); - expect(result).toHaveProperty('submissions'); - expect(Array.isArray(result.submissions)).toBe(true); - }); + it("should fetch submissions for an assignment", async () => { + const assignmentNumber = 21000; + const result = await controller.getSubmissions(assignmentNumber, true); + expect(result).toHaveProperty("submissions"); + expect(Array.isArray(result.submissions)).toBe(true); + }); - it('should fetch questions for an assignment', async () => { - const assignmentNumber = 21000; - const result = await controller.getQuestions(assignmentNumber, true); - expect(result).toHaveProperty('questions'); - expect(Array.isArray(result.questions)).toBe(true); - }); + it("should fetch questions for an assignment", async () => { + const assignmentNumber = 21000; + const result = await controller.getQuestions(assignmentNumber, true); + expect(result).toHaveProperty("questions"); + expect(Array.isArray(result.questions)).toBe(true); + }); - it('should fetch groups for an assignment', async () => { - const assignmentNumber = 21000; - const result = await controller.getGroups(assignmentNumber, true); - expect(result).toHaveProperty('groups'); - expect(Array.isArray(result.groups)).toBe(true); - }); + it("should fetch groups for an assignment", async () => { + const assignmentNumber = 21000; + const result = await controller.getGroups(assignmentNumber, true); + expect(result).toHaveProperty("groups"); + expect(Array.isArray(result.groups)).toBe(true); + }); - it('should handle fetching a non-existent assignment', async () => { - const assignmentNumber = 99999; // Non-existent assignment ID - await expect(controller.getByNumber(assignmentNumber)).rejects.toThrow(); - }); + it("should handle fetching a non-existent assignment", async () => { + const assignmentNumber = 99999; // Non-existent assignment ID + await expect(controller.getByNumber(assignmentNumber)).rejects.toThrow(); + }); - it('should handle deleting a non-existent assignment', async () => { - const assignmentNumber = 99999; // Non-existent assignment ID - await expect(controller.deleteAssignment(assignmentNumber)).rejects.toThrow(); - }); + it("should handle deleting a non-existent assignment", async () => { + const assignmentNumber = 99999; // Non-existent assignment ID + await expect(controller.deleteAssignment(assignmentNumber)).rejects.toThrow(); + }); }); diff --git a/frontend/tests/controllers/classes-controller.test.ts b/frontend/tests/controllers/classes-controller.test.ts index e8c07b12..8768aee8 100644 --- a/frontend/tests/controllers/classes-controller.test.ts +++ b/frontend/tests/controllers/classes-controller.test.ts @@ -1,8 +1,8 @@ -import { describe, expect, it } from 'vitest'; -import { ClassController } from '../../src/controllers/classes'; +import { describe, expect, it } from "vitest"; +import { ClassController } from "../../src/controllers/classes"; -describe('Test controller classes', () => { - it('Get classes', async () => { +describe("Test controller classes", () => { + it("Get classes", async () => { const controller = new ClassController(); const data = await controller.getAll(true); expect(data.classes).to.have.length.greaterThan(0); diff --git a/frontend/tests/controllers/groups.controller.test.ts b/frontend/tests/controllers/groups.controller.test.ts index 30d90c09..cb7b9d75 100644 --- a/frontend/tests/controllers/groups.controller.test.ts +++ b/frontend/tests/controllers/groups.controller.test.ts @@ -1,9 +1,9 @@ -import { describe, expect, it } from 'vitest'; -import { GroupController } from '../../src/controllers/groups'; +import { describe, expect, it } from "vitest"; +import { GroupController } from "../../src/controllers/groups"; -describe('Test controller groups', () => { - it('Get groups', async () => { - const classId = '8764b861-90a6-42e5-9732-c0d9eb2f55f9'; +describe("Test controller groups", () => { + it("Get groups", async () => { + const classId = "8764b861-90a6-42e5-9732-c0d9eb2f55f9"; const assignmentNumber = 21000; const controller = new GroupController(classId, assignmentNumber); diff --git a/frontend/tests/controllers/learning-paths-controller.test.ts b/frontend/tests/controllers/learning-paths-controller.test.ts index d742d5ef..28e4cda2 100644 --- a/frontend/tests/controllers/learning-paths-controller.test.ts +++ b/frontend/tests/controllers/learning-paths-controller.test.ts @@ -1,6 +1,6 @@ -import { beforeEach, describe, expect, it } from 'vitest'; -import { LearningPathController } from '../../src/controllers/learning-paths'; -import { Language } from '../../src/data-objects/language'; +import { beforeEach, describe, expect, it } from "vitest"; +import { LearningPathController } from "../../src/controllers/learning-paths"; +import { Language } from "../../src/data-objects/language"; describe("Test controller learning paths", () => { let controller: LearningPathController; diff --git a/frontend/tests/controllers/student-controller.test.ts b/frontend/tests/controllers/student-controller.test.ts index 09688720..1a4905a6 100644 --- a/frontend/tests/controllers/student-controller.test.ts +++ b/frontend/tests/controllers/student-controller.test.ts @@ -1,25 +1,25 @@ -import { StudentController } from '../../src/controllers/students'; -import { beforeEach, describe, expect, it } from 'vitest'; +import { StudentController } from "../../src/controllers/students"; +import { beforeEach, describe, expect, it } from "vitest"; -describe('Test controller students', () => { +describe("Test controller students", () => { let controller: StudentController; beforeEach(async () => { controller = new StudentController(); }); - it('Get students', async () => { + it("Get students", async () => { const data = await controller.getAll(true); expect(data.students).to.have.length.greaterThan(0); }); - it('Get student by username', async () => { - const username = 'testleerling1'; + it("Get student by username", async () => { + const username = "testleerling1"; const data = await controller.getByUsername(username); expect(data.student.username).to.equal(username); }); - it('Get classes of student', async () => { + it("Get classes of student", async () => { const students = await controller.getAll(true); for (const student of students.students) { diff --git a/frontend/tests/controllers/submissions-controller.test.ts b/frontend/tests/controllers/submissions-controller.test.ts index ca2799df..450ddd7a 100644 --- a/frontend/tests/controllers/submissions-controller.test.ts +++ b/frontend/tests/controllers/submissions-controller.test.ts @@ -1,6 +1,6 @@ -import { describe, expect, it } from 'vitest'; -import { SubmissionController } from '../../src/controllers/submissions'; -import { Language } from '../../src/data-objects/language'; +import { describe, expect, it } from "vitest"; +import { SubmissionController } from "../../src/controllers/submissions"; +import { Language } from "../../src/data-objects/language"; describe("Test controller submissions", () => { it("Get submission by number", async () => { diff --git a/frontend/tests/controllers/teacher-controller.test.ts b/frontend/tests/controllers/teacher-controller.test.ts index b0d7ba98..9d634225 100644 --- a/frontend/tests/controllers/teacher-controller.test.ts +++ b/frontend/tests/controllers/teacher-controller.test.ts @@ -1,5 +1,5 @@ -import { beforeEach, describe, expect, it } from 'vitest'; -import { TeacherController } from '../../src/controllers/teachers'; +import { beforeEach, describe, expect, it } from "vitest"; +import { TeacherController } from "../../src/controllers/teachers"; describe("Test controller teachers", () => { let controller: TeacherController; diff --git a/frontend/tests/utils/assingment-rules.test.ts b/frontend/tests/utils/assingment-rules.test.ts index c070aa0a..3b6e5bfd 100644 --- a/frontend/tests/utils/assingment-rules.test.ts +++ b/frontend/tests/utils/assingment-rules.test.ts @@ -1,82 +1,82 @@ -import { describe, expect, it } from 'vitest'; +import { describe, expect, it } from "vitest"; import { assignmentTitleRules, classRules, deadlineRules, descriptionRules, learningPathRules, -} from '../../src/utils/assignment-rules'; +} from "../../src/utils/assignment-rules"; -describe('Validation Rules', () => { - describe('assignmentTitleRules', () => { - it('should return true for a valid title', () => { - const result = assignmentTitleRules[0]('Valid Title'); +describe("Validation Rules", () => { + describe("assignmentTitleRules", () => { + it("should return true for a valid title", () => { + const result = assignmentTitleRules[0]("Valid Title"); expect(result).toBe(true); }); - it('should return an error message for an empty title', () => { - const result = assignmentTitleRules[0](''); - expect(result).toBe('Title cannot be empty.'); + it("should return an error message for an empty title", () => { + const result = assignmentTitleRules[0](""); + expect(result).toBe("Title cannot be empty."); }); }); - describe('learningPathRules', () => { - it('should return true for a valid learning path', () => { - const result = learningPathRules[0]({ hruid: '123', title: 'Path Title' }); + describe("learningPathRules", () => { + it("should return true for a valid learning path", () => { + const result = learningPathRules[0]({ hruid: "123", title: "Path Title" }); expect(result).toBe(true); }); - it('should return an error message for an invalid learning path', () => { - const result = learningPathRules[0]({ hruid: '', title: '' }); - expect(result).toBe('You must select a learning path.'); + it("should return an error message for an invalid learning path", () => { + const result = learningPathRules[0]({ hruid: "", title: "" }); + expect(result).toBe("You must select a learning path."); }); }); - describe('classRules', () => { - it('should return true for a valid class', () => { - const result = classRules[0]('Class 1'); + describe("classRules", () => { + it("should return true for a valid class", () => { + const result = classRules[0]("Class 1"); expect(result).toBe(true); }); - it('should return an error message for an empty class', () => { - const result = classRules[0](''); - expect(result).toBe('You must select at least one class.'); + it("should return an error message for an empty class", () => { + const result = classRules[0](""); + expect(result).toBe("You must select at least one class."); }); }); - describe('deadlineRules', () => { - it('should return true for a valid future deadline', () => { + describe("deadlineRules", () => { + it("should return true for a valid future deadline", () => { const futureDate = new Date(Date.now() + 1000 * 60 * 60).toISOString(); const result = deadlineRules[0](futureDate); expect(result).toBe(true); }); - it('should return an error message for a past deadline', () => { + it("should return an error message for a past deadline", () => { const pastDate = new Date(Date.now() - 1000 * 60 * 60).toISOString(); const result = deadlineRules[0](pastDate); - expect(result).toBe('The deadline must be in the future.'); + expect(result).toBe("The deadline must be in the future."); }); - it('should return an error message for an invalid date', () => { - const result = deadlineRules[0]('invalid-date'); - expect(result).toBe('Invalid date or time.'); + it("should return an error message for an invalid date", () => { + const result = deadlineRules[0]("invalid-date"); + expect(result).toBe("Invalid date or time."); }); - it('should return an error message for an empty deadline', () => { - const result = deadlineRules[0](''); - expect(result).toBe('You must set a deadline.'); + it("should return an error message for an empty deadline", () => { + const result = deadlineRules[0](""); + expect(result).toBe("You must set a deadline."); }); }); - describe('descriptionRules', () => { - it('should return true for a valid description', () => { - const result = descriptionRules[0]('This is a valid description.'); + describe("descriptionRules", () => { + it("should return true for a valid description", () => { + const result = descriptionRules[0]("This is a valid description."); expect(result).toBe(true); }); - it('should return an error message for an empty description', () => { - const result = descriptionRules[0](''); - expect(result).toBe('Description cannot be empty.'); + it("should return an error message for an empty description", () => { + const result = descriptionRules[0](""); + expect(result).toBe("Description cannot be empty."); }); }); }); diff --git a/frontend/tests/utils/deep-equals.test.ts b/frontend/tests/utils/deep-equals.test.ts index fcad2869..8142b621 100644 --- a/frontend/tests/utils/deep-equals.test.ts +++ b/frontend/tests/utils/deep-equals.test.ts @@ -1,68 +1,68 @@ -import { describe, it, expect } from 'vitest'; -import { deepEquals } from '../../src/utils/deep-equals'; +import { describe, it, expect } from "vitest"; +import { deepEquals } from "../../src/utils/deep-equals"; -describe('deepEquals', () => { - it('should return true for identical primitive values', () => { - expect(deepEquals(1, 1)).toBe(true); - expect(deepEquals('test', 'test')).toBe(true); - expect(deepEquals(true, true)).toBe(true); - }); +describe("deepEquals", () => { + it("should return true for identical primitive values", () => { + expect(deepEquals(1, 1)).toBe(true); + expect(deepEquals("test", "test")).toBe(true); + expect(deepEquals(true, true)).toBe(true); + }); - it('should return false for different primitive values', () => { - expect(deepEquals(1, 2)).toBe(false); - expect(deepEquals('test', 'other')).toBe(false); - expect(deepEquals(true, false)).toBe(false); - }); + it("should return false for different primitive values", () => { + expect(deepEquals(1, 2)).toBe(false); + expect(deepEquals("test", "other")).toBe(false); + expect(deepEquals(true, false)).toBe(false); + }); - it('should return true for identical objects', () => { - const obj1 = { a: 1, b: { c: 2 } }; - const obj2 = { a: 1, b: { c: 2 } }; - expect(deepEquals(obj1, obj2)).toBe(true); - }); + it("should return true for identical objects", () => { + const obj1 = { a: 1, b: { c: 2 } }; + const obj2 = { a: 1, b: { c: 2 } }; + expect(deepEquals(obj1, obj2)).toBe(true); + }); - it('should return false for different objects', () => { - const obj1 = { a: 1, b: { c: 2 } }; - const obj2 = { a: 1, b: { c: 3 } }; - expect(deepEquals(obj1, obj2)).toBe(false); - }); + it("should return false for different objects", () => { + const obj1 = { a: 1, b: { c: 2 } }; + const obj2 = { a: 1, b: { c: 3 } }; + expect(deepEquals(obj1, obj2)).toBe(false); + }); - it('should return true for identical arrays', () => { - const arr1 = [1, 2, [3, 4]]; - const arr2 = [1, 2, [3, 4]]; - expect(deepEquals(arr1, arr2)).toBe(true); - }); + it("should return true for identical arrays", () => { + const arr1 = [1, 2, [3, 4]]; + const arr2 = [1, 2, [3, 4]]; + expect(deepEquals(arr1, arr2)).toBe(true); + }); - it('should return false for different arrays', () => { - const arr1 = [1, 2, [3, 4]]; - const arr2 = [1, 2, [3, 5]]; - expect(deepEquals(arr1, arr2)).toBe(false); - }); + it("should return false for different arrays", () => { + const arr1 = [1, 2, [3, 4]]; + const arr2 = [1, 2, [3, 5]]; + expect(deepEquals(arr1, arr2)).toBe(false); + }); - it('should return false for objects and arrays compared', () => { - expect(deepEquals({ a: 1 }, [1])).toBe(false); - }); + it("should return false for objects and arrays compared", () => { + expect(deepEquals({ a: 1 }, [1])).toBe(false); + }); - it('should return true for null compared to null', () => { - expect(deepEquals(null, null)).toBe(true); - }); + it("should return true for null compared to null", () => { + expect(deepEquals(null, null)).toBe(true); + }); - it('should return false for null compared to an object', () => { - expect(deepEquals(null, {})).toBe(false); - }); + it("should return false for null compared to an object", () => { + expect(deepEquals(null, {})).toBe(false); + }); - it('should return false for undefined compared to null', () => { - expect(deepEquals(undefined, null)).toBe(false); - }); + it("should return false for undefined compared to null", () => { + expect(deepEquals(undefined, null)).toBe(false); + }); - it('should return true for deeply nested identical structures', () => { - const obj1 = { a: [1, { b: 2, c: [3, 4] }] }; - const obj2 = { a: [1, { b: 2, c: [3, 4] }] }; - expect(deepEquals(obj1, obj2)).toBe(true); - }); + it("should return true for deeply nested identical structures", () => { + const obj1 = { a: [1, { b: 2, c: [3, 4] }] }; + const obj2 = { a: [1, { b: 2, c: [3, 4] }] }; + expect(deepEquals(obj1, obj2)).toBe(true); + }); - it('should return false for deeply nested different structures', () => { - const obj1 = { a: [1, { b: 2, c: [3, 4] }] }; - const obj2 = { a: [1, { b: 2, c: [3, 5] }] }; - expect(deepEquals(obj1, obj2)).toBe(false); - }); + it("should return false for deeply nested different structures", () => { + const obj1 = { a: [1, { b: 2, c: [3, 4] }] }; + const obj2 = { a: [1, { b: 2, c: [3, 5] }] }; + expect(deepEquals(obj1, obj2)).toBe(false); + }); });