Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,11 @@ public void testDashboard_statisticBox() {

@Test
public void testLinkageReports() {
Nodes reports = details.query("//p[@class='jar-linkage-report']");
// appengine-api-sdk, shown as first item in linkage errors, has these errors
assertThat(trimAndCollapseWhiteSpace(reports.get(0).getValue()))
.isEqualTo("4 target classes causing linkage errors referenced from 4 source classes.");

Nodes dependencyPaths = details.query("//p[@class='linkage-check-dependency-paths']");
Node dependencyPathMessageOnProblem = dependencyPaths.get(dependencyPaths.size() - 1);
Assert.assertEquals(
Expand Down Expand Up @@ -291,12 +296,6 @@ public void testComponent_linkageCheckResult_java8() throws IOException, Parsing
assertThat(trimAndCollapseWhiteSpace(reports.get(0).getValue()))
.isEqualTo("100 target classes causing linkage errors referenced from 540 source classes.");


Nodes artifactDetailsReports = details.query("//p[@class='jar-linkage-report']");
// appengine-api-sdk, shown as first item in linkage errors, has these errors
assertThat(trimAndCollapseWhiteSpace(artifactDetailsReports.get(0).getValue()))
.isEqualTo("4 target classes causing linkage errors referenced from 4 source classes.");

Nodes causes = document.query("//p[@class='jar-linkage-report-cause']");
assertWithMessage(
"google-http-client-appengine should show linkage errors for RpcStubDescriptor")
Expand All @@ -314,11 +313,6 @@ public void testComponent_linkageCheckResult_java11() throws IOException, Parsin
int javaMajorVersion = Integer.parseInt(javaVersion.split("\\.")[0]);
Assume.assumeTrue(javaMajorVersion >= 11);

Nodes artifactDetailsReports = details.query("//p[@class='jar-linkage-report']");
// appengine-api-sdk, shown as first item in linkage errors, has these errors
assertThat(trimAndCollapseWhiteSpace(artifactDetailsReports.get(0).getValue()))
.isEqualTo("5 target classes causing linkage errors referenced from 5 source classes.");

// The version used in libraries-bom 1.0.0. The google-http-client-appengine has been known to
// have linkage errors in its dependency appengine-api-1.0-sdk:1.9.71.
Document document =
Expand Down
2 changes: 1 addition & 1 deletion dependencies/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Linkage Checker is a tool that finds [linkage errors](
path. It scans the class files in the class path for references to other
classes and reports any reference that cannot be satisfied.

### User Documentation
#### User Documentation

Linkage Checker can be used from Maven or Gradle.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ public void testBom_java11()
LinkageCheckerMain.main(new String[] {"-b", "com.google.cloud:libraries-bom:1.0.0"});
fail("LinkageCheckerMain should throw LinkageCheckResultException upon errors");
} catch (LinkageCheckResultException expected) {
assertEquals("Found 758 linkage errors", expected.getMessage());
assertEquals("Found 756 linkage errors", expected.getMessage());
}

String output = readCapturedStdout();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,6 @@ assert !buildLog.text.contains("NullPointerException")

// 4 linkage errors are references to java.util.concurrent.Flow class, which does not exist in
// Java 8 runtime yet.
def expectedErrorCount = System.getProperty("java.version").startsWith("1.8.") ? 111 : 108
def expectedErrorCount = System.getProperty("java.version").startsWith("1.8.") ? 111 : 107

assert buildLog.text.contains("Linkage Checker rule found $expectedErrorCount errors:")