Skip to content

[react-testing-01] Предложение по листингу  #1

@severgun

Description

@severgun

react-101/README.md

Lines 156 to 169 in 3ce15d0

```js
// sum.test.js
import { describe, expect, test } from 'vitest';
import { sum } from './sum.js';
describe('sum', () => {
test('should correctly sum two numbers', () => {
const expected = 3;
const result = sum(1, 2);
expect(result).toBe(expected);
});
});
```

Возможно для более наглядного понимания что именно объединяет describe стоит добавить еще 1-2 теста и дать более понятное описание блока? Плюс возможно добавить пример вывода консоли

 // sum.test.js 
 import { describe, expect, test } from 'vitest'; 
  
 import { sum } from './sum.js'; 
  
 describe('Test sum function', () => { 
   test('Should correctly sum two positive numbers', () => { 
     const expected = 3; 
     const result = sum(1, 2); 
     expect(result).toBe(expected); 
   });

   test('Should correctly sum two negative numbers', () => { 
     const expected = -9; 
     const result = sum(-4, -5); 
     expect(result).toBe(expected); 
   });
 }); 
 ✓ src/sum.test.js (2)
   ✓ Test sum function (2)
     ✓ Should correctly sum two positive numbers
     ✓ Should correctly sum two negative numbers

Тут видимо пропущено слово иметь после должен. И в целом какая то странная формулировка.
Несколько фаз в одном тесте... Три это тоже несколько. Возможно стоит оставить только Наличие более трех фаз описанных как AAA. Или указать что ошибкой является несколько фаз одного типа.

react-101/README.md

Lines 254 to 255 in 3ce15d0

- **Несколько фаз в одном тесте, более чем три описанных как AAA**. Один тест должен три фазы, то есть по одной фазе из
AAA.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions