Browse Source

fix(spec): fix spec (#764)

* fix(spec): fix spec

* fix(spec): only use spec in dev
Michael Wong 5 years ago
parent
commit
cc8987fa67
2 changed files with 8 additions and 7 deletions
  1. 1 1
      src/main/frontend/handler/repo.cljs
  2. 7 6
      src/main/frontend/spec.cljs

+ 1 - 1
src/main/frontend/handler/repo.cljs

@@ -571,7 +571,7 @@
 
 (defn rebuild-index!
   [{:keys [id url] :as repo}]
-  (spec/validate :me/repos+ repo)
+  (spec/validate :repos/repo repo)
   (db/remove-conn! url)
   (db/clear-query-state!)
   (-> (p/let [_ (db/remove-db! url)

+ 7 - 6
src/main/frontend/spec.cljs

@@ -12,12 +12,13 @@
 (defn validate
   "This function won't crash the current thread, just log error."
   [spec value]
-  (if (s/explain-data spec value)
-    (let [error-message (expound/expound-str spec value)
-          ex (ex-info "Error in validate" nil)]
-      (log/error :exception ex :spec/validate-failed error-message)
-      false)
-    true))
+  (when config/dev?
+    (if (s/explain-data spec value)
+     (let [error-message (expound/expound-str spec value)
+           ex (ex-info "Error in validate" nil)]
+       (log/error :exception ex :spec/validate-failed error-message)
+       false)
+     true)))
 
 ;; repo