Skip to content

Fix wrong path for api objects on Azure. Add unit test for that case. #606

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 9, 2022
Merged

Fix wrong path for api objects on Azure. Add unit test for that case. #606

merged 3 commits into from
Jun 9, 2022

Conversation

@claudiamurialdo
Copy link
Collaborator

@claudiamurialdo claudiamurialdo commented Jun 7, 2022

@claudiamurialdo claudiamurialdo requested a review from sjuarezgx June 7, 2022 20:37
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 7, 2022 20:37 Inactive
@genexusbot
Copy link
Collaborator

genexusbot commented Jun 7, 2022

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

genexusbot commented Jun 8, 2022

Cherry pick to beta success

@claudiamurialdo claudiamurialdo had a problem deploying to external-storage-tests June 8, 2022 18:00 Failure
@claudiamurialdo claudiamurialdo mentioned this pull request Jun 8, 2022
Closed
@claudiamurialdo claudiamurialdo changed the title Add new test for Azure httphandler and API object. Fix wrong path for api objects on Azure. Add unit test for that case. Jun 8, 2022
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 8, 2022 23:45 Inactive
@genexusbot
Copy link
Collaborator

genexusbot commented Jun 8, 2022

Cherry pick to beta success

@claudiamurialdo claudiamurialdo merged commit c1c90c3 into master Jun 9, 2022
@claudiamurialdo claudiamurialdo deleted the azure-unit-test branch June 9, 2022 12:34
claudiamurialdo added a commit that referenced this pull request Jun 9, 2022
@claudiamurialdo
Fix wrong path for api objects on Azure. Add unit test for that case. (…
c4413cb 
…#606)

* Add new test for Azure httphandler and API object.

* Fix url casing for the test in order to reproduce error 500.

* Add the fix made at #607 but keeping the conditional operator OR.

(cherry picked from commit c1c90c3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sjuarezgx sjuarezgx Awaiting requested review from sjuarezgx

Assignees

No one assigned

Labels

bot closed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@claudiamurialdo @genexusbot