newrelic / newrelic-dotnet-agent

The New Relic .NET language agent.
Apache License 2.0
98 stars 62 forks source link

Tech debt: Refactor `RemoteApplication` in Integration Tests #2748

Open tippmar-nr opened 2 months ago

tippmar-nr commented 2 months ago

We currently have a number of classes that inherit RemoteApplication, and several of them have nearly identical implementations of Start(). We should look for opportunities to refactor and make the base class more flexible - for example, we might move the code that sets the (long and growing) list of required environment variables to the base class which would simplify the derived class implementations considerably.

There are also likely to be similar overrides of CopyToRemote() that we should try to refactor if it makes sense.

workato-integration[bot] commented 2 months ago

https://new-relic.atlassian.net/browse/NR-311700