Skip to content

Commit 858f18d

Browse files
committed
remove unnecessary assertion
1 parent b02ae7b commit 858f18d

File tree

1 file changed

+1
-14
lines changed

1 file changed

+1
-14
lines changed

tests/subscriptions_test.php

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -365,14 +365,6 @@ public function test_moodleoverflow_discussion_subscription_moodleoverflow_subsc
365365
[$author] = $this->helper_create_users($course, 1);
366366

367367
// Enrol the user in the moodleoverflow.
368-
// If a subscription was added, we get the record ID.
369-
$this->assertIsInt(subscriptions::subscribe_user(
370-
$author->id,
371-
$moodleoverflow,
372-
$modulecontext
373-
));
374-
375-
// If we already have a subscription when subscribing the user, we get a boolean (true).
376368
$this->assertTrue(subscriptions::subscribe_user($author->id, $moodleoverflow, $modulecontext));
377369

378370
// Check that the user is currently subscribed to the moodleoverflow.
@@ -383,12 +375,7 @@ public function test_moodleoverflow_discussion_subscription_moodleoverflow_subsc
383375
unset($post);
384376

385377
// Check that the user is subscribed to the discussion too.
386-
$this->assertTrue(subscriptions::is_subscribed(
387-
$author->id,
388-
$moodleoverflow,
389-
$modulecontext,
390-
$discussion
391-
));
378+
$this->assertTrue(subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion));
392379
}
393380

394381
/**

0 commit comments

Comments
 (0)