Skip to content
This repository was archived by the owner on Nov 14, 2024. It is now read-only.

Provide Logic App + ARM used to test "TemporaryUpdateAsync" and "RunAsync" #31

Closed
2 tasks done
mbraekman opened this issue Jun 9, 2020 · 4 comments · Fixed by #41
Closed
2 tasks done

Provide Logic App + ARM used to test "TemporaryUpdateAsync" and "RunAsync" #31

mbraekman opened this issue Jun 9, 2020 · 4 comments · Fixed by #41
Assignees
@mbraekman
Labels
automated-testing All issues related to automated testing
Milestone
v0.1.0

Comments

@mbraekman
Copy link
Contributor

mbraekman commented Jun 9, 2020 •
edited
Loading

Is your feature request related to a problem? Please describe.
Provide a Logic App which can be used to validate the functionality of the TemporaryUpdateAsync-method.

Describe the solution you'd like
These steps are to be taken:

  • create basic logic app for this purpose
  • provide sample ARM to perform the update of the logic app.
@mbraekman mbraekman added the automated-testing All issues related to automated testing label Jun 9, 2020
@mbraekman mbraekman self-assigned this Jun 9, 2020
@mbraekman mbraekman mentioned this issue Jun 9, 2020
Merged
@stijnmoreels stijnmoreels changed the title Provide Logic App + ARM used to test "TemporaryUpdateAsync" Provide Logic App + ARM used to test "TemporaryUpdateAsync" and "RunAsync" Jun 12, 2020
@stijnmoreels
Copy link
Contributor

Relates to: #27 (comment)

@stijnmoreels
Copy link
Contributor

Relates to #27 (comment)

@mbraekman
Copy link
Contributor Author

A new Logic App has been created, which can be used for the 2 test-cases as mentioned in the subject.
This Logic App still allows you to provide a "CorrelationId"-header, but does not require you to.

The new Logic App can be found here:

  • resource group: invictus-tst-testing
  • name: invictus-tst-testing-rcv-trigger-http

As it sits now, it is about to return a simple response:
{
"response": "Original"
}

After using the provided ARM-template, it should return this:
{
"response": "Updated"
}

mbraekman added a commit that referenced this issue Jun 12, 2020
@mbraekman
Provide ARM-template to update existing Logic App
a4919cd 
Provide ARM-template to update existing Logic App.
After pushing this logic app definition, the response will be
{
    "response": "Updated"
}

Will originally it was:
{
    "response": "Original"
}
Related to #31
@mbraekman
Copy link
Contributor Author

@stijnmoreels,

I've added the ARM-template underneath the folder "LogicAppDefinitions" within the Integration-testing project (see my commit in a separate branch).

Filename: rcv-trigger-http.json

@mbraekman mbraekman mentioned this issue Jun 12, 2020
Merged
mbraekman added a commit that referenced this issue Jun 15, 2020
@mbraekman
Provide ARM-template to update existing Logic App (#36)
4ed3539 
Provide ARM-template to update existing Logic App.
After pushing this logic app definition, the response will be
{
    "response": "Updated"
}

Will originally it was:
{
    "response": "Original"
}
Related to #31
@stijnmoreels stijnmoreels added this to the v0.1.0 milestone Jun 17, 2020
@stijnmoreels stijnmoreels mentioned this issue Jun 17, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mbraekman mbraekman

Labels
automated-testing All issues related to automated testing
Projects
None yet
Milestone
v0.1.0
2 participants
@stijnmoreels @mbraekman