arcade-services icon indicating copy to clipboard operation
arcade-services copied to clipboard
verified publisher icon dotnet

Add e2e scenario tests for the PCS

Open dkurepa opened this issue 1 year ago • 4 comments

We should have some kind of e2e tests for the PCS. These tests would just add simple jobs (we can just write Mock jobs that do nothing) to the Queue, and make sure everything is executed properly, this like scope creation, metrics writing, etc..

dkurepa avatar Feb 06 '24 15:02 dkurepa

As a part of this issue we also want to add some DependencyRegistrationTests to verify DI registrations. We'll have to deal with Keyed Service Registrations tho

dkurepa avatar Feb 20 '24 15:02 dkurepa

I am confused about this - I don't think this was part of the original epic requirements. I believe scenario tests should be done later when we have what to test. I logged this https://github.com/dotnet/arcade-services/issues/3319 so I'd close this one and open one for the DI tests.

premun avatar Feb 20 '24 16:02 premun

Ah my bad, do you think we should include the DI tests there too?

dkurepa avatar Feb 20 '24 16:02 dkurepa

No, DI tests need to happen soon ideally

premun avatar Feb 22 '24 09:02 premun

Closing this as it already exists in the new epic https://github.com/dotnet/arcade-services/issues/3837

dkurepa avatar Aug 27 '24 14:08 dkurepa