Paths

Table of Contentst

Differential D25820

Fix unit test failure in link-alternate
ClosedPublic
Actions

Authored by aklapper on Sat, Sep 7, 04:36.

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Commits

rPf25c531e5685: Fix unit test failure in link-alternate

Summary

arc unit --everything triggers a FAIL PhutilRemarkupEngineTestCase::testEngine due to adding the CSS class remarkup-link-ext in rP349f006904fabf1d4df31ff4840502af3ab379a7. Thus update it accordingly.

Test Plan

Run arc unit.

Diff Detail

Repository

rP Phorge

Branch

fixLinkAlternateTest

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1584
Build 1584: arc lint + arc unit

Event Timeline

aklapper created this revision.Sat, Sep 7, 04:36

Owners added a reviewer: O1: Blessed Committers.Sat, Sep 7, 04:36

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptSat, Sep 7, 04:36

aklapper requested review of this revision.Sat, Sep 7, 04:36

Harbormaster completed remote builds in B1584: Diff 2460.Sat, Sep 7, 04:36

aklapper retitled this revision from Update unit test link-alternate to Fix unit test failure in link-alternate.Sat, Sep 7, 07:35

Nice catch, thanks! Sorry for this regression in unit tests

This revision is now accepted and ready to land.Sat, Sep 7, 08:08

Closed by commit rPf25c531e5685: Fix unit test failure in link-alternate. · Explain WhySat, Sep 7, 09:00

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rPf25c531e5685: Fix unit test failure in link-alternate.

Revision Contents
Changeset List

Path

Size

Packages

src/

infrastructure/

markup/

remarkup/

__tests__/

remarkup/

link-alternate.txt

2 lines

O1: Blessed Committers

Diff 2460

View Options

src/infrastructure/markup/remarkup/tests/remarkup/link-alternate.txt

Fix unit test failure in link-alternateClosedPublicActions