yetanalytics / lrs

Protocols, specifications, and logic for building an xAPI Learning Record Store (LRS) in Clojure(Script).
https://www.yetanalytics.com/lrs
Apache License 2.0
4 stars 1 forks source link

LRS-41 allow sync async or both impls #43

Closed milt closed 3 years ago

milt commented 3 years ago

LRS-41 We've always had the memory impl automatically implement both sync and async LRS protos. It has ended up covering up bugs in the core lib, as we've only tested with the async (preferred) methods up til now.

This PR allows either or both impls to be active, and adds improved conf. tests for both modes (in clj anyways)

Extra Bonus: CI now runs conformance against cljs lrs too!