-
Notifications
You must be signed in to change notification settings - Fork 13
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
Use real Azure resources in our integration tests #174
Comments
Almost all things are (integration) tested. I see the following still open:
Some things to be investigated. |
So, we have already integration tests for Logic Apps, API Management is too expensive, and our Data Factory scripts are actually only a single one with a single function; so maybe also not the costs or effort worth? |
Only thing missing would indeed be Data Factory. From what I gather running costs would be low, but it would take some effort to build this as you would also need to create an actual flow within Data Factory. To me it seems like a bit too much effort for a single script and we can close this issue. @stijnmoreels what do you think? |
Yeah, @pim-simons , you're right. Looking at the code, the most important additions is exception handling, which is already tested in our unit tests. The script itself uses solely Azure PS commands, so seems indeed that this would be too big an effort. Closing this. |
Is your feature request related to a problem? Please describe.
In the very beginning, we only added scripts to the repo that were already tested in another repo. But this changes and so we are having some blind spots if it comes to the security and stability of our scripts.
Describe the solution you'd like
Use where we can real Azure resources (within the price limit) to test our current set of scripts.
The text was updated successfully, but these errors were encountered: