Babashka test runner

[borkdude]

The test runner is currently using a dev version of babashka which implements some previous special-forms as macros.

[borkdude]

I skipped some test namespaces due to the incompatibility with missionary:

Message:  Unable to resolve classname: org.reactivestreams.Publisher
Location: missionary/impl.clj:2:3

[borkdude]

I'm sorry that I made some unnecessary whitespace changes as a result of activating the formatter in my editor. I can revert those if you want.

[borkdude]

@dustingetz I released a new version of babashka and used that in the tests. PR should be good to go from my side.

[pmbauer]

Locally, I've used a version of RCF and babashka with this pull applied. Any blockers to getting it merged?

[dustingetz]

I attempted to merge today, there is a conflict with this commit: 2d1ef6f

@borkdude can you quickly spot the issue? I will take a closer look later, gtg

I f'ed up the PR, sorry, i guess i force pushed the wrong thing. Master currently has babashka support merged and pushed, but fails CI in babashka due to the conflict, which is a correct error (I bisected)

[dustingetz]

rcf git:(babashka-dustin4) ./ci/run_tests_bb.sh
Running Babashka tests
----- Error --------------------------------------------------------------------
Type:     clojure.lang.ExceptionInfo
Message:  Method getName on class sci.lang.Namespace not allowed!
Data:     {:type :sci/error, :line 20, :column 1, :file "/Users/dustin/src/hf/rcf/test/hyperfiddle/rcf_test.cljc"}
Location: /Users/dustin/src/hf/rcf/test/hyperfiddle/rcf_test.cljc:20:1
Phase:    macroexpand

----- Context ------------------------------------------------------------------
16:                     :form4 (quote (inc 1)) ; interpreted as if evaluated after code emission  
17:                     :form5 (quote (quote (inc 1))) ; escaping interpretation
18:                     )))
19: 
20: #?(:clj
    ^--- Method getName on class sci.lang.Namespace not allowed!
21:    (tests
22:      "Custom var meta on def symbol are interpreted as if they were evaluated after emission."
23:      ;; CLJ only, no vars in cljs.
24:      (my-def ^{:form1 (quote (inc 1))
25:                :form2 (inc 1)}        ; read and evaluated as usual

----- Stack trace --------------------------------------------------------------
hyperfiddle.rcf/tests                          - /Users/dustin/src/hf/rcf/src/hyperfiddle/rcf.cljc:106:1
hyperfiddle.rcf-test/apply                     - /Users/dustin/src/hf/rcf/test/hyperfiddle/rcf_test.cljc:20:1
hyperfiddle.rcf-test                           - /Users/dustin/src/hf/rcf/test/hyperfiddle/rcf_test.cljc:20:1
clojure.core/apply                             - <built-in>
runner                                         - /Users/dustin/src/hf/rcf/bb/runner.clj:15:5
exec-d0f5b07e-cbd6-42af-82a4-361e5b511f13/exec - <expr>:22:1
exec-d0f5b07e-cbd6-42af-82a4-361e5b511f13      - <expr>:4:1
user-a5406db3-03a4-4cbd-80cd-08ebb1b8978d      - <expr>:26:1

[borkdude]

Don't worry, I'll take a look :)

[dustingetz]

Confirmed electric test suite passes with the babashka support applied, so this is all clear as soon as we fix the conflict. I took a closer look at the conflict and it is nontrivial.

[borkdude]

@dustingetz Fixed in #79

[dustingetz]

thank you!

[pmbauer]

:chefskiss:

On July 20, 2023, GitHub ***@***.***> wrote: thank you!

— Reply to this email directly, view it on GitHub <#71 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe- auth/AADOXTA2R4AZIXHGJ5V6P7TXRFZEHANCNFSM6AAAAAASSJLNJA>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>