Fixed some unit tests - Githubissues

Samsung / Dexter

Dexter is a static analysis platform to find and remove defects efficiently and immediately during the coding-time.

BSD 2-Clause "Simplified" License

56 stars 33 forks source link

Fixed some unit tests #141

Closed KarolAntczak closed 7 years ago

KarolAntczak commented 7 years ago

Hi, I fixed failing unit tests to pass regardless of the environment - they require Dexter jar to work, which is now downloaded to /TestData dir. Also I fixed naming convention of my unit tests to be consistent with yours.

mdopark commented 7 years ago

@KarolAntczak Thanks for your fix. I confirmed that all tests were successful. But the three tests below take too long(about 27 seconds except download time).

Analyse_returnNotEmpty
Analyse_callOutputDataReceived
Analyse_callErrorDataReceived

So it would be better to add Explicit atturibute to these three tests. What do you think?

KarolAntczak commented 7 years ago

Sounds like a good idea. I added this attribute to these tests.