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
F13379070: D8016.diff
Sat, Jun 29, 5:36 AM
F13328592: D8016.id18135.diff
Sun, Jun 16, 11:50 AM
F13328591: D8016.id18134.diff
Sun, Jun 16, 11:50 AM
F13328590: D8016.id18133.diff
Sun, Jun 16, 11:50 AM
F13328473: D8016.id.diff
Sun, Jun 16, 10:10 AM
F13328425: D8016.diff
Sun, Jun 16, 9:09 AM
F13327413: D8016.diff
Sat, Jun 15, 7:03 PM
F13295138: D8016.diff
Wed, Jun 5, 10:49 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