Mattia Tommasone

Results 81 comments of Mattia Tommasone

@maverick1601 , can you please resolve the conflicts?

PR LGTM, but I can't set the secrets :(

Thanks a lot @oleksiyp! I think once we put in place the two protections that @aSemy suggested we should be fine.

@aSemy do you happen to have any updates on this?

Why are you using `CopyOnWriteArrayList` for argument capturing? `capture` supports `slot` and `MutableList`, I don't think it works with `CopyOnWriteArrayList`.

Yep this does indeed look like a bug to me. A way to fix this would be to make sure `wasNot` was being called on a mocked object and throw...

Ah, you're right, calling `clearAllMocks` on every test run is definitely going to break parallel execution. Sorry about this. Would it be helpful if we called `clearAllMocks` on every test...

I took a look at this and the fix was easier than I expected. I'm going to add a `requireParallelTesting` configuration setting to disable calling `clearAllMocks` after each test execution.

Ah, you're right, thanks for looking into this and sending #1238! I'll merge it as soon as the builds pass