Skip to content

Commit 77c5e6d

Browse files
lex111atomAltera
lex111
and
atomAltera
authored
Apply suggestions from code review
 Предложеные правки Co-Authored-By: atomAltera <atomAltera@users.noreply.github.com>
1 parent f720666 commit 77c5e6d

15 files changed

Lines changed: 39 additions & 43 deletions

File tree

‎1-js/03-code-quality/05-testing-mocha/3-pow-test-wrong/solution.md‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
Тест демонтрирует один из соблазнов с которым сталкиваюися разработчки при их написании.
1+
Тест демонстрирует один из соблазнов, с которым сталкиваются разработчики при их написании.
22

33
У нас тут по сути три теста, но они расположены в одной функции с тремя проверками.
44

5-
Иногда так проще писать, но если произойдет ошибка, то понять что же пошло не так будет сложнее.
5+
Иногда так проще писать, но если произойдет ошибка, то гораздо сложнее понять, что пошло не так.
66

7-
Если ошибка происходит в cложном процессе вычисления, нам придется выяснять какие были данные в этом месте и для этого придется *дебажить тест*.
7+
Если ошибка происходит в cложном процессе вычисления, нам придется выяснять, какие данные были в этом месте и для этого придется *отлаживать тест*.
88

9-
Гораздо лучше положить каждую проверку в свой отдельный блок `it` и ясно описать входные и ожидаемые данные.
9+
Гораздо лучше разбить тест на несколько блоков `it` и ясно описать входные и ожидаемые данные.
1010

1111
Примерно так:
1212
```js
@@ -25,7 +25,7 @@ describe("Возводит x в степень n", function() {
2525
});
2626
```
2727

28-
Мы заменили один `it` на `describe` и группу `it` блоков. Теперь, если что-то пойдет не так, мы точно увидим какие из условий и с какими данными не прошли проверку.
28+
Мы заменили один `it` на `describe` и группу блоков `it`. Теперь, если какой-либо из блоков завершится неудачно, мы точно увидим, с какими данными это произошло.
2929

3030
Также мы можем изолировать один тест и запускать только его написав `it.only` вместо `it`:
3131

‎1-js/03-code-quality/05-testing-mocha/3-pow-test-wrong/task.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
1-
важность: 5
1+
importance: 5
22

33
---
44

55
# Что не так с этим тестом?
66

7-
Что не так с тестом `pow` ниже?
7+
В чем заключается проблема нижеприведенного теста функции `pow`?
88

99
```js
1010
it("Возводит x в степень n", function() {
@@ -21,4 +21,4 @@ it("Возводит x в степень n", function() {
2121
});
2222
```
2323

24-
P.S. Тест не содержит синтаксических ошибок и не падает при выполнении.
24+
P.S. Тест не содержит синтаксических ошибок и успешно проходит.

‎1-js/03-code-quality/05-testing-mocha/beforeafter.view/index.html‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
<!DOCTYPE html>
22
<html>
33
<head>
4-
<!-- добавим стили mocha, для отрбражения результатов -->
4+
<!-- добавим стили mocha для отображения результатов -->
55
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.css">
6-
<!-- сам фреймворк mocha -->
6+
<!-- добавляем сам фреймворк mocha -->
77
<script src="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.js"></script>
88
<script>
9-
// режим bdd тестирования
9+
// включаем режим тестирования в стиле BDD
1010
mocha.setup('bdd');
1111
</script>
1212
<!-- добавим chai -->

‎1-js/03-code-quality/05-testing-mocha/beforeafter.view/test.js‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@ describe("тест", function () {
33
before(() => alert("Тестирование началось – перед тестами"));
44
after(() => alert("Тестирование закончилось – после всех тестов"));
55

6-
beforeEach(() => alert("Перед тестом – начинаем выпонять тест"));
7-
afterEach(() => alert("После теста – заначиваем выполнение теста"));
6+
beforeEach(() => alert("Перед тестом – начинаем выполнять тест"));
7+
afterEach(() => alert("После теста – заканчиваем выполнение теста"));
88

99
it('тест 1', () => alert(1));
1010
it('тест 2', () => alert(2));
1111

12-
});
12+
});

‎1-js/03-code-quality/05-testing-mocha/index.html‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
<!DOCTYPE html>
22
<html>
33
<head>
4-
<!-- добавим стили mocha, для отрбражения результатов -->
4+
<!-- добавим стили mocha для отображения результатов -->
55
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.css">
66
<!-- сам фреймворк mocha -->
77
<script src="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.js"></script>
88
<script>
9-
// режим bdd тестирования
9+
// включаем режим тестирования в стиле BDD
1010
mocha.setup('bdd');
1111
</script>
1212
<!-- добавим chai -->
@@ -21,7 +21,7 @@
2121

2222
<script>
2323
function pow(x, n) {
24-
/* Сюда будем писать реализацию, пока реализации нет */
24+
/* Здесь будет реализация функции, пока пусто */
2525
}
2626
</script>
2727

‎1-js/03-code-quality/05-testing-mocha/pow-1.view/index.html‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
<!DOCTYPE html>
22
<html>
33
<head>
4-
<!-- добавим стили mocha, для отрбражения результатов -->
4+
<!-- добавим стили mocha для отображения результатов -->
55
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.css">
6-
<!-- сам фреймворк mocha -->
6+
<!-- добавляем сам фреймворк mocha -->
77
<script src="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.js"></script>
88
<script>
9-
// режим bdd тестирования
9+
// включаем режим тестирования в стиле BDD
1010
mocha.setup('bdd');
1111
</script>
1212
<!-- добавим chai -->

‎1-js/03-code-quality/05-testing-mocha/pow-2.view/index.html‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
<!DOCTYPE html>
22
<html>
33
<head>
4-
<!-- добавим стили mocha, для отрбражения результатов -->
4+
<!-- добавим стили mocha для отображения результатов -->
55
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.css">
66
<!-- сам фреймворк mocha -->
77
<script src="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.js"></script>
88
<script>
9-
// режим bdd тестирования
9+
// включаем режим тестирования в стиле BDD
1010
mocha.setup('bdd');
1111
</script>
1212
<!-- добавим chai -->

‎1-js/03-code-quality/05-testing-mocha/pow-3.view/index.html‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
<!DOCTYPE html>
22
<html>
33
<head>
4-
<!-- добавим стили mocha, для отрбражения результатов -->
4+
<!-- добавим стили mocha для отображения результатов -->
55
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.css">
6-
<!-- сам фреймворк mocha -->
6+
<!-- добавляем сам фреймворк mocha -->
77
<script src="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.js"></script>
88
<script>
9-
// режим bdd тестирования
9+
// включаем режим тестирования в стиле BDD
1010
mocha.setup('bdd');
1111
</script>
1212
<!-- добавим chai -->

‎1-js/03-code-quality/05-testing-mocha/pow-4.view/index.html‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<!-- сам фреймворк mocha -->
77
<script src="https://cdnjs.cloudflare.com/ajax/libs/mocha/3.2.0/mocha.js"></script>
88
<script>
9-
// режим bdd тестирования
9+
// включаем режим тестирования в стиле BDD
1010
mocha.setup('bdd');
1111
</script>
1212
<!-- добавим chai -->

‎1-js/03-code-quality/05-testing-mocha/pow-4.view/test.js‎

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,15 @@
11
describe("pow", function() {
22

33
describe("возводит x в степень 3", function() {
4-
54
function makeTest(x) {
65
let expected = x * x * x;
76
it(`${x} в степени 3 будет ${expected}`, function () {
87
assert.equal(pow(x, 3), expected);
98
});
109
}
11-
1210
for (let x = 1; x <= 5; x++) {
1311
makeTest(x);
1412
}
15-
1613
});
17-
1814
// ... другие тесты. Можно писать и describe и it блоки.
19-
});
15+
});

0 commit comments

Comments
 (0)