Skip to content

Commit fa4bab8

Browse files
committed
refactor: AJDA-2680 rename detailSubscription() to getSubscription()
1 parent 38d140a commit fa4bab8

3 files changed

Lines changed: 8 additions & 8 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ foreach ($subscriptions as $subscription) {
7070
}
7171

7272
// fetch a single subscription by ID
73-
$detail = $subscriptionClient->detailSubscription($created->getId());
73+
$subscription = $subscriptionClient->getSubscription($created->getId());
7474

7575
// delete a subscription
7676
$subscriptionClient->deleteSubscription($created->getId());

src/SubscriptionClient.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public function deleteSubscription(string $id): void
5151
$this->sendRequest($request);
5252
}
5353

54-
public function detailSubscription(string $id): ResponseSubscription
54+
public function getSubscription(string $id): ResponseSubscription
5555
{
5656
$request = new Request(
5757
'GET',

tests/SubscriptionClientFunctionalTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ public function testDeleteSubscriptionHeaders(): void
146146
);
147147
}
148148

149-
public function testDetailSubscriptionHeaders(): void
149+
public function testGetSubscriptionHeaders(): void
150150
{
151151
$mock = new MockHandler([
152152
new Response(
@@ -172,7 +172,7 @@ public function testDetailSubscriptionHeaders(): void
172172
['handler' => $stack, 'backoffMaxTries' => 3, 'userAgent' => 'Test'],
173173
);
174174

175-
$result = $client->detailSubscription('subscription-123');
175+
$result = $client->getSubscription('subscription-123');
176176

177177
/** @var Request $request */
178178
$request = $requestHistory[0]['request'];
@@ -262,10 +262,10 @@ public function testListAndDeleteSubscriptionLifecycle(): void
262262
));
263263
self::assertNotEmpty($created->getId());
264264

265-
// detail — must return the same subscription
266-
$detail = $client->detailSubscription($created->getId());
267-
self::assertSame($created->getId(), $detail->getId());
268-
self::assertSame('job-failed', $detail->getEvent());
265+
// get — must return the same subscription
266+
$fetched = $client->getSubscription($created->getId());
267+
self::assertSame($created->getId(), $fetched->getId());
268+
self::assertSame('job-failed', $fetched->getEvent());
269269

270270
// list — must contain the new subscription
271271
$beforeDelete = $client->listSubscriptions();

0 commit comments

Comments
 (0)