Skip to content

Commit 831cce1

Browse files
authored
Merge pull request #108 from iMattPro/fixes
Fix tests
2 parents 84f3f9b + dc1fccb commit 831cce1

2 files changed

Lines changed: 7 additions & 20 deletions

File tree

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ jobs:
347347

348348
# START Other Tests Job (SQLite 3 and mssql)
349349
other-tests:
350-
runs-on: ubuntu-22.04
350+
runs-on: ubuntu-20.04
351351
strategy:
352352
matrix:
353353
include:

tests/notification/notification_method_webpush_test.php

Lines changed: 6 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -328,9 +328,7 @@ public function test_notification_webpush($notification_type, $post_data, $expec
328328
'post_subject' => '',
329329
'post_username' => '',
330330
'forum_name' => '',
331-
],
332-
333-
$post_data);
331+
], $post_data);
334332
$notification_options = [
335333
'item_id' => $post_data['post_id'],
336334
'item_parent_id' => $post_data['topic_id'],
@@ -379,9 +377,7 @@ public function test_get_subscription($notification_type, $post_data, $expected_
379377
'post_subject' => '',
380378
'post_username' => '',
381379
'forum_name' => '',
382-
],
383-
384-
$post_data);
380+
], $post_data);
385381
$notification_options = [
386382
'item_id' => $post_data['post_id'],
387383
'item_parent_id' => $post_data['topic_id'],
@@ -425,9 +421,7 @@ public function test_notify_empty_queue($notification_type, $post_data, $expecte
425421
'post_subject' => '',
426422
'post_username' => '',
427423
'forum_name' => '',
428-
],
429-
430-
$post_data);
424+
], $post_data);
431425
$notification_options = [
432426
'item_id' => $post_data['post_id'],
433427
'item_parent_id' => $post_data['topic_id'],
@@ -477,9 +471,7 @@ public function test_notify_invalid_endpoint($notification_type, $post_data, $ex
477471
'post_subject' => '',
478472
'post_username' => '',
479473
'forum_name' => '',
480-
],
481-
482-
$post_data);
474+
], $post_data);
483475
$notification_options = [
484476
'item_id' => $post_data['post_id'],
485477
'item_parent_id' => $post_data['topic_id'],
@@ -534,7 +526,6 @@ public function test_notify_expired($notification_type, $post_data, $expected_us
534526
// Expire subscriptions for first user
535527
if (count($expected_users))
536528
{
537-
538529
$first_user_id = array_key_first($expected_users);
539530
$first_user_subs = $subscription_info[$first_user_id];
540531
unset($expected_delivered_users[$first_user_id]);
@@ -548,9 +539,7 @@ public function test_notify_expired($notification_type, $post_data, $expected_us
548539
'post_subject' => '',
549540
'post_username' => '',
550541
'forum_name' => '',
551-
],
552-
553-
$post_data);
542+
], $post_data);
554543
$notification_options = [
555544
'item_id' => $post_data['post_id'],
556545
'item_parent_id' => $post_data['topic_id'],
@@ -607,9 +596,7 @@ public function test_prune_notifications($notification_type, $post_data, $expect
607596
'post_subject' => '',
608597
'post_username' => '',
609598
'forum_name' => '',
610-
],
611-
612-
$post_data);
599+
], $post_data);
613600
$notification_options = [
614601
'item_id' => $post_data['post_id'],
615602
'item_parent_id' => $post_data['topic_id'],

0 commit comments

Comments
 (0)