[Product Reviews API] - minor changes for tests

Sylius · pamil · Jan 5, 2018 · Oct 4, 2017 · Oct 4, 2017 · Oct 5, 2017
commit 3cf247a960146d3daf88bfef40ff4b6973d3dc65
diff --git a/tests/Controller/ProductReviewApiTest.php b/tests/Controller/ProductReviewApiTest.php
@@ -288,7 +288,7 @@ public function it_does_not_allows_accept_product_review_if_it_has_not_new_statu
         $product = $productReviewsData['product1'];
 
          /** @var ReviewInterface $productReview */
-        $productReview = $productReviewsData['productReview2'];
+        $productReview = $productReviewsData['productReview3'];
 
         $this->client->request('POST', $this->getReviewUrl($product, $productReview) . '/accept', [], [], static::$authorizedHeaderWithAccept);
         $response = $this->client->getResponse();

diff --git a/tests/DataFixtures/ORM/resources/product_reviews.yml b/tests/DataFixtures/ORM/resources/product_reviews.yml
@@ -26,18 +26,19 @@ Sylius\Component\Core\Model\ProductReview:
         rating: 4
         comment: "mug_review_best_comment"
         author: "@customer_example1"
+        status: "new"
         reviewSubject: "@product1"
     productReview2:
         title: mug_review_good
         rating: 3
         comment: "mug_review_good_comment"
         author: "@customer_example1"
-        status: "rejected"
+        status: "new"
         reviewSubject: "@product1"
     productReview3:
         title: mug_review_bad
         rating: 1
         comment: "mug_review_bad_comment"
         author: "@customer_example1"
-        status: "accepted"
+        status: "rejected"
         reviewSubject: "@product1"
diff --git a/tests/Responses/Expected/product_review/accept_response.json b/tests/Responses/Expected/product_review/accept_response.json
@@ -49,7 +49,7 @@
                         }
                     }
                 },
-                "status": "rejected",
+                "status": "new",
                 "createdAt": @string@,
                 "updatedAt": @string@
             },
@@ -71,12 +71,12 @@
                         }
                     }
                 },
-                "status": "accepted",
+                "status": "rejected",
                 "createdAt": @string@,
                 "updatedAt": @string@
               }
         },
-        "averageRating": 2.5,
+        "averageRating": 4,
         "images": [],
         "_links": {
             "self": {

diff --git a/tests/Responses/Expected/product_review/create_response.json b/tests/Responses/Expected/product_review/create_response.json
@@ -69,7 +69,7 @@
                         }
                     }
                 },
-                "status": "rejected",
+                "status": "new",
                 "createdAt": @string@,
                 "updatedAt": @string@
             },
@@ -91,7 +91,7 @@
                         }
                     }
                 },
-                "status": "accepted",
+                "status": "rejected",
                 "createdAt": @string@,
                 "updatedAt": @string@
             }

diff --git a/tests/Responses/Expected/product_review/reject_response.json b/tests/Responses/Expected/product_review/reject_response.json
@@ -49,7 +49,7 @@
                         }
                     }
                 },
-                "status": "rejected",
+                "status": "new",
                 "createdAt": @string@,
                 "updatedAt": @string@
             },
@@ -71,7 +71,7 @@
                         }
                     }
                 },
-                "status": "accepted",
+                "status": "rejected",
                 "createdAt": @string@,
                 "updatedAt": @string@
             }

diff --git a/tests/Responses/Expected/product_review/show_response.json b/tests/Responses/Expected/product_review/show_response.json
@@ -49,7 +49,7 @@
                         }
                     }
                 },
-                "status": "rejected",
+                "status": "new",
                 "createdAt": @string@,
                 "updatedAt": @string@
             },
@@ -71,7 +71,7 @@
                         }
                     }
                 },
-                "status": "accepted",
+                "status": "rejected",
                 "createdAt": @string@,
                 "updatedAt": @string@
             }