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.

Sign up for GitHub

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

Jump to bottom

Once found "true", never goes "false" #19

Open
Teej42 opened this issue Mar 20, 2022 · 5 comments
Open

Once found "true", never goes "false" #19

Teej42 opened this issue Mar 20, 2022 · 5 comments

Comments

@Teej42
Copy link

Teej42 commented Mar 20, 2022

Run this using attached file on ftp-server 1.0.6. One example - Print Statement - returns True correctly for clienttests/SymbolicLinkTest.java, but remains True even right after that with no print statement - clienttests/ListPassiveTest.java. This behavior is observed on many code smell categories, indicating that clearing of status is not properly done.

Output_TestSmellDetection_1647781715304.csv
@Teej42
Copy link
Author

Teej42 commented Mar 20, 2022

Issue is present in Sensitive Equality.

@Teej42
Copy link
Author

Teej42 commented Mar 20, 2022

Issue is present in Sleepy Test.

@Teej42
Copy link
Author

Teej42 commented Mar 20, 2022

Issue is present in Duplicate Assert.

@Teej42
Copy link
Author

Teej42 commented Mar 20, 2022

Basically ALL of the code smell categories have this bug.

@effendiian
Copy link

I was able to replicate the same issue. Tested on a forked branch of RxJava: https://github.com/rimij405/RxJava

Output_TestSmellDetection_1648526004884.csv

shehan added a commit that referenced this issue Jun 14, 2022
@shehan
A fix for issue #19
b7e1aaf
marvinkreis added a commit to marvinkreis/TestSmellDetector that referenced this issue Oct 1, 2022
@marvinkreis
Fix TestSmellDetector ignoring smells set by setTestSmells
7357a05 
Save smell factories instead of smells in TestSmellDetector, so the
configured tests can be re-initialized whenever needed. (See TestSmells#19)
marvinkreis added a commit to marvinkreis/TestSmellDetector that referenced this issue Oct 1, 2022
@marvinkreis
Fix TestSmellDetector ignoring smells set by setTestSmells
112fd0b 
Save smell factories instead of smells in TestSmellDetector, so the
configured tests can be re-initialized whenever needed. (See TestSmells#19)
@marvinkreis marvinkreis mentioned this issue Oct 1, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@effendiian @Teej42