Skip to content

Commit 87fa5f5

Browse files
committed
Apply fixes from StyleCI
1 parent 723301d commit 87fa5f5

20 files changed

+32
-76
lines changed

tests/Api/EnvironmentsTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ public function shouldRemoveEnvironment(): void
185185
$api->expects($this->once())
186186
->method('delete')
187187
->with('projects/1/environments/3')
188-
->willReturn($expectedBool);;
188+
->willReturn($expectedBool);
189189
$this->assertEquals($expectedBool, $api->remove(1, 3));
190190
}
191191

@@ -200,7 +200,7 @@ public function shouldStopEnvironment(): void
200200
$api->expects($this->once())
201201
->method('post')
202202
->with('projects/1/environments/3/stop')
203-
->willReturn($expectedBool);;
203+
->willReturn($expectedBool);
204204
$this->assertEquals($expectedBool, $api->stop(1, 3));
205205
}
206206

tests/Api/GroupBoardsTest.php

-2
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ public function shouldRemoveIssueBoard(): void
101101
->method('delete')
102102
->with('groups/1/boards/2')
103103
->willReturn($expectedBool);
104-
;
105104

106105
$this->assertEquals($expectedBool, $api->remove(1, 2));
107106
}
@@ -234,7 +233,6 @@ public function shouldDeleteList(): void
234233
->method('delete')
235234
->with('groups/1/boards/2/lists/3')
236235
->willReturn($expectedBool);
237-
;
238236

239237
$this->assertEquals($expectedBool, $api->deleteList(1, 2, 3));
240238
}

tests/Api/GroupsEpicsTest.php

-1
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ public function shouldRemoveEpic(): void
101101
->method('delete')
102102
->with('groups/1/epics/2')
103103
->willReturn($expectedBool);
104-
;
105104

106105
$this->assertEquals($expectedBool, $api->remove(1, 2));
107106
}

tests/Api/GroupsMilestonesTest.php

-1
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,6 @@ public function shouldRemoveMilestone(): void
201201
->method('delete')
202202
->with('groups/1/milestones/2')
203203
->willReturn($expectedBool);
204-
;
205204

206205
$this->assertEquals($expectedBool, $api->remove(1, 2));
207206
}

tests/Api/GroupsTest.php

+3-8
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,6 @@ public function shouldTransferProjectToGroup(): void
216216
->method('post')
217217
->with('groups/1/projects/2')
218218
->willReturn($expectedBool);
219-
;
220219

221220
$this->assertEquals($expectedBool, $api->transfer(1, 2));
222221
}
@@ -326,7 +325,6 @@ public function shouldRemoveMember(): void
326325
->method('delete')
327326
->with('groups/1/members/2')
328327
->willReturn($expectedBool);
329-
;
330328

331329
$this->assertEquals($expectedBool, $api->removeMember(1, 2));
332330
}
@@ -343,7 +341,6 @@ public function shouldRemoveGroup(): void
343341
->method('delete')
344342
->with('groups/1')
345343
->willReturn($expectedBool);
346-
;
347344

348345
$this->assertEquals($expectedBool, $api->remove(1));
349346
}
@@ -455,7 +452,6 @@ public function shouldRemoveLabel(): void
455452
->method('delete')
456453
->with('groups/1/labels/456', [])
457454
->willReturn($expectedBool);
458-
;
459455

460456
$this->assertEquals($expectedBool, $api->removeLabel(1, 456));
461457
}
@@ -591,7 +587,6 @@ public function shouldRemoveVariable(): void
591587
->method('delete')
592588
->with('groups/1/variables/ftp_password')
593589
->willReturn($expectedBool);
594-
;
595590

596591
$this->assertEquals($expectedBool, $api->removeVariable(1, 'ftp_password'));
597592
}
@@ -846,7 +841,7 @@ public function shouldGetActiveDeployTokens(): void
846841
$api->expects($this->once())
847842
->method('get')
848843
->with('groups/1/deploy_tokens', ['active' => true])
849-
->willReturn([]);;
844+
->willReturn([]);
850845

851846
$this->assertEquals([], $api->deployTokens(1, true));
852847
}
@@ -875,7 +870,7 @@ public function shouldGetInactiveDeployTokens(): void
875870
$api->expects($this->once())
876871
->method('get')
877872
->with('groups/1/deploy_tokens', ['active' => false])
878-
->willReturn([]);;
873+
->willReturn([]);
879874

880875
$this->assertEquals([], $api->deployTokens(1, false));
881876
}
@@ -936,7 +931,7 @@ public function shouldDeleteDeployToken(): void
936931
$api->expects($this->once())
937932
->method('delete')
938933
->with('groups/1/deploy_tokens/2')
939-
->willReturn($expectedBool);;
934+
->willReturn($expectedBool);
940935

941936
$this->assertEquals($expectedBool, $api->deleteDeployToken(1, 2));
942937
}

tests/Api/IssueBoardsTest.php

-2
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ public function shouldRemoveIssueBoard(): void
101101
->method('delete')
102102
->with('projects/1/boards/2')
103103
->willReturn($expectedBool);
104-
;
105104

106105
$this->assertEquals($expectedBool, $api->remove(1, 2));
107106
}
@@ -234,7 +233,6 @@ public function shouldDeleteList(): void
234233
->method('delete')
235234
->with('projects/1/boards/2/lists/3')
236235
->willReturn($expectedBool);
237-
;
238236

239237
$this->assertEquals($expectedBool, $api->deleteList(1, 2, 3));
240238
}

tests/Api/IssueSubscribeTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public function testSubscribeIssue(): void
2828
$api->expects($this->once())
2929
->method('post')
3030
->with('projects/1/issues/2/subscribe')
31-
->willReturn($expectedValue);;
31+
->willReturn($expectedValue);
3232

3333
$this->assertEquals($expectedValue, $api->subscribe(1, 2));
3434
}
@@ -40,7 +40,7 @@ public function testUnsubscribeIssue(): void
4040
$api->expects($this->once())
4141
->method('post')
4242
->with('projects/1/issues/2/unsubscribe')
43-
->willReturn($expectedValue);;
43+
->willReturn($expectedValue);
4444

4545
$this->assertEquals($expectedValue, $api->unsubscribe(1, 2));
4646
}

tests/Api/IssuesTest.php

-3
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,6 @@ public function shouldRemoveNote(): void
304304
->method('delete')
305305
->with('projects/1/issues/2/notes/3')
306306
->willReturn($expectedBool);
307-
;
308307

309308
$this->assertEquals($expectedBool, $api->removeNote(1, 2, 3));
310309
}
@@ -409,7 +408,6 @@ public function shouldRemoveDiscussionNote(): void
409408
->method('delete')
410409
->with('projects/1/issues/2/discussions/abc/notes/3')
411410
->willReturn($expectedBool);
412-
;
413411

414412
$this->assertEquals($expectedBool, $api->removeDiscussionNote(1, 2, 'abc', 3));
415413
}
@@ -531,7 +529,6 @@ public function shouldRevokeAwardEmoji(): void
531529
->method('delete')
532530
->with('projects/1/issues/2/award_emoji/3')
533531
->willReturn($expectedBool);
534-
;
535532

536533
$this->assertEquals(true, $api->removeAwardEmoji(1, 2, 3));
537534
}

tests/Api/JobsTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ public function shouldGetArtifacts(): void
138138
->method('getAsResponse')
139139
->with('projects/1/jobs/3/artifacts')
140140
->willReturn($returnedStream);
141-
;
142141

143142
$this->assertEquals('foobar', $api->artifacts(1, 3)->getContents());
144143
}
@@ -155,7 +154,6 @@ public function shouldGetArtifactsByJobId(): void
155154
->method('getAsResponse')
156155
->with('projects/1/jobs/3/artifacts/artifact_path')
157156
->willReturn($returnedStream);
158-
;
159157

160158
$this->assertEquals('foobar', $api->artifactByJobId(1, 3, 'artifact_path')->getContents());
161159
}
@@ -174,7 +172,6 @@ public function shouldGetArtifactsByRefName(): void
174172
'job' => 'job name',
175173
])
176174
->willReturn($returnedStream);
177-
;
178175

179176
$this->assertEquals('foobar', $api->artifactsByRefName(1, 'master', 'job name')->getContents());
180177
}
@@ -192,7 +189,7 @@ public function shouldGetArtifactByRefName(): void
192189
'job' => 'job name',
193190
])
194191
->willReturn($returnedStream);
195-
;
192+
196193
$this->assertEquals('foobar', $api->artifactByRefName(1, 'master', 'job name', 'artifact_path')->getContents());
197194
}
198195

@@ -208,7 +205,6 @@ public function shouldGetTrace(): void
208205
->method('get')
209206
->with('projects/1/jobs/3/trace')
210207
->willReturn($expectedString);
211-
;
212208

213209
$this->assertEquals($expectedString, $api->trace(1, 3));
214210
}

tests/Api/MergeRequestsTest.php

+1-4
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,6 @@ public function shouldRemoveNote(): void
345345
->method('delete')
346346
->with('projects/1/merge_requests/2/notes/3')
347347
->willReturn($expectedBool);
348-
;
349348

350349
$this->assertEquals($expectedBool, $api->removeNote(1, 2, 3));
351350
}
@@ -535,7 +534,6 @@ public function shouldRemoveDiscussionNote(): void
535534
->method('delete')
536535
->with('projects/1/merge_requests/2/discussions/abc/notes/3')
537536
->willReturn($expectedBool);
538-
;
539537

540538
$this->assertEquals($expectedBool, $api->removeDiscussionNote(1, 2, 'abc', 3));
541539
}
@@ -657,7 +655,6 @@ public function shouldRevokeMergeRequestAwardEmoji(): void
657655
->method('delete')
658656
->with('projects/1/merge_requests/2/award_emoji/3')
659657
->willReturn($expectedBool);
660-
;
661658

662659
$this->assertEquals(true, $api->removeAwardEmoji(1, 2, 3));
663660
}
@@ -868,7 +865,7 @@ public function shoudDeleteLevelRule(): void
868865
$api->expects($this->once())
869866
->method('delete')
870867
->with('projects/1/merge_requests/2/approval_rules/3')
871-
->willReturn($expectedValue);;
868+
->willReturn($expectedValue);
872869

873870
$this->assertEquals($expectedValue, $api->deleteLevelRule(1, 2, 3));
874871
}

tests/Api/MilestonesTest.php

-1
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ public function shouldRemoveMilestone(): void
101101
->method('delete')
102102
->with('projects/1/milestones/2')
103103
->willReturn($expectedBool);
104-
;
105104

106105
$this->assertEquals($expectedBool, $api->remove(1, 2));
107106
}

tests/Api/PackagesTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ public function shouldRemovePackage(): void
9797
$api->expects($this->once())
9898
->method('delete')
9999
->with('projects/1/packages/1')
100-
->willReturn($expectedBool);;
100+
->willReturn($expectedBool);
101101

102102
$this->assertEquals($expectedBool, $api->remove(1, 1));
103103
}
@@ -113,7 +113,7 @@ public function shouldRemovePackageFile(): void
113113
$api->expects($this->once())
114114
->method('delete')
115115
->with('projects/1/packages/1/package_files/25')
116-
->willReturn($expectedBool);;
116+
->willReturn($expectedBool);
117117

118118
$this->assertEquals($expectedBool, $api->removeFile(1, 1, 25));
119119
}

0 commit comments

Comments
 (0)