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

How to change default test root directory? #15

Open
ocsarpo opened this issue Jan 12, 2021 · 4 comments
Open

How to change default test root directory? #15

ocsarpo opened this issue Jan 12, 2021 · 4 comments
Assignees
Labels

Comments

@ocsarpo
Copy link

ocsarpo commented Jan 12, 2021

I'm using minitest for the test.
But rails-fast-nav's default test root direc is "spec"
I want to change that to "test"

But I don't know How to setting for this.

Is it impossible?

@jemmyw
Copy link
Owner

jemmyw commented Jan 13, 2021

nothing is impossible.

In any case, according to https://github.com/jemmyw/vscode-rails-fast-nav/blob/master/src/rails-workspace.ts#L195 it should default to "test" unless there is a "spec" directory. If you have both then it'll look in "spec". Do you have both for some reason?

@jemmyw jemmyw self-assigned this Jan 13, 2021
@ocsarpo
Copy link
Author

ocsarpo commented Jan 18, 2021

In my Project, there is no 'spec' directory. I'm using only minitest now.
But I'm using the 'minitest-spec-rails' Gem for test, can this gem the cause of my problem?

If that be so.., I want to set default directory option at '.vscode/settings.json' file.
But for now, It can't be, So, I'll be wait these feature.....

@drnic
Copy link

drnic commented Jul 29, 2023

Perhaps there's something not quite right here? https://github.com/jemmyw/vscode-rails-fast-nav/blob/master/src/rails-workspace.ts#L57-L71

@igordepolli
Copy link
Contributor

igordepolli commented May 6, 2024

For me, looks like the problem is here: https://github.com/jemmyw/vscode-rails-fast-nav/blob/master/src/rails-workspace.ts#L213-L214. The return of getTestPath with spec properties. I proposed PR #35 to fix it. @jemmyw see if it makes sense or if I'm wrong.

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

No branches or pull requests

4 participants