@robert-stuttaford There is a more recent clj selenium library than clj-webdriver, I believe

https://github.com/tmarble/webica

@robert-stuttaford are there features in Selenium that you need that clj-webdriver doesn’t have?

we’re still using clj-webdriver FWIW

@anmonteiro webica abuses introspection to expose the most recent Selenium. At the moment it's a little tricky to expose the API docs (a la codox), but it allows you to use the latest

@robert-stuttaford On several projects I've worked on we've just stuck with cljs.test (or clojurescript.test as it was called back then IIRC), and ran code only tests. That does leave you open to UI bugs, but I have yet to find a solution that that isn't super fragile. And doing code level tests forces you to decouple DOM elements from your business logic, which is a good idea anyways.

the Clojurescript compiler throws me:

Unsupported binding form: 1 at line 43, column 5 in file test/cljs/budb/cljs/firebase_test.cljs

(macroexpand '(test-firebase S 1))
=> (budb.test.helpers/test-async S 
       (budb.test.helpers/test-within S 4000 
         (superv.async/go-try S (do 1))))

@lsenta that error is from destructuring something, so if you have something like (let [:foo 1] ...)

maybe budb.test.helpers/test-within

or budb.test.helpers/test-async if any of these are macros

you are emitting (let [1 :foo] ...) somewhere

which is what Unsupported binding form: 1 is saying

or (fn [1] ...) or any other place you could have bindings

Hum, test-async and test-within are functions, and go-try is a superv.async macro

does just this work (superv.async/go-try S (do 1))?

Yea, even the macro expands work

maybe something in that macro doesn't like the 1?

(macroexpand '(test-firebase S 1))
=> (budb.test.helpers/test-async S 
       (budb.test.helpers/test-within S 4000 
         (superv.async/go-try S (do 1))))

If I copy the result (or write it myself without the namespaces)

it compiles

(test-firebase S :foo) does this change the error to Unsupported binding form: :foo?

It does

so the error is in test-firebase macro

It's so basic, could you let me know if you see something weird?

(defmacro test-firebase
     [supervisor-sym & body]
     `(test-async ~supervisor-sym
        (test-within ~supervisor-sym 4000
          (go-try ~supervisor-sym
            (do ~@body)))))

@anmonteiro Is there a problem, loading system libraries from nodejs? I recompiled the closure compiler and clojurescript to use the latest version. Now it throws multiple errors like WARNING: JSC_JS_MODULE_LOAD_WARNING. Failed to load module "util" at /Users/danielgrosse/dev/clojure/npm-compile-test/node_modules/json/lib/json.js line 14 :

thanks @john @anmonteiro @tbaldridge — all good points!

@lsenta no idea, reduce that until the error goes away. probably the go-try since that is where the 1 ends up

@thheller thanks for the help, I still don't see what's wrong I'll work around it for now.

@lsenta yeah it a weird one, be aware that line/column information may be off if the form is actually inside another macro

so maybe you are looking at the wrong thing, probably not though since the :foo test worked

@piotr2b not possible in CLJS

@thheller thx for your answer. I was feeling the same. How could we prove it? Perhaps by saying there is no such thing as Var in cljs?

pretty sure it says that somewhere

static vars are ok and possible, dynamic vars are not

@zerusski don't extend protocols onto native types, they are mutable collections. you will get yourself into trouble 🙂

but the interwebs told me I should

array-seq is the way to go and as you see it works 😉

although still mutable so be careful

Could you point me to a longwinded explanation as to why please?

I'm not too immersed in cljs ecosystem and seems like there's much dated information on the net

Is there perhaps a "best practices" and "dos and don'ts" for cljs?

no idea where that is written down, if at all. it is my personal recommendation based on experience but I'm sure others share my opinion

but you are right, that should probably be written down somewhere. maybe on http://clojurescript.org

I understand js and browser could be a moving target, but why doesn't it work in this particular case?

I'm not mutating anything, nor have I reloaded the page

it may depend on your browser ... some browsers in the past didn't let you extend native objects

the only thing that comes to mind is that this isn't "the droid I'm looking for". Different type that serialises to the same name

I think that was IE though

you might not be working with a HTMLCollection as well. native types do not strictly follow the JS inheritance chain sometimes

native types are scary

yeah, that's my suspicion here, too. Thank you for answers.

I'm happy to submit an issue if you feel I should

haha, @thheller I think you are absolutely right. To test I put some divs on the page, grabbed em with .getElementsByClass. And extend-type worked beautifully. So I suspect the stuff I was getting wasn't HTMLCollection but reported as being one. I guess not worth reporting that

that is my experience with protocols on native types as well .. sometimes they work ... sometimes they don't for reasons unknown 🙂

@zerusski iframes have their own js sandbox, so you'd need to extend-type from inside it

(= (type (.. (.-contentDocument frame) -body -children))
   (type (.. js/document -body -children)))
;false

hi, what blog post or talk would you recommend, if you could choose only one, to inspire a js & frontend developer to try out fp & ClojureScript. we're building a poc app with react native and this could be an opportunity to try out clojurescript. (not for me)

@sakalli haven't seen this one personally but this one gets recommended a lot: https://www.youtube.com/watch?v=gsffg5xxFQI&list=PLZdCLR02grLrl5ie970A24kvti21hGiOf&index=8

I'd also show them the FigWheel talk at ClojureWest https://www.youtube.com/watch?v=j-kj2qwJa_E

@zerusski actually now that I poke around, you can extend-type iframe types like this:

(def frame (-> js/document (.getElementsByTagName "iframe") (aget 0)))

(extend-type frame.contentWindow.HTMLCollection
  ISeqable
  (-seq [o] (array-seq o 0)))

@selfsame this is very good to know. Thank you very much for investigating!

@danielgrosse are you compiling with :target :nodejs?

Oh that's just a warning anyway. It'll work at runtime since it's internal

I have a clojurescript macro expansion question for anyone knowledgable here: in a nutshell, the issue I can't figure out is that a simple interop form like (.foo bar baz) correctly expands to bar.foo(baz); but expanding that same form within a macro seems to be adding a phantom .call(null) and I dont understand why

e.g. it turns into bar.foo(baz).call(null); inside the macro usage

any help appreciated

do you have an example?

@jr its part of opensource, so let me push my working tree and show you

@jr (keep in mind this is a push from mid-hack), here is the basic function I am trying to macroify

https://github.com/ghaskins/fabric-chaintool/blob/macro-question/examples/example02/client/cljs/src/example02/fabric_sdk/core.cljs#L41

essentially, hfc.newDefaultKeyStore() uses ES6 promises, and I am trying to wrap them in promesa promises

I have this macro: https://github.com/ghaskins/fabric-chaintool/blob/macro-question/examples/example02/client/cljs/src/example02/fabric_sdk/macros.clj

with the idea being I want to do something like (m/pwrap (.newDefaultKeyValStore hfc #js {:path path}))

with the idea that the form passed in is expanded in the same way as when I use it natively

hmmm, i think I just rubberduck'd it

😉

I think the issue was the [& expr] on the macro, when I wanted [expr]

forest for the trees...

I’m looking at cljs test runners, are there any good options besides https://github.com/bensu/doo ?

@jfntn I’ve been using Doo successfully in all my CLJS projects

if you’re using Boot, this is what you want (uses Doo too): https://github.com/crisptrutski/boot-cljs-test/

@anmonteiro thanks that looks good

@anmonteiro do you any cljs-specifc reasons for using boot over lein besides personal preference?

mostly personal preference

in some cases I need custom build pipelines

(e.g. Lumo)