Page MenuHomePhabricator

Remove two problematic XML linter unit tests
ClosedPublic

Authored by epriestley on Aug 25 2015, 3:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 6:39 PM
Unknown Object (File)
Fri, Apr 19, 11:33 PM
Unknown Object (File)
Thu, Apr 18, 9:26 AM
Unknown Object (File)
Thu, Apr 11, 10:41 AM
Unknown Object (File)
Thu, Mar 28, 5:42 PM
Unknown Object (File)
Mar 14 2024, 4:05 AM
Unknown Object (File)
Mar 8 2024, 1:23 PM
Unknown Object (File)
Feb 3 2024, 6:25 AM
Subscribers
None

Details

Summary

Fixes T7215. See D13991. These test cases have been failing intermittently for a while.

I think the XML stuff (which we don't control) changed where it raises these warnings: an old version raised them at the end of the attribute, while the new version raises them at the beginning of the attribute. Not totally 100% sure about this since installing multiple versions is fairly inconvenient, but as far as I know both versions raise the warnings, just at different character offsets.

We could do various things to fix these tests (allow the warning to raise at any character, skip the tests based on versions, etc) but I think it's easier to just remove the tests. They don't seem valuable.

Test Plan

Tests failed prior to change; now pass.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove two problematic XML linter unit tests.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, joshuaspence.

arc diff didn't trigger the relevant tests for this, which I expected it to. I ran with arc unit --everything, but I'll see if I can figure out why the tests weren't auto-included...

This revision is now accepted and ready to land.Aug 25 2015, 3:52 AM
This revision was automatically updated to reflect the committed changes.