[SangCT] Do homework #26

sangct-1630 · 2021-05-19T19:14:30Z

Purpose/Notes

Write UnitTest Module 3->7

Screenshot

Module 03
Module 04
Module 05
Module 06
Module 07

Checklist (*)

Pull request has been self-reviewed
Pull request has a meaningful description of its purpose
All commits are accompanied by meaningful commit messages
All CI builds passed successfully (all builds are green)

tuanpt-0634 · 2021-05-25T16:05:22Z

Modules/Exercise03/Tests/Feature/Http/Controllers/ProductControllerTest.php

+    /**
+     * A basic unit test example.
+     *
+     * @return void
+     */
+    public function test__contruct()
+    {
+        $controller = new ProductController($this->productServiceMock);
+
+        $this->assertInstanceOf(ProductController::class, $controller);
+
+    }


Test constructor có ý nghĩa gì không em?

tuanpt-0634 · 2021-05-25T16:09:17Z

Modules/Exercise03/Tests/Feature/Http/Controllers/ProductControllerTest.php

+
+    }
+
+    public function test_index()


Test index là test cái gì, tên test case nên thể hiện được mục đích của test case

Mục 1 trong checklist:

[1] Self-describing test method](https://sun-unit-test-training.github.io/php-testing-guideline/06-checklist/#1-self-describing-test-method)

tuanpt-0634 · 2021-05-25T16:12:02Z

Modules/Exercise03/Tests/Feature/Http/Controllers/ProductControllerTest.php

+        $request = $this->mock(CheckoutRequest::class);
+        $request->shouldReceive('input')
+            ->once()
+            ->with('total_products')
+            ->andReturn($input);
+
+        $this->productServiceMock->shouldReceive('calculateDiscount')
+            ->with($input)
+            ->once()
+            ->andReturn(7);
+
+        $controller = new ProductController($this->productServiceMock);
+
+        $this->assertInstanceOf(JsonResponse::class, $controller->checkout($request));


Cái này viết theo cách dùng HTTP tests như ở test_index được không? Sao case này lại viết cách khác nhỉ?

tuanpt-0634 · 2021-05-25T16:13:52Z

Modules/Exercise03/Tests/Feature/Http/Requests/CheckoutRequestTest.php

+    public function test_rules()
+    {
+        $request = new CheckoutRequest();
+
+        $this->assertSame([
+            'total_products' => 'required|array',
+            'total_products.*' => 'nullable|integer|min:0',
+        ], $request->rules());
+    }


Viết như này ko sai, nhưng không khuyến khích
https://sun-unit-test-training.github.io/php-testing-guideline/laravel/03-controller/#unit-test

tuanpt-0634 · 2021-05-25T17:43:32Z

Modules/Exercise03/Tests/Unit/Services/ProductServiceTest.php

+    /**
+     * @param $totalProducts
+     * @param $expectedValue
+     * @dataProvider provideValidTotalProductsData
+     * */
+    public function test_calculate_discount_with_valid_data($totalProducts, $expectedValue)
+    {
+        $discount = $this->productService->calculateDiscount($totalProducts);
+        $this->assertEquals($expectedValue, $discount);
+    }
+
+    public function provideValidTotalProductsData()
+    {
+        return [
+            [
+                [
+                    1 => 2,
+                    2 => 2,
+                    3 => 2,
+                ], 5
+            ],
+            [
+                [
+                    1 => 0,
+                    2 => 4,
+                    3 => 4,
+                ], 7
+            ],
+            [
+                [
+                    1 => 2,
+                    2 => 3,
+                    3 => 3,
+                ], 12
+            ],
+        ];
+    }


Viết như này đọc khá khó hiểu, ít nhất nên đặt tên cho từng dataset hoặc là tách ra các case riêng biệt

Input chỉ bao gồm các số, nhìn như các số ngẫu nhiên, không thể hiện được điều kiện input đầu vào là gì => Các key input nó đã được đặt tên bằng const trong class Product thì nên sử dụng ở đây

Với cả thiếu case nhé?

Anh thấy bài này có thể chia case theo kết quả mong muốn thì sẽ rất dễ sinh input, có 4 kết quả mong muốn:

Exception => invalid data

Discount = 5 => only cravat white discount

Discount = 7 => only total quantity discout

Discount = 12 => both discounts

Hoặc có thể bám theo lý thuyết C1

if ($cravat < 0 || $whiteShirt < 0 || $others < 0) { // (1) throw new InvalidArgumentException(); } // else (2) if ($cravat > 0 && $whiteShirt > 0) { // (3) $discount = self::CRAVAT_WHITE_SHIRT_DISCOUNT; } // else (4) if (($cravat + $whiteShirt + $others) >= self::TOTAL_PRODUCT_TO_DISCOUNT) { // (5) $discount += self::QUANTITY_DISCOUNT; } // else (6)

(1) => End

(2) (3) (5) => End

...

Và nên thêm 1 case để test cận cho điều kiện >= 7, vì dấu >= và dấu > thường rất dễ nhầm

tuanpt-0634

A đã comment gợi ý cho 1 vài bài, nên đọc lại guideline, checklist và refactor cả những phần khác

namttdh · 2021-05-29T10:53:38Z

Modules/Exercise03/Tests/Feature/Http/Controllers/ProductControllerTest.php

+
+        $controller = new ProductController($this->productServiceMock);
+
+        $this->assertInstanceOf(JsonResponse::class, $controller->checkout($request));


Đoạn này nên có thêm assert cho
->json(['discount' => $discount]);
Để xem xem là người dùng có truyền discount đúng cái trả về từ service không

namttdh · 2021-05-29T10:55:58Z

Modules/Exercise03/Tests/Unit/Services/ProductServiceTest.php

+        $this->assertEquals($expectedValue, $discount);
+    }
+
+    public function provideValidTotalProductsData()


Ở đây thiếu miss mất 1 case.
Giả sử các trường hợp dưới đây không tồn tại thì ra sao?

$totalProducts[Product::CRAVAT_TYPE] $totalProducts[Product::OTHER_TYPE] $totalProducts[Product::WHITE_SHIRT_TYPE]

Ngoài thì thì a cũng để ý các trương logic so sánh >= hiện tại anh chỉ đang test các trường hợp > nhưng đối với các trường hợp = thì đang không test

namttdh · 2021-05-29T10:57:58Z

Modules/Exercise04/Tests/Feature/Http/Controllers/CalendarControllerTest.php

+
+    public function test_index()
+    {
+        $this->calendarServiceMock->shouldReceive('getDateClass')


Trong controller này có logic, nên phải làm sao test đưuọc cả logic đấy. Muốn test được logic đấy thì có 2 cách
1 là cái mock này không nên chỉ shouldReceive và andReturn luôn.
Mình nên làm để biết thêm là cái method getDateClass gọi bao nhiêu lần và những tham số truyền vào là gì
2 là
$response->assertViewHas('calendars'); thì mình phải assert xem cả cái data của nó nữa

namttdh · 2021-05-29T10:58:48Z

Modules/Exercise05/Tests/Unit/Services/OrderServiceTest.php

+     * */
+    public function test_handle_discount($detailOrder, $expectedValue)
+    {
+        $resultOrder = $this->orderService->handleDiscount($detailOrder);


Trong provider này nên thêm có 1 case nữa check độ chính xác tới 2 chữ số thập phân vì cái round nó có thêm tham số thứ 2 là 2

$infoBill['price'] = round(($detailOrder['price'] * 80) / 100, 2);

Cái này nice to have 😄

Ngoài ra nữa thì anh để ý các logic >= nhá

namttdh · 2021-05-29T10:59:06Z

Modules/Exercise06/Tests/Unit/Services/CalculateServiceTest.php

+        $this->assertEquals($expectedValue, $time);
+    }
+
+    public function provideData()


Ở đây sẽ có 1 case, là giả sử không truyền vào has_watch thì kết quả sẽ ra sao.
Tất nhiên là không truyền vào has_watch thì default nó là false thì nó sẽ trùng 1 trong các số test của mình. Nhưng về cơ bản thì vẫn phải có case là không truyền vào cái gì cả

sangct-1630 added 6 commits May 19, 2021 01:40

UnitTest Module 03

7a6b87c

UnitTest Module 04

2accd82

UnitTest Module 05

0b2139a

UnitTest Module 06

17d41da

UnitTest Module 07

98bb649

Select exercises to do

3e838a2

sangct-1630 changed the title ~~Do homework~~ [SangCT] Do homework May 20, 2021

tuanpt-0634 reviewed May 25, 2021

View reviewed changes

tuanpt-0634 requested changes May 25, 2021

View reviewed changes

tuanpt-0634 mentioned this pull request May 25, 2021

[TruongBT] - UT Unit 2 #25

Closed

4 tasks

namttdh reviewed May 29, 2021

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[SangCT] Do homework #26

[SangCT] Do homework #26

Uh oh!

sangct-1630 commented May 19, 2021

Uh oh!

tuanpt-0634 May 25, 2021

Uh oh!

tuanpt-0634 May 25, 2021

Uh oh!

tuanpt-0634 May 25, 2021

Uh oh!

tuanpt-0634 May 25, 2021

Uh oh!

tuanpt-0634 May 25, 2021

Uh oh!

tuanpt-0634 left a comment

Uh oh!

namttdh May 29, 2021

Uh oh!

namttdh May 29, 2021 •

edited

Loading

Uh oh!

namttdh May 29, 2021

Uh oh!

namttdh May 29, 2021

Uh oh!

namttdh May 29, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


		$controller = new ProductController($this->productServiceMock);

		$this->assertInstanceOf(JsonResponse::class, $controller->checkout($request));

[SangCT] Do homework #26

Are you sure you want to change the base?

[SangCT] Do homework #26

Uh oh!

Conversation

sangct-1630 commented May 19, 2021

Purpose/Notes

Screenshot

Checklist (*)

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tuanpt-0634 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

namttdh May 29, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

namttdh May 29, 2021 •

edited

Loading