Add test cases for issue 122 #131
Conversation
| @@ -0,0 +1,20 @@ | |||
| package p; | |||
There was a problem hiding this comment.
What is the purpose of this test?
There was a problem hiding this comment.
In this test case, n() has a dependency on m(), but in the first test case, n() and m() are independent.
There was a problem hiding this comment.
I see. In an effort to reduce test time, let's keep the file but remove it from the test case.
| @@ -0,0 +1,23 @@ | |||
| package p; | |||
There was a problem hiding this comment.
What is the purpose of this test?
There was a problem hiding this comment.
I just follow your instruction: " For the case created above, create another case by copying the above but removing one of the @entrypoint annotations." #122
I guess you want to create a comparison for the test case with multiple entry points.
There was a problem hiding this comment.
I see. I don't think we need a case with a dependency between the two methods. I'm unsure as to what that is going to buy us. Please let me know if you disagree.
| @@ -0,0 +1,20 @@ | |||
| package p; | |||
There was a problem hiding this comment.
I see. In an effort to reduce test time, let's keep the file but remove it from the test case.
| @@ -0,0 +1,23 @@ | |||
| package p; | |||
There was a problem hiding this comment.
I see. I don't think we need a case with a dependency between the two methods. I'm unsure as to what that is going to buy us. Please let me know if you disagree.
2 participants
#122