Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IcedTestNs, IcedTestAll need manual require but IcedTestUnderCursor doesn't #292

Open
Cnly opened this issue Nov 28, 2020 · 1 comment
Open

Comments

@Cnly
Copy link
Contributor

Cnly commented Nov 28, 2020

Let's say I'm working on a deftest form. After editing, if I run IcedTestUnderCursor immediately, the latest test code will be executed.

However, if I instead run IcedTestNs or IcedTestAll, vim-iced will run the old test code loaded when I previously ran IcedRequire. In other words, I need to manually IcedRequire in order to make IcedTestNs and IcedTestAll use the latest code.

I'm not sure if this is a bug or a feature, but from my view it's an inconsistent behaviour. Anyway, thanks for the great tool :)

@liquidz
Copy link
Owner

liquidz commented Nov 28, 2020

@Cnly Thanks for your reporting!

IcedTestUnderCursor evaluates the form under the cursor to identify the var for testing, thus it leads testing the latest code.
On the other hand, the ns name is not evaluated at present because it can be extracted without evaluation.

However, it is certain that the behavior is inconsistent, so I'll consider how to deal with it. (There are also IcedTestRedo and IcedTestRerunLast)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants