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

Linux support? #6

Closed
victorgveloso opened this issue Jun 18, 2020 · 2 comments
Closed

Linux support? #6

victorgveloso opened this issue Jun 18, 2020 · 2 comments

Comments

@victorgveloso
Copy link

When I try to run TestSmellDetector on a UNIX-like system the following error is printed out:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 4 out of bounds for length 1
        at testsmell.TestFile.getTagName(TestFile.java:44)
        at Main.main(Main.java:91)

This error references the following line of code:

    public String getTagName(){
        return testFilePath.split("\\\\")[4];
    }

Which explicitly deals with backslashes (used on windows path). So assuming TestSmellDetector is Windows-only: Is Linux support planned?

@victorgveloso
Copy link
Author

victorgveloso commented Jun 18, 2020

Actually there is a PR (#5) that fix this problem. I'll try that and post a review whether it works or not.
EDIT: It works! It'd be great if you merge it.
EDIT²: I've changed my mind. The changes from #5 introduces bugs that corrupts the output file.

@victorgveloso
Copy link
Author

#5 fixed and merged!

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

1 participant