Allow empty lrs-statements anyways

yetanalytics · Aug 27, 2024 · 63614c7 · 63614c7
1 parent 6f01e8d
commit 63614c7
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/src/xapi_schema/spec.cljc b/src/xapi_schema/spec.cljc
@@ -1326,13 +1326,8 @@
                    (some-> s :statement/object :statement-ref/objectType)
                    true)))))
 
-;; Statement batches in general, e.g. when being inserted into an LRS, should
-;; allow empty arrays.
-
-;; Statement batches being retrieved from an LRS should always be non-emtpy.
-
 (s/def ::statements
   (s/coll-of ::statement :into []))
 
 (s/def ::lrs-statements
-  (s/coll-of ::lrs-statement :into [] :min-count 1))
+  (s/coll-of ::lrs-statement :into []))
diff --git a/test/xapi_schema/spec_test.cljc b/test/xapi_schema/spec_test.cljc
@@ -636,5 +636,4 @@
   (testing "LRS retrieval statement batch"
     (should-satisfy+ ::xs/lrs-statements
                      [d/statement] ; This statement has ID and other required fields
-                     :bad
                      [])))