Azure-Player / azure.datafactory.tools

Tools for deploying Data Factory (v2) in Microsoft Azure
https://azureplayer.net/adftools
MIT License
209 stars 69 forks source link

Fix #301 #302

Closed josephniblo closed 1 year ago

josephniblo commented 1 year ago

Use explicit datetime formatting to ensure replaced datetime parameters in json files include the timezone identifier, as expected by Start-AzDataFactoryV2Trigger.

josephniblo commented 1 year ago

@NowinskiK As requested, I've updated an existing unit test to represent the bug. This fails on the current master branch, but passes with my changes.

I'm not very familiar with unit testing Powershell and had some issues testing locally, so any feedback is welcome.

Thanks

NowinskiK commented 1 year ago

Thanks, I will review it as soon as I can.