-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[#55939] OneDrive/SharePoint storage with AMPF can be added as manual folder #16007
[#55939] OneDrive/SharePoint storage with AMPF can be added as manual folder #16007
Conversation
@@ -0,0 +1,30 @@ | |||
module OAuthConnectionsHelpers | |||
def mock_one_drive_authorization_validation(with: {}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If someone has a better suggestion on how to name this method please let me know 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eventually this will vanish for a VCR cassette, so I wouldn't bother too much. Name is fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actuall I tried using a VCR cassette first (it "shows no option for manually managed permissions", vcr: "auth/one_drive/user_token" do
) but had some issues with it, because it's a feature spec:
An HTTP request has been made that VCR does not know how to handle:
GET http://172.18.0.9:3000/__identify__
(The IP is from the docker container and probably needs to be ignored by VCR - I didn't dive into this).
e851245
to
c70ad6a
Compare
c70ad6a
to
1858a34
Compare
…as manual folder https://community.openproject.org/work_packages/55939 Because managing of permissions is complex on OneDrive, a storage with automatically managed access and folders can not host projects with manually managed permissions. --- Please notice that an existing file had to be renamed due to rubocop complaining.
1858a34
to
3248469
Compare
modules/storages/spec/contracts/storages/project_storages/base_contract_spec.rb
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,30 @@ | |||
module OAuthConnectionsHelpers | |||
def mock_one_drive_authorization_validation(with: {}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eventually this will vanish for a VCR cassette, so I wouldn't bother too much. Name is fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
https://community.openproject.org/work_packages/55939
Because managing of permissions is complex on OneDrive, a storage with automatically managed access and folders can not host projects with manually managed permissions.
Please notice that an existing file had to be renamed due to rubocop complaining.