Page MenuHomePhabricator
Differential D8016

Add basic test coverage for the Subversion parser
ClosedPublic
Actions

Authored by epriestley on Jan 20 2014, 9:32 PM.
Tags
None
Referenced Files
F19862637: D8016.id18133.diff
Mar 14 2026, 10:41 AM
F19862253: D8016.id18134.diff
Mar 14 2026, 10:06 AM
F19775051: D8016.id18133.diff
Feb 21 2026, 8:17 PM
F19775041: D8016.id18133.diff
Feb 21 2026, 8:17 PM
F19684809: D8016.id.diff
Feb 9 2026, 6:12 AM
F19623858: D8016.id18133.diff
Feb 4 2026, 5:12 PM
F19347111: D8016.id18133.diff
Dec 26 2025, 1:43 AM
F18904981: D8016.id.diff
Nov 8 2025, 12:12 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4327: Problems trying to import project
Commits
Restricted Diffusion Commit
rPdc74da0abed3: Add basic test coverage for the Subversion parser
Summary

Ref T4327. Adds the same basic battery of tests to the SVN parser as the other parsers have.

___________________________ < THIS IS AN AMAZING DIFF > --------------------------- \ ^__^ \ (oo)\_______ (__)\ )\/\ ||----w | || ||
Test Plan

Ran unit tests against SVN change parsing.

bwahaha

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 20 2014, 9:33 PM
  • Make the SVN TODO about symlinks slightly more accurate.
btrahan accepted this revision.Jan 20 2014, 9:41 PM

AMAZEBALLS

epriestley closed this revision.Jan 21 2014, 1:25 AM

Closed by commit rPdc74da0abed3 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
engine/
__tests__/
data/
worker/
__tests__/
198 lines
commitchangeparser/
85 lines

Diff 18135

View Options

src/applications/repository/engine/__tests__/data/CHC.svn.tgz

Loading...
View Options

src/applications/repository/worker/__tests__/PhabricatorChangeParserTestCase.php

Loading...
View Options

src/applications/repository/worker/commitchangeparser/PhabricatorRepositorySvnCommitChangeParserWorker.php

Loading...
Phabricator · Built by Phacility