Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tech debt: Refactor RemoteApplication in Integration Tests #2748

Open
tippmar-nr opened this issue Sep 10, 2024 · 1 comment
Open

Tech debt: Refactor RemoteApplication in Integration Tests #2748

tippmar-nr opened this issue Sep 10, 2024 · 1 comment

Comments

@tippmar-nr
Copy link
Member

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
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant