Merge pull request #14 from ccedric/master

Fix display Gatling Reports, close gatling/gatling#3154

Author: slandelle

src/main/java/io/gatling/jenkins/GatlingPublisher.java

@@ -65,16 +65,22 @@ public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListen
     }
 
     logger.println("Archiving Gatling reports...");
-    List<BuildSimulation> sims = saveFullReports(build.getWorkspace(), build.getRootDir());
-    if (sims.isEmpty()) {
-      logger.println("No newer Gatling reports to archive.");
-      return true;
-    }
+    FilePath workspace = build.getWorkspace();
+    if (workspace != null) {
+      List<BuildSimulation> sims = saveFullReports(workspace, build.getRootDir());
+      if (sims.isEmpty()) {
+        logger.println("No newer Gatling reports to archive.");
+        return true;
+      }
 
-    GatlingBuildAction action = new GatlingBuildAction(build, sims);
+      GatlingBuildAction action = new GatlingBuildAction(build, sims);
 
-    build.addAction(action);
-    return true;
+      build.addAction(action);
+      return true;
+    } else {
+      logger.println("Failed to access workspace, it may be on a non-connected slave.");
+      return false;
+    }
   }
 
   @Override

src/main/resources/io/gatling/jenkins/GatlingBuildAction/report.jelly

@@ -4,16 +4,16 @@
 	<l:layout title="Build #${it.build.number} : ${simName}">
 		<l:breadcrumb title="${simName}" href="."/>
 		<l:side-panel>
-			<st:include it="${it.build}" page="sidepanel.jelly"/>
+			<st:include it="${it.run}" page="sidepanel.jelly"/>
 		</l:side-panel>
 		<l:main-panel>
 			<script type="text/javascript">
 				function resizeReportFrame() {
-					var frame = document.getElementById("reportFrame")
-					frame.height = frame.contentWindow.document.body.scrollHeight + "px"
+					var html = document.documentElement;
+                    document.getElementById("reportFrame").height = Math.max(html.clientHeight, html.scrollHeight, html.offsetHeight) - document.getElementById("page-head").scrollHeight - document.getElementById("menuSelector").scrollHeight - document.getElementById("newTab").scrollHeight;
 				}
 			</script>
-			<h3>
+			<h3 id="newTab">
 				<a href="source" target="_blank">${%OpenNewPage}</a>
 			</h3>
 			<iframe id="reportFrame" src="source" width="100%" height="100%" frameborder="0"