Handle slash without anything before it

This avoids an NPE on input of "/".
ninjudd · Jan 3, 2014 · cb31512 · cb31512
1 parent e106888
commit cb31512
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/src/complete/core.clj b/src/complete/core.clj
@@ -95,12 +95,13 @@
 
 (defmethod potential-completions :scoped
   [prefix ns]
-  (let [scope (symbol (first (.split prefix "/")))]
-    (map #(str scope "/" %)
-         (if-let [class (resolve-class scope)]
-           (static-members class)
-           (when-let [ns (or (find-ns scope) (scope (ns-aliases ns)))]
-             (ns-public-vars ns))))))
+  (when-let [prefix-scope (first (.split prefix "/"))]
+    (let [scope (symbol prefix-scope)]
+      (map #(str scope "/" %)
+           (if-let [class (resolve-class scope)]
+             (static-members class)
+             (when-let [ns (or (find-ns scope) (scope (ns-aliases ns)))]
+               (ns-public-vars ns)))))))
 
 (defmethod potential-completions :class
   [prefix ns]

diff --git a/test/complete/core_test.clj b/test/complete/core_test.clj
@@ -20,6 +20,10 @@
   (is (= '("java.lang.System/out")
          (completions "java.lang.System/out")))
 
+  (is (= () (completions "fake-ns-here/")))
+
+  (is (= () (completions "/")))
+
   (is (some #{"String/valueOf"} (completions "String/")))
 
   (is (not (some #{"String/indexOf" ".indexOf"} (completions "String/")))))