Skip to content

Commit b4fcc49

Browse files
committed
Adding permanent flag to deleteSubject Request
1 parent 5f1ac52 commit b4fcc49

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

src/Requests.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,13 +185,16 @@ public static function validateSchemaId($schemaId): string
185185

186186
/**
187187
* @param string $subjectName
188+
* @param bool $permanent
188189
* @return RequestInterface
189190
*/
190-
public static function deleteSubjectRequest(string $subjectName): RequestInterface
191+
public static function deleteSubjectRequest(string $subjectName, bool $permanent): RequestInterface
191192
{
193+
$query = $permanent ? "true" : "false";
194+
192195
return new Request(
193196
'DELETE',
194-
Utils::uriFor("/subjects/$subjectName"),
197+
Utils::uriFor("/subjects/$subjectName?permanent=$query"),
195198
Constants::ACCEPT_HEADER
196199
);
197200
}

test/RequestsTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -364,10 +364,10 @@ public function it_should_validate_valid_schema_ids(): void
364364
*/
365365
public function it_should_produce_a_valid_subject_deletion_request(): void
366366
{
367-
$request = Requests::deleteSubjectRequest('test');
367+
$request = Requests::deleteSubjectRequest('test', false);
368368

369369
self::assertEquals('DELETE', $request->getMethod());
370-
self::assertEquals('subjects/test', $request->getUri());
370+
self::assertEquals('subjects/test?permanent=false', $request->getUri());
371371
self::assertEquals([Constants::ACCEPT => [Constants::ACCEPT_HEADER[Constants::ACCEPT]]], $request->getHeaders());
372372
}
373373

0 commit comments

Comments
 (0)