move all inner TEST classes to the test hierarchy

TechnionYP5779 / team4

29 stars 2 forks source link

move all inner TEST classes to the test hierarchy #88

Closed yossigil closed 5 years ago

menhel commented 5 years ago

Remaining test to move:

[x] il.org.spartan - Utils.java
[x] fluent.ly - as.java
[ ] fluent.ly - prune.java
[ ] fluent.ly - seperate.java

ron4548 commented 5 years ago

fluent.ly.prune
fluent.ly.seperate tests contains testing of private methods, which cannot be extracted to outer test class All other inner tests were moved to test directory. Took 4 hours

yossigil commented 5 years ago

if you do not have a test source hierarchy, maven will not work, and you will not be able to maintain coverage. You must have a source hierachy of tests.

Please read about this, and do not close this issue without @dormaayan approval

ron4548 commented 5 years ago

@dormaayan This is the test hierarchy we created. If this is not what you mean please let us know what is expected

ameeramer commented 5 years ago