Merge pull request #37 from nubank/nodely/fix-get-contenxt-to-accept-nil

[FIX] Adapting inference context operation to accept nil cases

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 1.19.1 / 2024-08-28
+- Fix Contextual protocol to accept nil context cases
+
 ## 1.19.0 / 2024-08-15
 - Remove applicative engine dependency from funcool/cats
 

project.clj

@@ -1,4 +1,4 @@
-(defproject dev.nu/nodely "1.19.0"
+(defproject dev.nu/nodely "1.19.1"
   :description "Decoupling data fetching from data dependency declaration"
   :url "https://github.com/nubank/nodely"
   :license {:name "MIT"}

src/nodely/engine/applicative/core.clj

@@ -22,6 +22,10 @@
   java.lang.Object
   (-get-context [_] nil))
 
+(extend-protocol p/Contextual
+  nil
+  (-get-context [_] nil))
+
 (defn infer
   "Given an optional value infer its context. If context is already set, it
   is returned as is without any inference operation."

test/nodely/fixtures.clj

@@ -22,8 +22,10 @@
 
 (defn leaf-gen
   [env]
-  (gen/let [inputs (subset (keys env))]
-    (data/leaf inputs identity)))
+  (gen/let [inputs (subset (keys env))
+            f (gen/frequency [[1 (gen/return (constantly nil))]
+                              [9 (gen/return identity)]])]
+    (data/leaf inputs f)))
 
 (defn scalar-gen
   [env]