[TruongBT] - UT Unit 2 by truongbt-0855 · Pull Request #25 · sun-unit-test-training/php-unit-test-homework

truongbt-0855 · 2021-05-18T15:45:17Z

Purpose/Notes

Screenshot

Ex 03

Ex 04

Ex 05

Ex 07

Ex 06

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-25T17:46:00Z

+    public function test_index_return_view_success()
+    {
+        $products = ['name' => 'test_product'];
+        $this->productService->shouldReceive('getAllProducts')->andReturn($products);
+        $response = $this->productController->index();
+        $this->assertEquals('exercise03::index', $response->getName());
+        $this->assertEquals(compact('products'), $response->getData());
+    }
+
+    public function test_checkout_return_view_success()
+    {
+        $discount = 10;
+        $paramRequest = [
+            1 => 5,
+            2 => 6,
+        ];
+        $this->productService->shouldReceive('calculateDiscount')->with($paramRequest)->andReturn($discount);
+
+        $url = action([ProductController::class, 'checkout']);
+        $response = $this->post($url, ['total_products' => $paramRequest]);
+        $response->assertOk();
+        $response->assertJsonPath('discount', $discount);
+    }


Cùng test controller mà dùng cả 2 cách trong cùng 1 class có gây khó hiểu không a?

không nha =))
anh cố tình viết vậy để hiểu hơn thôi

tuanpt-0634 · 2021-05-25T17:46:54Z

+        $this->assertEquals(compact('products'), $response->getData());
+    }
+
+    public function test_checkout_return_view_success()


Method này có thể nói cụ thể hơn là nó return json, ko phải return view

https://github.com/sun-unit-test-training/php-unit-test-homework/blob/master/Modules/Exercise03/Http/Controllers/ProductController.php#L37

chỗ này nó return view mà nhỉ ?

Đây đang test method checkout mà anh

tuanpt-0634 · 2021-05-25T17:54:42Z

+        $dataAll = ['abc' => 'def'];
+        $this->productModel->shouldReceive('all')->andReturn($dataAll);
+
+        $this->assertEquals($dataAll, $this->productRepository->all());


OK 100% code coverage. Nhưng function này expect return mảng Product Model thì tốt hơn, expect nó return như kia sẽ gây hiểu nhầm về cấu trúc data trả về

Theo quan điểm của em ở đây "unit test" là ko cần thiết, ko mang lại nhiều ý nghĩa, mà nên integration test với db.

tuanpt-0634 · 2021-05-25T17:57:55Z

+
+    public function test_get_all_products()
+    {
+        $dataAll = ['abc' => 'def'];


Như trên, dummy data thì cũng cho nó giống thật tí, service này nó không bao giờ trả về data dạng này, có chăng chỉ cùng kiểu là array

tuanpt-0634 · 2021-05-25T18:01:16Z

+    public function test_calculate_discount_white_shirt()
+    {
+        $discount = $this->productService->calculateDiscount([
+            Product::CRAVAT_TYPE => 2,
+            Product::WHITE_SHIRT_TYPE => 2,
+            Product::OTHER_TYPE => 1,
+        ]);
+
+        $this->assertEquals(ProductService::CRAVAT_WHITE_SHIRT_DISCOUNT, $discount);
+    }
+
+    public function test_calculate_discount_with_shirt_quantity()
+    {
+        $expected = ProductService::CRAVAT_WHITE_SHIRT_DISCOUNT + ProductService::QUANTITY_DISCOUNT;
+        $discount = $this->productService->calculateDiscount([
+            Product::CRAVAT_TYPE => 2,
+            Product::WHITE_SHIRT_TYPE => 2,
+            Product::OTHER_TYPE => 10,
+        ]);
+
+        $this->assertEquals($expected, $discount);
+    }
+
+    public function test_calculate_empty_input()
+    {
+        $discount = $this->productService->calculateDiscount([]);
+        $this->assertEquals(0, $discount);
+    }


Thiếu case anh nhé, case chỉ áp dụng qty discount. Cụ thể thì e có comment 1 vài cách xác định case ở đây => #26 (comment)

tuanpt-0634 · 2021-05-25T18:04:44Z

+    /**
+     * @dataProvider provider_get_date_class
+     */
+    public function test_get_date_class($date, $class)
+    {
+        $response =$this->calendarService->getDateClass($date, ['2021-05-14']);
+
+        $this->assertEquals($class, $response);
+    }
+
+    function provider_get_date_class()
+    {
+        return [
+            [
+                Carbon::createFromFormat('Y-m-d', '2021-05-16'),
+                CalendarService::COLOR_RED,
+            ],
+            [
+                Carbon::createFromFormat('Y-m-d', '2021-05-15'),
+                CalendarService::COLOR_BLUE,
+            ],
+            [
+                Carbon::createFromFormat('Y-m-d', '2021-05-14'),
+                CalendarService::COLOR_RED,
+            ],
+        ];
+    }


Nên có case để thể hiện màu đỏ ưu tiên hơn màu xanh không a? Tức là ngày lễ trùng vào thứ 7

Và thiếu case cho ngày bình thường

tuanpt-0634 · 2021-05-25T18:06:31Z

+        $this->assertEquals(compact('optionCoupons', 'optionReceives'), $response->getData());
+    }
+
+    public function test_store()


Expect là cần phải đặt tên "dài" hơn
https://sun-unit-test-training.github.io/php-testing-guideline/06-checklist/#1-self-describing-test-method

tuanpt-0634 · 2021-05-25T18:08:14Z

+    function provider_test_validation_fail()
+    {
+        return [
+            [
+                [],
+                [
+                    'price' => '',
+                    'option_receive' => '',
+                    'option_coupon' => '',
+                ],
+                [
+                    'price' => null,
+                    'option_receive' => null,
+                    'option_coupon' => null,
+                ],
+                [
+                    'price' => 'abc',
+                    'option_receive' => 3,
+                    'option_coupon' => 3,
+                ],
+            ]
+        ];
+    }


Có gì đó sai sai? Provider này chỉ đang return 1 data set

namttdh · 2021-05-29T10:42:36Z

+
+    public function test_index()
+    {
+        $this->calendarService->shouldReceive('getDateClass')->andReturn(1);


Đoạn mock này mình 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ì, vì trong controller nó cũng có logic, nếu viết test kiểu này sẽ không test được logic trong controller

namttdh · 2021-05-29T10:44:17Z

+use Modules\Exercise05\Services\OrderService;
+use Tests\TestCase;
+
+class OrderServiceTest extends TestCase


Nên test thêm các case = nữa.
Hiện tại những case so sánh >= thì anh chỉ test case > nhưng case = thì hổng có test nha

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 😄

https://github.com/truongbt-0855/php-unit-test-homework/blob/eff93c32c243259ce19c1f76868aa9ffed02595c/Modules/Exercise05/Services/OrderService.php#L14

hàm này làm gì có cái >= đâu e nhỉ ?

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

+        list($minBill2, $freeTime2) = CalculateService::CASE_2;
+        $freeTime = CalculateService::FREE_TIME_FOR_MOVIE;
+
+        return [


Ở đâ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ả

tuanpt-0634

Thanks!
PHPUnit C1 is 100%, C2 is 50% => still can improve

Code Coverage Report:       
  2021-06-22 09:58:30       
                            
 Summary:                   
  Classes: 100.00% (16/16)  
  Methods: 100.00% (28/28)  
  Paths:   50.00% (45/90)   
  Branches:   100.00% (71/71)
  Lines:   100.00% (115/115)

Infection Mutation Testing MSI: 80%

Metrics:
         Mutation Score Indicator (MSI): 80%
         Mutation Code Coverage: 100%
         Covered Code MSI: 80%

Final result: PASS

d10cn2btt added 5 commits May 18, 2021 22:42

exercise 03

9df5934

exercise 04

849abf8

exercise 05

5453023

exercise 07

6ded85f

exercise 06

9bcd644

truongbt-0855 changed the title ~~[WIP] - UT Unit 2~~ [TruongBT] - UT Unit 2 May 19, 2021

tuanpt-0634 reviewed May 25, 2021

View reviewed changes

tuanpt-0634 requested changes May 25, 2021

View reviewed changes

namttdh reviewed May 29, 2021

View reviewed changes

d10cn2btt added 2 commits June 21, 2021 16:04

fix comment & add case for coverage

46e424c

fix CI

dc684f2

tuanpt-0634 approved these changes Jun 25, 2021

View reviewed changes

anhnv-0443 approved these changes Jun 25, 2021

View reviewed changes

tuanpt-0634 added the passed label Jul 25, 2021

tuanpt-0634 closed this Jul 25, 2021

Uh oh!

Conversation

truongbt-0855 commented May 18, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

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!

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!

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!

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

truongbt-0855 commented May 18, 2021 •

edited

Loading