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

Detected smells are wrong #14

Open
eblio opened this issue Nov 8, 2021 · 0 comments
Open

Detected smells are wrong #14

eblio opened this issue Nov 8, 2021 · 0 comments

Comments

@eblio
Copy link

eblio commented Nov 8, 2021

In the latest version of this tool, whether it is release v2.1 or on the latest commit on main, the reported smells tend to be innexact.

When using the tool, we noted inconsistencies in the reported smells. The results would usually look like this:

File Smell 1 Smell 2 Smell 3
test1.java false true false
test2.java true true false
test3.java true true false
test4.java true true false

This is a sample table, displaying why we thought there was an issue, not representing real data.

When a smell gets detected in a file, it systematically gets detected in the next file, which is particularly shocking when a project has more than 20 test files. For over 500 of projects, the mean number of detected smells was around 8, which is a lot compared to what has been observed in other studies.

To obtain such result we input a .csv file mapping each test file to its production file (when possible), which means several files will be processed in a row.

I plan to do a pull request fixing this issue. More information on the technical details will be available there.

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