On 20/09/2019 9:33 am, Mandy Chung wrote:
On 9/19/19 4:06 PM, David Holmes wrote:

2. You implemented the new method in the test class.

I don't understand why you did that. The test can't be calling the new method. Now that it is a default method we will get past the compilation failure that caused the problem. So no change to the test should be needed AFAICS.


Maybe.  It can verify easily.   I don't know what the test does and whether it expects a good return value rather than UOE.

The test doesn't call it. It is a new method that the test knows nothing about.

David
-----

Implementing the new method has no harm but as I commented early, it is a long attribute and not an operation.

I'm okay with either way.

Mandy

Reply via email to