]> source.dussan.org Git - gitblit.git/commitdiff
[findbugs] Reorder asserts to avoid potential NPE in JGitUtilsTest
authorJames Moger <james.moger@gitblit.com>
Fri, 18 Apr 2014 02:51:54 +0000 (22:51 -0400)
committerJames Moger <james.moger@gitblit.com>
Fri, 18 Apr 2014 03:08:07 +0000 (23:08 -0400)
src/test/java/com/gitblit/tests/JGitUtilsTest.java

index 05bea915010e3fa6f137d69178a723b8702cd1a3..68cb4f86e2f321c1d42ceb4de24f6139a2532ac8 100644 (file)
@@ -420,6 +420,11 @@ public class JGitUtilsTest extends GitblitUnitTest {
                String contentA = JGitUtils.getStringContent(repository, (RevTree) null, "java.java");\r
                RevCommit commit = JGitUtils.getCommit(repository, Constants.HEAD);\r
                String contentB = JGitUtils.getStringContent(repository, commit.getTree(), "java.java");\r
+\r
+               assertTrue("ContentA is null!", contentA != null && contentA.length() > 0);\r
+               assertTrue("ContentB is null!", contentB != null && contentB.length() > 0);\r
+               assertTrue(contentA.equals(contentB));\r
+\r
                String contentC = JGitUtils.getStringContent(repository, commit.getTree(), "missing.txt");\r
 \r
                // manually construct a blob, calculate the hash, lookup the hash in git\r
@@ -429,9 +434,6 @@ public class JGitUtilsTest extends GitblitUnitTest {
                String sha1 = StringUtils.getSHA1(sb.toString());\r
                String contentD = JGitUtils.getStringContent(repository, sha1);\r
                repository.close();\r
-               assertTrue("ContentA is null!", contentA != null && contentA.length() > 0);\r
-               assertTrue("ContentB is null!", contentB != null && contentB.length() > 0);\r
-               assertTrue(contentA.equals(contentB));\r
                assertNull(contentC);\r
                assertTrue(contentA.equals(contentD));\r
        }\r