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

[test-ony][full-ci] adding tests for invitation property which was missing in permissions responses #9675

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

ishabaral
Copy link
Contributor

@ishabaral ishabaral commented Jul 23, 2024

Description

This PR adds tests for invitation property which was missing while listing permissions via the .../graph/v1beta1/drives/{driveId}/items/{itemId}/permissions endpoint

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1: locally
  • test case 2: CI
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

Copy link

update-docs bot commented Jul 23, 2024

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.

@ishabaral ishabaral force-pushed the tests/invitation_in_permission_responses branch 4 times, most recently from 4a9b5dd to 843076e Compare July 23, 2024 09:15
@ishabaral ishabaral self-assigned this Jul 23, 2024
@ishabaral ishabaral marked this pull request as ready for review July 23, 2024 09:45
@ishabaral ishabaral force-pushed the tests/invitation_in_permission_responses branch 2 times, most recently from f0c6b59 to 50afd38 Compare July 23, 2024 11:19
@ishabaral ishabaral force-pushed the tests/invitation_in_permission_responses branch from 50afd38 to 190e269 Compare July 24, 2024 04:09
@ishabaral ishabaral force-pushed the tests/invitation_in_permission_responses branch from 190e269 to 009a4a5 Compare July 24, 2024 05:01
Copy link

Copy link
Member

@saw-jan saw-jan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@saw-jan saw-jan merged commit ad0e8a6 into master Jul 24, 2024
4 checks passed
@saw-jan saw-jan deleted the tests/invitation_in_permission_responses branch July 24, 2024 06:10
ownclouders pushed a commit that referenced this pull request Jul 24, 2024
…_responses

[test-ony][full-ci] adding tests for `invitation` property which was missing in permissions responses
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.

2 participants