async icon indicating copy to clipboard operation
async copied to clipboard

Add execution order test

Open kelunik opened this issue 3 years ago • 2 comments

This test currently has a different execution order in my adapter, so let's document the expected execution order.

kelunik avatar Aug 27 '22 09:08 kelunik

@SimonFrings Done. 👍

kelunik avatar Sep 07 '22 18:09 kelunik

@kelunik thanks for the update 👍

The test itself in its functionality and the overall design looks great. I talked with @clue about this and it looks like your testcase is already covered by other tests and didn't raise the overall code coverage.

I would argue that it wouldn't make sense to add a test just to show a specific example of what is possible.

SimonFrings avatar Sep 09 '22 11:09 SimonFrings

I'm closing this for now as it hasn't received any input in a while and I believe this has been answered. If you feel this is still relevant, please come back with more details and we can always reopen this :+1:

Thank you for your effort nonetheless, keep it up! :+1:

clue avatar Oct 24 '23 15:10 clue