Skip to content

Commit 410ce1d

Browse files
AxelHoweAxelHowe
committed
feat: lab2
1 parent 3c5992a commit 410ce1d

5 files changed

Lines changed: 152 additions & 76 deletions

File tree

‎lab1/main_test.js‎

Lines changed: 5 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -4,91 +4,20 @@ const { MyClass, Student } = require('./main');
44

55
test("Test MyClass's addStudent", () => {
66
// TODO
7-
8-
const myClass = new MyClass();
9-
10-
const student = new Student();
11-
student.setName("Kodak");
12-
const id = myClass.addStudent(student);
13-
assert.strictEqual(id, 0);
14-
15-
// 測試新增第二個學生
16-
const student2 = new Student();
17-
student2.setName("Contax");
18-
const id2 = myClass.addStudent(student2);
19-
assert.strictEqual(id2, 1); // 第二個學生的 ID 應該是 1
20-
21-
// 測試新增非 Student 實例
22-
const invalidStudent = { name: "Invalid" };
23-
const invalidId = myClass.addStudent(invalidStudent);
24-
assert.strictEqual(invalidId, -1); // 應該返回 -1
25-
26-
//throw new Error("Test not implemented");
7+
throw new Error("Test not implemented");
278
});
289

2910
test("Test MyClass's getStudentById", () => {
3011
// TODO
31-
32-
const myClass = new MyClass();
33-
const student = new Student();
34-
student.setName("Kodak");
35-
myClass.addStudent(student);
36-
37-
// 測試有效的 ID
38-
const foundStudent = myClass.getStudentById(0);
39-
assert.strictEqual(foundStudent, student);
40-
41-
// 測試負數 ID
42-
const nullStudent1 = myClass.getStudentById(-1);
43-
assert.strictEqual(nullStudent1, null);
44-
45-
// 測試超出範圍的 ID
46-
const nullStudent2 = myClass.getStudentById(999);
47-
assert.strictEqual(nullStudent2, null);
48-
//throw new Error("Test not implemented");
12+
throw new Error("Test not implemented");
4913
});
5014

5115
test("Test Student's setName", () => {
5216
// TODO
53-
const student = new Student();
54-
55-
// 測試設定有效的名字
56-
student.setName("Kodak");
57-
assert.strictEqual(student.getName(), "Kodak");
58-
59-
// 測試設定空字串
60-
student.setName("");
61-
assert.strictEqual(student.getName(), "");
62-
63-
// 測試設定非字串值
64-
student.setName(123); // 應該被忽略
65-
assert.strictEqual(student.getName(), ""); // 名字應該保持不變
66-
67-
student.setName(null); // 應該被忽略
68-
assert.strictEqual(student.getName(), ""); // 名字應該保持不變
69-
70-
student.setName(undefined); // 應該被忽略
71-
assert.strictEqual(student.getName(), ""); // 名字應該保持不變
72-
//throw new Error("Test not implemented");
17+
throw new Error("Test not implemented");
7318
});
7419

7520
test("Test Student's getName", () => {
7621
// TODO
77-
const student = new Student();
78-
79-
// 測試初始狀態（未設定名字）
80-
assert.strictEqual(student.getName(), "");
81-
82-
// 測試設定名字後的狀態
83-
student.setName("Kodak");
84-
assert.strictEqual(student.getName(), "Kodak");
85-
86-
// 測試設定空字串
87-
student.setName("");
88-
assert.strictEqual(student.getName(), "");
89-
90-
// 直接修改 name 為 undefined
91-
student.name = undefined;
92-
assert.strictEqual(student.getName(), "");
93-
//throw new Error("Test not implemented");
94-
});
22+
throw new Error("Test not implemented");
23+
});

‎lab2/README.md‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# Lab2
2+
3+
## Introduction
4+
5+
In this lab, you will write unit tests for functions implemented in `main.js`. You can learn how to use classes and functions in it by uncommenting the code in it. (But remember don't commit them on GitHub)
6+
7+
## Requirement
8+
9+
1. Write test cases in `main_test.js` and achieve 100% code coverage. Remember to use Mock, Spy, or Stub when necessary, you need to at least use one of them in your test cases. (100%)
10+
11+
You can run `validate.sh` in your local to test if you satisfy the requirements.
12+
13+
Please note that you must not alter files other than `main_test.js`. You will get 0 points if
14+
15+
1. you modify other files to achieve requirements.
16+
2. you can't pass all CI on your PR.
17+
18+
## Submission
19+
20+
You need to open a pull request to your branch (e.g. 311XXXXXX, your student number) and contain the code that satisfies the abovementioned requirements.
21+
22+
Moreover, please submit the URL of your PR to E3. Your submission will only be accepted when you present at both places.

‎lab2/main.js‎

Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
const fs = require('fs');
2+
const util = require('util');
3+
const readFile = util.promisify(fs.readFile);
4+
5+
class MailSystem {
6+
write(name) {
7+
console.log('--write mail for ' + name + '--');
8+
const context = 'Congrats, ' + name + '!';
9+
return context;
10+
}
11+
12+
send(name, context) {
13+
console.log('--send mail to ' + name + '--');
14+
// Interact with mail system and send mail
15+
// random success or failure
16+
const success = Math.random() > 0.5;
17+
if (success) {
18+
console.log('mail sent');
19+
} else {
20+
console.log('mail failed');
21+
}
22+
return success;
23+
}
24+
}
25+
26+
class Application {
27+
constructor() {
28+
this.people = [];
29+
this.selected = [];
30+
this.mailSystem = new MailSystem();
31+
this.getNames().then(([people, selected]) => {
32+
this.people = people;
33+
this.selected = selected;
34+
});
35+
}
36+
37+
async getNames() {
38+
const data = await readFile('name_list.txt', 'utf8');
39+
const people = data.split('\n');
40+
const selected = [];
41+
return [people, selected];
42+
}
43+
44+
getRandomPerson() {
45+
const i = Math.floor(Math.random() * this.people.length);
46+
return this.people[i];
47+
}
48+
49+
selectNextPerson() {
50+
console.log('--select next person--');
51+
if (this.people.length === this.selected.length) {
52+
console.log('all selected');
53+
return null;
54+
}
55+
let person = this.getRandomPerson();
56+
while (this.selected.includes(person)) {
57+
person = this.getRandomPerson();
58+
}
59+
this.selected.push(person);
60+
return person;
61+
}
62+
63+
notifySelected() {
64+
console.log('--notify selected--');
65+
for (const x of this.selected) {
66+
const context = this.mailSystem.write(x);
67+
this.mailSystem.send(x, context);
68+
}
69+
}
70+
}
71+
72+
// const app = new Application();
73+
// app.selectNextPerson();
74+
// app.selectNextPerson();
75+
// app.selectNextPerson();
76+
// app.notifySelected();
77+
78+
module.exports = {
79+
Application,
80+
MailSystem,
81+
};

‎lab2/main_test.js‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
const test = require('node:test');
2+
const assert = require('assert');
3+
const { Application, MailSystem } = require('./main');
4+
5+
// TODO: write your tests here
6+
// Remember to use Stub, Mock, and Spy when necessary

‎lab2/validate.sh‎

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
#!/bin/bash
2+
3+
# Check for unwanted files
4+
for file in *; do
5+
if [[ $file != "main.js" && $file != "main_test.js" && $file != "README.md" && $file != "validate.sh" ]]; then
6+
echo "[!] Unwanted file detected: $file."
7+
exit 1
8+
fi
9+
done
10+
11+
node=$(which node)
12+
test_path="${BASH_SOURCE[0]}"
13+
solution_path="$(realpath .)"
14+
tmp_dir=$(mktemp -d -t lab2-XXXXXXXXXX)
15+
16+
cd $tmp_dir
17+
18+
rm -rf *
19+
cp $solution_path/*.js .
20+
result=$($"node" --test --experimental-test-coverage) ; ret=$?
21+
if [ $ret -ne 0 ] ; then
22+
echo "[!] testing fails"
23+
exit 1
24+
else
25+
coverage=$(echo "$result" | grep 'all files' | awk -F '|' '{print $2}' | sed 's/ //g')
26+
if (( $(echo "$coverage < 100" | bc -l) )); then
27+
echo "[!] Coverage is only $coverage%"
28+
exit 1
29+
else
30+
echo "[V] Coverage is 100%"
31+
fi
32+
fi
33+
34+
rm -rf $tmp_dir
35+
36+
exit 0
37+
38+
# vim: set fenc=utf8 ff=unix et sw=2 ts=2 sts=2:

0 commit comments

Comments
 (0)