From 9afd7df701142922a516894e516c8e0b26656d35 Mon Sep 17 00:00:00 2001 From: Justus Dieckmann Date: Mon, 13 May 2024 17:48:33 +0200 Subject: [PATCH] Fix codestyle --- step/email/tests/privacy/provider_test.php | 14 ++++++++++++++ tests/process_error_test.php | 4 +++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/step/email/tests/privacy/provider_test.php b/step/email/tests/privacy/provider_test.php index 4ba4da1e..c0031044 100644 --- a/step/email/tests/privacy/provider_test.php +++ b/step/email/tests/privacy/provider_test.php @@ -48,6 +48,20 @@ */ final class provider_test extends provider_testcase { + /** @var stdClass test user1 */ + private $user1; + /** @var stdClass test user2 */ + private $user2; + /** @var stdClass test user3 */ + private $user3; + + /** @var stdClass test course1 */ + private $course1; + /** @var stdClass test course2 */ + private $course2; + /** @var stdClass test course3 */ + private $course3; + /** * Basic setup for the provider tests. * diff --git a/tests/process_error_test.php b/tests/process_error_test.php index 49259054..217af94d 100644 --- a/tests/process_error_test.php +++ b/tests/process_error_test.php @@ -112,8 +112,10 @@ public function test_process_error_in_table(): void { $this->assertEquals($this->course->id, $record->courseid); if (version_compare(PHP_VERSION, '8.0', '<')) { $this->assertStringContainsString("Trying to get property 'id' of non-object", $record->errormessage); - } else { + } else if (version_compare(PHP_VERSION, '8.3', '<')) { $this->assertStringContainsString("Attempt to read property \"id\" on bool", $record->errormessage); + } else { + $this->assertStringContainsString("Attempt to read property \"id\" on false", $record->errormessage); } $this->assertEquals($process->id, $record->id); }