Merge pull request #170 from KostyaSha/coverity

Improvements
hub4j · Apr 13, 2015 · 7ff9734 · 7ff9734
2 parents 68dda3a + 9ebc9c0
commit 7ff9734
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/main/java/org/kohsuke/github/GHEventPayload.java b/src/main/java/org/kohsuke/github/GHEventPayload.java
@@ -165,6 +165,7 @@ public GHRepository getRepository() {
 
         @Override
         void wrapUp(GitHub root) {
+            super.wrapUp(root);
             if (repository!=null)
                 repository.wrap(root);
         }

diff --git a/src/test/java/org/kohsuke/github/AbstractGitHubApiTestBase.java b/src/test/java/org/kohsuke/github/AbstractGitHubApiTestBase.java
@@ -19,9 +19,9 @@ public void setUp() throws Exception {
         if (f.exists()) {
             // use the non-standard credential preferentially, so that developers of this library do not have
             // to clutter their event stream.
-            gitHub = GitHubBuilder.fromPropertyFile(f.getPath()).build();
+            gitHub = GitHubBuilder.fromPropertyFile(f.getPath()).withRateLimitHandler(RateLimitHandler.FAIL).build();
         } else {
-            gitHub = GitHub.connect();
+            gitHub = GitHubBuilder.fromCredentials().withRateLimitHandler(RateLimitHandler.FAIL).build();
         }
     }
 

diff --git a/src/test/java/org/kohsuke/github/AppTest.java b/src/test/java/org/kohsuke/github/AppTest.java
@@ -673,8 +673,8 @@ public void testReadme() throws IOException {
     public void testTrees() throws IOException {
     	GHTree masterTree = gitHub.getRepository("kohsuke/github-api").getTree("master");
     	boolean foundReadme = false;
-    	for(GHTreeEntry e : masterTree.getTree()){ 
-    		if("readme".equalsIgnoreCase(e.getPath().replaceAll(".md", ""))){
+    	for(GHTreeEntry e : masterTree.getTree()){
+    		if("readme".equalsIgnoreCase(e.getPath().replaceAll("\\.md", ""))){
     			foundReadme = true;
     			break;
     		}