Merge pull request #106 from bzz/test/empty-content

Add failing test for empty file content
src-d · Sep 22, 2017 · 3d63a63 · 3d63a63
2 parents a6faaaf + 9a36e8f
commit 3d63a63
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/java/src/main/java/tech/sourced/enry/GoUtils.java b/java/src/main/java/tech/sourced/enry/GoUtils.java
@@ -58,7 +58,7 @@ static String[] toJavaStringArray(GoSlice slice) {
     static GoSlice.ByValue toGoByteSlice(byte[] bytes) {
         int length = 0;
         Pointer ptr = null;
-        if (bytes != null) {
+        if (bytes != null && bytes.length > 0) {
             length = bytes.length;
             ptr = ptrFromBytes(bytes);
         }

diff --git a/java/src/test/java/tech/sourced/enry/EnryTest.java b/java/src/test/java/tech/sourced/enry/EnryTest.java
@@ -23,6 +23,12 @@ public void getLanguageWithNullContent() {
         assertEquals("Python", Enry.getLanguage("foo.py",  null));
     }
 
+    @Test
+    public void getLanguageWithEmptyContent() {
+        assertEquals("Go", Enry.getLanguage("baz.go",  "".getBytes()));
+        assertEquals("Go", Enry.getLanguage("baz.go",  null));
+    }
+
     @Test
     public void getLanguageWithNullFilename() {
         byte[] content = "#!/usr/bin/env python".getBytes();