Fork me on GitHub
#cursive
<
2020-12-17
>
steveb8n07:12:32

Q: I have a couple of questions about the testing integrations in Cursive…

steveb8n07:12:08

1/ how does the new test runner feature work within the REPL? Or does it not?

steveb8n07:12:51

I use “Run test in REPL” all the time. I can’t see how the new features add new capabilities

steveb8n07:12:20

2/ is anyone using Greenlight integration tests with a Cursive integration?

AJ Jaro16:12:35

@U0510KXTU I have taken a look at the test runner but don’t use it as part of my workflow. Based on what I saw, this is independent of the REPL. No, to part 2. I don’t use Greenlight

steveb8n21:12:49

@UGMAVSMUM thanks for the reply. That’s what I suspected

cfleming21:12:29

Right, the new test integration is independent of the REPL, at least for now. I would like to add a “Run tests in open REPL” option, but that’s tricky for intellij reasons which I haven’t had time to look at yet.

cfleming21:12:45

I don’t know of anyone using Greenlight, no, sorry.

steveb8n08:12:49

Thanks for the clarification

wilkerlucio22:12:04

@cfleming hello, one idea I just had here for the deps window, would be nice if it could make separated groups for project vs global aliases, so its easier to understand what alias come from a project, makes sense?

cfleming23:12:17

Yes, definitely.

kenny22:12:58

Refactor > rename is broken in 1.10.0-2020.3. Take this code.

(s/def ::foo int?)
Right click on the keyword. Go to Refactor > Rename. This pops open the modal seen in screenshot 1. Changing "foo" to "foo2" results in a red message saying "'foo2' is not a valid identifier". If I add a double colon in from of "foo2" and click Refactor, the modal closes and nothing happens.

kenny22:12:15

Opened an issue. https://github.com/cursive-ide/cursive/issues/2473 This is a big issue for my workflow 😬

kenny22:12:37

It's kinda like it's now treating s/def's as symbols.

cfleming23:12:25

Ugh, this is probably a side effect of https://github.com/cursive-ide/cursive/issues/2457. There were a bunch of platform changes affecting navigation, I’ll get this fixed ASAP.

cfleming23:12:35

Sorry for the breakage, a lot of this stuff is really hard to test in automated tests since it’s quite tied to the UI.

kenny23:12:22

Yeah, that's crossed my mind before. I have no idea how you do it 🙂

cfleming23:12:26

There are actually some facilities for GUI testing now, but they’re still a pain to use. I actually use them for generating the screenshots in the documentation, so if I document something then as a side effect it gets tested when I regenerate the doc 🙂

kenny21:12:12

@cfleming Potentially related to this is usually when your cursor is on a keyword, it will highlight all keywords that are the same. That is not happening in the latest version.