Fork me on GitHub
#cursive
<
2020-12-04
>
salam03:12:31

i just tried the new test runner and it works like a charm! this is huge, especially for folks who are used to the traditional tdd approach in intellij idea. i just found a minor bug as shown in the screenshot attached. anyway, thanks a lot!

cfleming03:12:48

Thanks! I’m glad it’s useful, I already use it all the time. With that screenshot, did the error message persist even after entering a display name, or had that been defaulted when you opened it?

salam03:12:47

yes and yes: when i created the run configuration, the error message was already there even though the Display Name was already pre-populated with the default value that was the same as that for Name (“Unnamed”). changing the Display Name didn’t make the error message go away.

cfleming04:12:08

Hmm, strange, thanks - I’ll try to reproduce that.

cfleming04:12:19

I can’t reproduce this. I’ll see if anyone else has the same problem and and see if I can see any pattern in who’s getting it.

salam05:12:34

that’s strange. i’ll go back and try it again. this might just be me nitpicking but the other thing that’s been bugging me about this run configuration window is the way some options are indented. i feel like the labels “Host”, “Port,” and “Context Module” can use more indentation to make it clear that they are the children of the “Connect to server” radio button (same amount of indentation that the “Project” label has under the radio button “Use port from nREPL file”.)

cfleming05:12:25

Good idea, I’ll fix that.

kenny15:12:55

I see a new play button next to my deftests. If I click it I see several options: Run test, debug test, run with cpu profiler, ... If I click run with cpu profiler, I get a red popup in the corner pointing to the Run tab saying "Error running 'Test user/my-test': Internal error: Don't know how to patch selected configuration" Is running with cpu profiler not currently supported?

cfleming19:12:55

I’m not sure about that actually, I’ll check it. I’m hoping to add coverage support, but it’s more complicated than I hoped because of how Clojure works.

cfleming01:12:47

Actually @UDCGPTV9R sorry, I can repro your error, I was trying it with a local REPL not a remote one. I’ll fix that.

👍 3
cfleming02:12:53

Also fixed the indentation, and the marking elements disabled is now more consistent and obvious too.

🎉 3
❤️ 3
salam05:12:01

i just checked out the new EAP build and it looks great! now that the tabs have a close button (X) in them, is there any difference between that button and the one in the toolbar of the REPL tool window? the latter seems redundant now?

cfleming07:12:57

Possibly, I think mostly it makes that consistent with the run config action toolbar, although you’re right that it’s probably not needed.

ikitommi19:12:53

copy to scratch file 👍 👍 👍

AJ Jaro19:12:14

Thanks @U0567Q30W for fixing the namespaced keywords!! Huge!!

cfleming19:12:07

Thanks everyone! @UGMAVSMUM In recent IntelliJ versions that is becoming a complex and subtle area of the code which is also difficult to test automatically - let me know if you see any more weirdness.

AJ Jaro19:12:17

For sure I definitely will do. I know how much I appreciate this tool so anyway I can help is great!

kenny22:12:59

@U0567Q30W Not sure if this is the display name bug you mentioned. I updated to 1.10.0-eap2-2020.3 and all my run configs have this red X next to them.

cfleming02:12:07

@U083D6HK9 Did you get that with eap1?

cfleming02:12:17

So that’s a different problem that I talked about in the release notes - the migration which fixes the display name field for existing run configs doesn’t fix IDE-local ones, only project-shared ones.

cfleming02:12:49

If you choose edit configurations and open those configs and then apply, that should fix it - the defaulting will be done in the editor.

kenny02:12:46

What’s the difference between IDE-local and project-shared?

cfleming02:12:25

If you look in the REPL run config, there’s a checkbox in the top right - “Store as project file”. If that’s checked, then the run config will be stored as a separate file in .idea and can be shared in VCS etc.

kenny02:12:58

Ah, got it. Will try this tomorrow. Thanks.

roklenarcic12:12:59

What’s the trick for getting to connect to remote Babashka REPL? The connect process just waits

pez16:12:23

Have you started babashka with an nrepl server?

borkdude16:12:03

if that info is out of date, please let me know