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

[tests-only][full-ci] Backport refactoring webdav context removing setresponse() in given/then step #7480

Merged
merged 2 commits into from
Oct 13, 2023

Conversation

KarunAtreya
Copy link
Contributor

Description

This pr backports #7403 from master to stable-4.0

refactoring the given steps in the webdav context

refactored remaining given/when/then steps

return file id

trim trailing from the folder path

add missing when step and make other minor corrections

set responseXml and correct status code check

correct the naming of steps and helper functions
@update-docs
Copy link

update-docs bot commented Oct 13, 2023

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@KarunAtreya KarunAtreya changed the title refactoring webdav context (setting response in when step only) [tests-only][full-ci] refactoring webdav context (setting response in when step only) Oct 13, 2023
@KarunAtreya KarunAtreya changed the title [tests-only][full-ci] refactoring webdav context (setting response in when step only) [tests-only][full-ci] Backport refactoring webdav context removing setresponse() in given/then step Oct 13, 2023
@KarunAtreya KarunAtreya force-pushed the backport_refactor_webdav_context branch from e7d8fcb to a42cacb Compare October 13, 2023 06:34
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@KarunAtreya KarunAtreya requested a review from SagarGi October 13, 2023 10:09
@saw-jan saw-jan merged commit 46fde62 into stable-4.0 Oct 13, 2023
2 checks passed
@delete-merged-branch delete-merged-branch bot deleted the backport_refactor_webdav_context branch October 13, 2023 11:06
ownclouders pushed a commit that referenced this pull request Oct 13, 2023
…tresponse() in given/then step (#7480)

* refactoring webdav context (setting response in when step only)

refactoring the given steps in the webdav context

refactored remaining given/when/then steps

return file id

trim trailing from the folder path

add missing when step and make other minor corrections

set responseXml and correct status code check

correct the naming of steps and helper functions

* setting response
ownclouders pushed a commit that referenced this pull request Oct 14, 2023
…tresponse() in given/then step (#7480)

* refactoring webdav context (setting response in when step only)

refactoring the given steps in the webdav context

refactored remaining given/when/then steps

return file id

trim trailing from the folder path

add missing when step and make other minor corrections

set responseXml and correct status code check

correct the naming of steps and helper functions

* setting response
ownclouders pushed a commit that referenced this pull request Oct 16, 2023
…tresponse() in given/then step (#7480)

* refactoring webdav context (setting response in when step only)

refactoring the given steps in the webdav context

refactored remaining given/when/then steps

return file id

trim trailing from the folder path

add missing when step and make other minor corrections

set responseXml and correct status code check

correct the naming of steps and helper functions

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

Successfully merging this pull request may close these issues.

3 participants