Skip to content

Commit 2911c81

Browse files
committed
test: align feature context with psalm 6.16
Signed-off-by: Vitor Mattos <vitor@php.rio>
1 parent a4337fe commit 2911c81

1 file changed

Lines changed: 9 additions & 7 deletions

File tree

features/bootstrap/FeatureContext.php

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
use Behat\Gherkin\Node\PyStringNode;
44
use Behat\Gherkin\Node\TableNode;
5-
use Behat\Step\Given;
65
use donatj\MockWebServer\MockWebServer;
76
use donatj\MockWebServer\RequestInfo;
87
use donatj\MockWebServer\Response as MockWebServerResponse;
@@ -23,17 +22,17 @@ public function __construct(?array $parameters = []) {
2322

2423
/**
2524
* @inheritDoc
25+
* @psalm-suppress MissingOverrideAttribute
2626
*/
27-
#[\Override]
2827
public function setCurrentUser(string $user): void {
2928
parent::setCurrentUser($user);
3029
Assert::assertEquals($this->currentUser, $user);
3130
}
3231

3332
/**
3433
* @inheritDoc
34+
* @psalm-suppress MissingOverrideAttribute
3535
*/
36-
#[\Override]
3736
public function assureUserExists(string $user): void {
3837
parent::assureUserExists($user);
3938
$lastRequest = $this->getLastREquest();
@@ -58,16 +57,17 @@ private function getLastRequest(): RequestInfo {
5857
* When whe run the test suit of this repository at GitHub Actions, is
5958
* necessary to consider that we haven't Nextcloud installed and mock
6059
* the real path of files.
60+
* @psalm-suppress MissingOverrideAttribute
6161
*/
62-
#[\Override]
6362
public static function findParentDirContainingFile(string $filename): string {
6463
return __DIR__;
6564
}
6665

6766
/**
6867
* @inheritDoc
68+
* @param TableNode|PyStringNode|array|null $body
69+
* @psalm-suppress MissingOverrideAttribute
6970
*/
70-
#[\Override]
7171
public function sendRequest(string $verb, string $url, $body = null, array $headers = [], array $options = []): void {
7272
parent::sendRequest($verb, $url, $body, $headers, $options);
7373
$lastRequest = $this->getLastRequest();
@@ -131,7 +131,9 @@ private function hasNestedPayload(array $payload): bool {
131131
return false;
132132
}
133133

134-
#[Given('set the response to:')]
134+
/**
135+
* @Given set the response to:
136+
*/
135137
public function setTheResponseTo(PyStringNode $response): void {
136138
// Mock response to be equal to body of request
137139
$this->mockServer->setDefaultResponse(new MockWebServerResponse(
@@ -141,8 +143,8 @@ public function setTheResponseTo(PyStringNode $response): void {
141143

142144
/**
143145
* @inheritDoc
146+
* @psalm-suppress MissingOverrideAttribute
144147
*/
145-
#[\Override]
146148
public function theResponseShouldBeAJsonArrayWithTheFollowingMandatoryValues(TableNode $table): void {
147149
$lastRequest = $this->getLastRequest();
148150
$parsedInput = $this->getParsedInputFromRequest($lastRequest);

0 commit comments

Comments
 (0)