Page MenuHomeDevCentral
Differential D2537

Test incomplete reports
ClosedPublic
Actions

Authored by dereckson on Feb 17 2022, 21:44.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 22, 13:15
Unknown Object (File)
Fri, Sep 20, 23:16
Unknown Object (File)
Wed, Sep 18, 11:08
Unknown Object (File)
Tue, Sep 17, 14:44
Unknown Object (File)
Mon, Sep 16, 22:02
Unknown Object (File)
Sat, Sep 14, 04:25
Unknown Object (File)
Sat, Sep 14, 04:25
Unknown Object (File)
Sat, Sep 14, 04:25
View All Files
Subscribers
None

Details

Reviewers
DorianWinty
Commits
rKREPORT29ee3dae8454: Test incomplete reports
Summary

Provide 4 flavours of report: full, empty, no metadata, only metadata.

That allows to test code robustness when any part is missing.

Test Plan

Provide new tests. They pass for full report and fail for the others.

This change is only to provide tests, follow-up changes will fix code.

Diff Detail

Repository
rKREPORT Keruald Report
Lint
Lint Passed
Unit
Test Failures
Branch
arcpatch-D2537
Build Status
Buildable 3968
Build 4220: arc lint + arc unit

Unit TestsFailed
View All

TimeTest
5 msKeruald\Reporting\Tests\Output\HTMLOutputTest
4 msKeruald\Reporting\Tests\Output\MarkdownOutputTest
5 msKeruald\Reporting\Tests\Output\XMLOutputTest

Event Timeline

dereckson requested review of this revision.Feb 17 2022, 21:44
dereckson created this revision.
Harbormaster completed remote builds in B3964: Diff 6401.Feb 17 2022, 21:44
DorianWinty mentioned this in D2538: Handle incomplete reports.Feb 19 2022, 08:49
DorianWinty added a child revision: D2538: Handle incomplete reports.Feb 19 2022, 08:52
dereckson planned changes to this revision.Feb 19 2022, 09:16

XML part should be amended per D2538 changes.

DorianWinty accepted this revision.Feb 19 2022, 10:57
DorianWinty updated this revision to Diff 6406.Feb 19 2022, 11:04
  • modify error
This revision is now accepted and ready to land.Feb 19 2022, 11:04
Harbormaster completed remote builds in B3968: Diff 6406.Feb 19 2022, 11:04
DorianWinty accepted this revision.Feb 19 2022, 11:05
This revision was landed with ongoing or failed builds.Feb 19 2022, 12:52
Closed by commit rKREPORT29ee3dae8454: Test incomplete reports (authored by DorianWinty). · Explain Why
This revision was automatically updated to reflect the committed changes.
DorianWinty added a commit: rKREPORT29ee3dae8454: Test incomplete reports.
DorianWinty mentioned this in rKREPORT3cd3b60859ba: Handle incomplete reports.Feb 22 2022, 17:32

Revision Contents
Changeset List

Diff 6406

View Options

tests/Output/HTMLOutputTest.php

Loading...
View Options

tests/Output/MarkdownOutputTest.php

Loading...
View Options

tests/Output/XMLOutputTest.php

Loading...
View Options

tests/WithSampleReport.php

Loading...
View Options

tests/data/empty.html

Loading...
View Options

tests/data/empty.md

Loading...
View Options

tests/data/empty.xml

Loading...
View Options

tests/data/no_metadata.html

Loading...
View Options

tests/data/no_metadata.md

Loading...
View Options

tests/data/no_metadata.xml

Loading...
View Options

tests/data/only_metadata.html

Loading...
View Options

tests/data/only_metadata.md

Loading...
View Options

tests/data/only_metadata.xml

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator