@@ -22,13 +22,10 @@ public function testCreateSubscription(): void
2222 $ subscription = $ this ->getSubscriptionExample ();
2323 $ subscription = $ this ->client ->createSubscription ($ subscription );
2424
25- // Subscription tests
2625 self ::assertEquals (SubscriptionStatus::DRAFT , $ subscription ->getStatus ());
2726
28- // Validate schedule is either 'monthly' or a valid monthly cron expression
2927 $ this ->assertValidMonthlySchedule ($ subscription ->getSchedule ());
3028
31- // BillData tests
3229 self ::assertEquals (3.0 , $ subscription ->getBillData ()->getItems ()[0 ]->getPrice ());
3330 self ::assertEquals (2 , $ subscription ->getBillData ()->getItems ()[0 ]->getQuantity ());
3431 self ::assertEquals (1 , $ subscription ->getBillData ()->getItems ()[1 ]->getQuantity ());
@@ -46,7 +43,6 @@ public function testGetSubscription(): void
4643
4744 self ::assertEquals (SubscriptionStatus::DRAFT , $ subscription ->getStatus ());
4845
49- // Validate schedule is either 'monthly' or a valid monthly cron expression
5046 $ this ->assertValidMonthlySchedule ($ subscription ->getSchedule ());
5147
5248 self ::assertCount (2 , $ subscription ->getBillData ()->getItems ());
@@ -120,7 +116,6 @@ private function assertValidMonthlySchedule(string $schedule): void
120116 return ; // Standard monthly string is valid
121117 }
122118
123- // Split the cron expression by spaces
124119 $ parts = explode (' ' , $ schedule );
125120
126121 // A proper cron expression should have 6 parts: second minute hour dayOfMonth month dayOfWeek
@@ -140,14 +135,10 @@ private function assertValidMonthlySchedule(string $schedule): void
140135 self ::assertGreaterThanOrEqual (0 , (int )$ hour , "Hour must be ≥ 0: " . $ schedule );
141136 self ::assertLessThanOrEqual (23 , (int )$ hour , "Hour must be ≤ 23: " . $ schedule );
142137
143- // For a monthly schedule, day should be 1-28 (valid for all months)
144138 self ::assertGreaterThanOrEqual (1 , (int )$ dayOfMonth , "Day of month must be ≥ 1: " . $ schedule );
145139 self ::assertLessThanOrEqual (28 , (int )$ dayOfMonth , "Day of month must be ≤ 28: " . $ schedule );
146140
147- // For monthly schedule, month should be *
148141 self ::assertEquals ('* ' , $ month , "Month must be * for monthly schedule: " . $ schedule );
149-
150- // For monthly schedule, day of week should be *
151142 self ::assertEquals ('* ' , $ dayOfWeek , "Day of week must be * for monthly schedule: " . $ schedule );
152143 }
153144}
0 commit comments