Merge pull request #36 from Mr-Basirnia:develop

🧪 test(database): delete record
Mr-Basirnia · Oct 20, 2022 · 0b8390f · 0b8390f
2 parents f0d5043 + 8d9dc93
commit 0b8390f
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/tdd-project/index.php b/tdd-project/index.php
@@ -47,4 +47,9 @@ function request()
 if ($_SERVER['REQUEST_METHOD'] === 'GET') {
     $result = $pdoQuery->table('bugs')->find(request()['id']);
     jsonResponse($result, 200);
+}
+if ($_SERVER['REQUEST_METHOD'] === 'DELETE') {
+
+    $result = $pdoQuery->table('bugs')->where('id', request()['id'])->delete();
+    jsonResponse(null, 204);
 }
diff --git a/tdd-project/tests/functional/CrudTest.php b/tdd-project/tests/functional/CrudTest.php
@@ -89,6 +89,30 @@ public function testItCanFetchDataWithApi(array $data): void
         $this->assertArrayHasKey('id', json_decode($response->getBody(), true));
     }
 
+    /**
+     * test delete record with delete api request
+     *
+     * @depends testItCanCreateDataWithApi
+     * @param array $data created record
+     * @return void
+     */
+    public function testItCanDeleteDataWithApi(array $data): void
+    {
+        $response = $this->httpClient->delete('index.php', [
+            'json' => [
+                'id' => $data['id']
+            ]
+        ]);
+
+        // if record is deleted, return null
+        $result = $this->queryBuilder
+            ->table('bugs')
+            ->find($data['id']);
+
+        $this->assertEquals(204, $response->getStatusCode());
+        $this->assertNull($result);
+    }
+
     /**
      * @throws ConfigFileNotValidException|PdoDatabaseException
      */