Update Test method parameter checks to account for CancellationToken

[manfred-brands]

Fixes #663

[mikkelbu]

General comment. It seems like all the code (analyzer and tests) only checks the case where the CancelAfterAttribute is placed directly on the test-method, but I guess this attribute could also have been placed on the test fixture?

[manfred-brands]

General comment. It seems like all the code (analyzer and tests) only checks the case where the CancelAfterAttribute is placed directly on the test-method, but I guess this attribute could also have been placed on the test fixture?

Yes. The CancelAfterAttribute is allowed on the class. Myself I don't see proper use of it, what if you add more tests, then the fixture needs more time even though all individual tests are fine.

But as NUnit supports it, I added tests and support for looking up the attribute on the class.

[mikkelbu]

+1LGTM. I'm happy to merge this when you are @manfred-brands. Thanks for the work 👍