Skip to content
This repository has been archived by the owner on Jun 4, 2022. It is now read-only.

[Fix #359] Workaround the transit-cljs map entry issue #360

Closed
wants to merge 1 commit into from

Conversation

arichiardi
Copy link
Collaborator

@arichiardi arichiardi force-pushed the workaround-359 branch 4 times, most recently from 2b43018 to bb461d1 Compare February 21, 2018 00:12
@arichiardi
Copy link
Collaborator Author

It looks like this has some side effect I was not able to investigate further. I see a very weird instrumentation failure that does not happen with lumo 1.8.0 vanilla:

Call to #'ep-cloud.event-store/query-events! did not conform to spec:
val: "order" fails at: [:args] predicate: (cat :domain :event/domain :entity :event/entity :opts :query-events/opts)
:cljs.spec.alpha/spec  #object[cljs.spec.alpha.t_cljs$spec$alpha21371]
:cljs.spec.alpha/value  "order"
:cljs.spec.alpha/args  "order"
:cljs.spec.alpha/failure  :instrument

Error: Call to #'ep-cloud.event-store/query-events! did not conform to spec:
val: "order" fails at: [:args] predicate: (cat :domain :event/domain :entity :event/entity :opts :query-events/opts)
:cljs.spec.alpha/spec  #object[cljs.spec.alpha.t_cljs$spec$alpha21371]
:cljs.spec.alpha/value  "order"
:cljs.spec.alpha/args  "order"
:cljs.spec.alpha/failure  :instrument

    at new cljs.core.ExceptionInfo (<embedded>:2053:47)
    at Function.cljs.core.ex_info.cljs$core$IFn$_invoke$arity$3 (<embedded>:2056:72)
    at Function.cljs.core.ex_info.cljs$core$IFn$_invoke$arity$2 (<embedded>:2055:449)
    at cljs/spec/test/alpha.js:131:25
    at Function.G__3670__delegate [as cljs$core$IFn$_invoke$arity$variadic] (cljs/spec/test/alpha.js:162:1)
    at Function.ep_cloud.event_store.query_events_BANG_.cljs$lang$applyTo (evalmachine.<anonymous>:54:48)
    at Function.cljs.core.apply.cljs$core$IFn$_invoke$arity$2 (<embedded>:878:190)
    at Function.G__3675__delegate (cljs/spec/test/alpha.js:210:28)
    at Function.G__3675 [as afn] (cljs/spec/test/alpha.js:224:26)
    at Function.e (<embedded>:594:8)

@arichiardi
Copy link
Collaborator Author

Superseded by #379.

@arichiardi arichiardi closed this Mar 24, 2018
@arichiardi arichiardi deleted the workaround-359 branch March 24, 2018 03:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant