Skip to content

Commit c5d1744

Browse files
CTHuaCTHua
committed
feat: lab3
1 parent c52bbaa commit c5d1744

File tree

5 files changed

+109
-78
lines changed

5 files changed

+109
-78
lines changed

‎lab2/main_test.js

Lines changed: 3 additions & 78 deletions
Original file line numberDiff line numberDiff line change
@@ -1,81 +1,6 @@
1-
const {describe, it, mock} = require('node:test');
1+
const test = require('node:test');
22
const assert = require('assert');
3-
const fs = require('fs');
4-
fs.readFile = (fileName, options, callback) => {process.nextTick(() => callback(null, 'Alice\nBob\nCharlie'));};
5-
const { Application, MailSystem} = require('./main');
3+
const { Application, MailSystem } = require('./main');
64

75
// TODO: write your tests here
8-
//test MailSystem
9-
describe('write test',() => {
10-
it('return expected string', () => {
11-
const mailSystem = new MailSystem();
12-
assert.strictEqual(mailSystem.write('Bob'),'Congrats, Bob!');
13-
});
14-
});
15-
describe('send test',() => {
16-
it('success send', () => {
17-
const mailSystem = new MailSystem();
18-
const mockSuccess = mock.method(Math, 'random', () => 1.0);
19-
const result = mailSystem.send('Bob','Success')
20-
assert.strictEqual(result,true);
21-
mockSuccess.mock.restore();
22-
});
23-
it('fail send', () => {
24-
const mailSystem = new MailSystem();
25-
const mockFail = mock.method(Math, 'random', () => 0);
26-
const result = mailSystem.send('Bob','Fail')
27-
assert.strictEqual(result,false);
28-
mockFail.mock.restore();
29-
});
30-
});
31-
//test Application
32-
describe('getNames test', () => {
33-
it('people & selected', async () => {
34-
const app = new Application();
35-
await app.getNames();
36-
assert.deepStrictEqual(app.people, ['Alice', 'Bob', 'Charlie']);
37-
assert.deepStrictEqual(app.selected, []);
38-
});
39-
});
40-
describe('selectNextPerson test', () => {
41-
it('remain one person',async () => {
42-
const app = new Application();
43-
await app.getNames();
44-
app.selected = ['Alice', 'Bob'];
45-
assert.strictEqual(app.selectNextPerson(),'Charlie');
46-
});
47-
it('if all person selected', async () => {
48-
const app = new Application();
49-
await app.getNames();
50-
app.selected = ['Alice','Bob','Charile'];
51-
assert.strictEqual(app.selectNextPerson(),null);
52-
});
53-
});
54-
describe('notifySelected test', () => {
55-
it('no selected', () => {
56-
const mailSystem = new MailSystem;
57-
const app = new Application();
58-
const mockWrite = mock.method(mailSystem, 'write', () => "Mocked Message");
59-
const mockSend = mock.method(mailSystem, 'send', () => "Mocked Send");
60-
app.mailSystem = mailSystem;
61-
app.notifySelected();
62-
assert.strictEqual(mockWrite.mock.calls.length,0);
63-
assert.strictEqual(mockSend.mock.calls.length,0);
64-
});
65-
it('selected all',async () => {
66-
const mailSystem = new MailSystem;
67-
const app = new Application();
68-
const mockWrite = mock.method(mailSystem, 'write', () => "Mocked Message");
69-
const mockSend = mock.method(mailSystem, 'send', () => "Mocked Send");
70-
app.mailSystem = mailSystem;
71-
await app.getNames();
72-
app.selected = ['Alice','Bob','Charile'];
73-
app.notifySelected();
74-
assert.strictEqual(mockWrite.mock.calls.length,3);
75-
assert.strictEqual(mockSend.mock.calls.length,3);
76-
mockWrite.mock.restore();
77-
mockSend.mock.restore();
78-
});
79-
});
80-
// Remember to use Stub, Mock, and Spy when necessary
81-
6+
// Remember to use Stub, Mock, and Spy when necessary

‎lab3/README.md

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# Lab3
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+
## Preparation (Important!!!)
8+
9+
1. Sync fork on GitHub
10+
2. `git checkout -b lab3` (**NOT** your student ID !!!)
11+
12+
## Requirement
13+
14+
1. (40%) Write test cases in `main_test.js` and achieve 100% code coverage.
15+
2. (30%) For each function, parameterize their testcases to test the error-results.
16+
3. (30%) For each function, use at least 3 parameterized testcases to test the non-error-results.
17+
18+
You can run `validate.sh` in your local to test if you satisfy the requirements.
19+
20+
Please note that you must not alter files other than `main_test.js`. You will get 0 points if
21+
22+
1. you modify other files to achieve requirements.
23+
2. you can't pass all CI on your PR.
24+
25+
## Submission
26+
27+
You need to open a pull request to your branch (e.g. 312XXXXXX, your student number) and contain the code that satisfies the abovementioned requirements.
28+
29+
Moreover, please submit the URL of your PR to E3. Your submission will only be accepted when you present at both places.

‎lab3/main.js

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
class Calculator {
2+
exp(x) {
3+
if (!Number.isFinite(x)) {
4+
throw Error('unsupported operand type');
5+
}
6+
const result = Math.exp(x);
7+
if (result === Infinity) {
8+
throw Error('overflow');
9+
}
10+
return result;
11+
}
12+
13+
log(x) {
14+
if (!Number.isFinite(x)) {
15+
throw Error('unsupported operand type');
16+
}
17+
const result = Math.log(x);
18+
if (result === -Infinity) {
19+
throw Error('math domain error (1)');
20+
}
21+
if (Number.isNaN(result)) {
22+
throw Error('math domain error (2)');
23+
}
24+
return result;
25+
}
26+
}
27+
28+
// const calculator = new Calculator();
29+
// console.log(calculator.exp(87));
30+
// console.log(calculator.log(48763));
31+
32+
module.exports = {
33+
Calculator
34+
};

‎lab3/main_test.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
const {describe, it} = require('node:test');
2+
const assert = require('assert');
3+
const { Calculator } = require('./main');
4+
5+
// TODO: write your tests here

‎lab3/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 lab3-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)