Skip to content
Permalink
Browse files

[Fix JENKINS-29731] Bad diff link from AssemblaWeb browser

  • Loading branch information...
MarkEWaite committed Aug 2, 2015
1 parent 5c7381c commit af01132e56f767f7ee944a3b64b17aa3a0c7f198
@@ -48,7 +48,7 @@ private QueryBuilder param(URL url) {
@Override
public URL getChangeSetLink(GitChangeSet changeSet) throws IOException {
URL url = getUrl();
return new URL(url, url.getPath() + "commit/" + changeSet.getId());
return new URL(url, url.getPath() + "commits/" + changeSet.getId());
}

/**
@@ -38,7 +38,7 @@ public static Collection permuteAuthorName() {
@Test
public void testGetChangeSetLink() throws Exception {
URL result = (new AssemblaWeb(repoUrl)).getChangeSetLink(sample.changeSet);
assertEquals(new URL(repoUrl + "commit/" + sample.id), result);
assertEquals(new URL(repoUrl + "commits/" + sample.id), result);
}

@Test
@@ -47,7 +47,7 @@ public void testGetDiffLink() throws Exception {
for (GitChangeSet.Path path : sample.changeSet.getPaths()) {
URL diffLink = assemblaWeb.getDiffLink(path);
EditType editType = path.getEditType();
URL expectedDiffLink = new URL(repoUrl + "commit/" + sample.id);
URL expectedDiffLink = new URL(repoUrl + "commits/" + sample.id);
String msg = "Wrong link for path: " + path.getPath() + ", edit type: " + editType.getName();
assertEquals(msg, expectedDiffLink, diffLink);
}

0 comments on commit af01132

Please sign in to comment.
You can’t perform that action at this time.