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

[WIP] Allow for selecting test case by line number #145

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blueyed
Copy link
Collaborator

@blueyed blueyed commented Dec 21, 2017

This parses the line number from foo.vader:123 and executes only the
case ("Do" or "Execute") around this line number.

Any previous Given/Before/After/Then is copied (like s:parse_vader would
do it).

TODO:

This parses the line number from `foo.vader:123` and executes only the
case ("Do" or "Execute") around this line number.

Any previous Given/Before/After/Then is copied (like s:parse_vader would
do it).
@blueyed blueyed requested a review from junegunn December 22, 2017 01:38
@junegunn
Copy link
Owner

junegunn commented Dec 22, 2017

Thanks. vader#run is already too long, and I'd like to trim it down a bit before adding more lines to it. Can you rebase your work onto the following commit? (I didn't merge it to master branch)

2581d00

Any previous Given/Before/After/Then is copied (like s:parse_vader would do it).

A Then block is supposed come after Execute or Do, do we need it here?

@blueyed
Copy link
Collaborator Author

blueyed commented Dec 22, 2017

Ok, will do it.
I've noticed that it does not work as-is anyway, since it uses the fpos additions from #107.

@blueyed blueyed changed the title Allow for selecting test case by line number [WIP] Allow for selecting test case by line number Dec 22, 2017
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

Successfully merging this pull request may close these issues.

2 participants