Details

Reviewers

dereckson

Maniphest Tasks

T1725: Upgrade Laravel

Commits

rNOTIF327396a753b0: Update Laravel 8

Summary

Update to Laravel 8

Ref T1725

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Errors

Unit

Test Failures

Branch

arcpatch-D2674

Build Status

Buildable 4204
Build 4456: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	9 ms	Nasqueron\Notifications\Tests\Actions\ActionsReportTest Nasqueron\Notifications\Tests\Actions\ActionsReportTest::testReport data/report.json and rendered report differ too much. Try $this->assertEquals($expectedReport, $actualReport) to see a diff.
	324 ms	Nasqueron\Notifications\Tests\PayloadFullTest Nasqueron\Notifications\Tests\PayloadFullTest::testPost Unable to find JSON fragment:
	170 ms	Nasqueron\Notifications\Tests\Jobs\NotifyNewCommitsToDiffusionTest Nasqueron\Notifications\Tests\Jobs\NotifyNewCommitsToDiffusionTest::testJobWhenThereIsNoPhabricatorInstanceForTheProject This test did not perform any assertions
	195 ms	Nasqueron\Notifications\Tests\PlaceholderTest Nasqueron\Notifications\Tests\PlaceholderTest::testPlaceholder This test is annotated with "@doesNotPerformAssertions" but performed 2 assertions
	118 ms	Nasqueron\Notifications\Tests\StatusTest Nasqueron\Notifications\Tests\StatusTest::testStatus This test is annotated with "@doesNotPerformAssertions" but performed 2 assertions
		View Full Test Results (2 Failed · 3 Broken · 54 Passed)

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Harbormaster completed remote builds in B4197: Diff 6758.Apr 18 2022, 17:39

correct arc lint bug

Harbormaster completed remote builds in B4198: Diff 6759.Apr 18 2022, 20:37

DorianWinty added a project: Restricted Project.Apr 19 2022, 07:43

DorianWinty edited the summary of this revision. (Show Details)Apr 19 2022, 07:59

DorianWinty added a task: T1725: Upgrade Laravel.

adding codestyle
adding on the up of file the class that is use
adding using str_ instead of str::

Harbormaster completed remote builds in B4199: Diff 6760.Apr 19 2022, 18:45

wip

Harbormaster completed remote builds in B4200: Diff 6763.Apr 19 2022, 20:09

removing last str::

Harbormaster completed remote builds in B4202: Diff 6766.Apr 19 2022, 22:29

dereckson added a subscriber: dereckson.Apr 20 2022, 08:44

dereckson added inline comments.

app/Actions/AMQPAction.php
34 ↗	(On Diff #6766)	Not in the scope of the upgrade. Those esthetic changes are welcome, but: it's the other way around, ie the PREVIOUS `public function __construct )` was the correct one it should be done in a separate change against main branch (we can merge esthetic changes before upgrade without any issue)
app/Actions/Action.php
20 ↗	(On Diff #6766)	Probably created by a `phpcbf` with MediaWiki setting. The previous code was correct.
33 ↗	(On Diff #6766)	Same issue, previous file was fully correct in style.

correction err

Harbormaster completed remote builds in B4203: Diff 6767.Apr 22 2022, 15:50

correct tests
update some packages

Harbormaster completed remote builds in B4204: Diff 6768.Apr 22 2022, 22:10

correcting some Tests

Harbormaster completed remote builds in B4214: Diff 6779.May 4 2022, 19:39

correct to make work the tests

Harbormaster completed remote builds in B4215: Diff 6781.May 7 2022, 19:24

DorianWinty published this revision for review.May 7 2022, 19:31

DorianWinty retitled this revision from WIP Update Laravel 8 to Update Laravel 8.

modify test
remove unused use

Harbormaster completed remote builds in B4217: Diff 6785.May 7 2022, 20:44

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/253/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/253/console

adding back one removed file

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/254/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/254/console

Harbormaster failed remote builds in B4222: Diff 6790!May 8 2022, 14:52

dereckson mentioned this in D2683: Modernize build.xml.May 8 2022, 15:17

rebased

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/256/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/256/console

Harbormaster failed remote builds in B4225: Diff 6796!May 8 2022, 17:01

dereckson requested changes to this revision.May 8 2022, 17:33

dereckson added inline comments.

app/Analyzers/GitHub/Events/Event.php
36	Extraneous added line
51	extraneous added line
app/Analyzers/GitHub/Events/IssueCommentEvent.php
18–20	MediaWiki style
app/Analyzers/GitHub/Events/PullRequestEvent.php
18	MediaWiki
tests/TestCase.php
5	Restore that division Our app External libs Global space

This revision now requires changes to proceed.May 8 2022, 17:33

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/257/ for more details.

DorianWinty retitled this revision from Update Laravel 8 to WIP Update Laravel 8.May 9 2022, 18:14

modify folowing comments but need more to reading

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/258/ for more details.

Harbormaster completed remote builds in B4228: Diff 6799.May 9 2022, 18:31

DorianWinty retitled this revision from WIP Update Laravel 8 to Update Laravel 8.May 9 2022, 18:31

correcting style problem due to mediawiki phpcs style

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/259/ for more details.

Harbormaster completed remote builds in B4229: Diff 6800.May 10 2022, 17:38

correcting MW style to old style

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/260/ for more details.

Harbormaster completed remote builds in B4230: Diff 6801.May 10 2022, 17:54

phpcbf

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/261/ for more details.

Harbormaster completed remote builds in B4231: Diff 6802.May 10 2022, 18:05

dereckson requested changes to this revision.May 12 2022, 11:37

dereckson added inline comments.

app/Config/Reporting/BaseReportEntry.php
11–13	"public abstract" is more convenient as it's coherent with "public static": That would mean an order `<visibility> [special status] function <function name>`. More generally, run automated tools and accept ANY modification they do is a probably not a good idea, `git add -p` to filter the useful ones is very convenient with those "magic" tools.
app/Providers/RouteServiceProvider.php
36–37	MediaWiki style
composer.json
23–24	Two needs to be restored now we're up to date: phploc phpcpd
53	For phpmd, it's well ruleset.xml
tests/Analyzers/GitHub/GitHubPayloadAnalyzerTest.php
68	MediaWiki style
tests/Analyzers/Phabricator/PhabricatorPayloadAnalyzerTest.php
90	MW
tests/Config/FeaturesTest.php
25	MW
tests/Console/Commands/NotificationsPayloadTest.php
49	MW (the two lines)
tests/Notifications/DockerHubNotificationTest.php
10	Extraneous line