Skip to content

Commit d4185cf

Browse files
committed
Adding permanent flag to deleteSubject Request
1 parent 879ac47 commit d4185cf

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
@@ -333,10 +333,10 @@ public function it_should_validate_valid_schema_ids(): void
333333
*/
334334
public function it_should_produce_a_valid_subject_deletion_request(): void
335335
{
336-
$request = Requests::deleteSubjectRequest('test');
336+
$request = Requests::deleteSubjectRequest('test', false);
337337

338338
self::assertEquals('DELETE', $request->getMethod());
339-
self::assertEquals('/subjects/test', $request->getUri());
339+
self::assertEquals('/subjects/test?permanent=false', $request->getUri());
340340
self::assertEquals([Constants::ACCEPT => [Constants::ACCEPT_HEADER[Constants::ACCEPT]]], $request->getHeaders());
341341
}
342342

0 commit comments

Comments
 (0)