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] Adjust expected failures after commit id update #2003

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

jasson99
Copy link
Contributor

@jasson99 jasson99 commented Aug 18, 2021

Description

The related issue is "Implement expiration date for shares"

  1. The scenarios related to that issue had ended up in a couple of different places. This PR puts the scenarios together in the expected-failures file in 3 sections, for user, group and public link shares. This should make it easier when a developer starts to implement this some day.

  2. And bump the core commit id.

  3. Also, the skipOnOcis filter tag is also added in the drone.star file

Related Issue

How Has This Been Tested?

CI

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:

@jasson99 jasson99 marked this pull request as ready for review August 18, 2021 06:39
@jasson99 jasson99 requested a review from labkode as a code owner August 18, 2021 06:39
@jasson99 jasson99 force-pushed the bumpCommitId-2021-08-18 branch from 6048f17 to f601458 Compare August 18, 2021 06:40
@jasson99
Copy link
Contributor Author

#1963 should be merged first

@jasson99 jasson99 force-pushed the bumpCommitId-2021-08-18 branch from f601458 to 94c274c Compare August 18, 2021 07:31
@phil-davis
Copy link
Contributor

@jasson99 the other PR was merged. You can rebase this.

@jasson99 jasson99 force-pushed the bumpCommitId-2021-08-18 branch from 014135d to 8309156 Compare August 23, 2021 04:21
@jasson99 jasson99 force-pushed the bumpCommitId-2021-08-18 branch from 7a63f60 to 9311d87 Compare August 23, 2021 07:27
@jasson99 jasson99 changed the title [Tests-Only] Adjust expected failures for share expiration date [Tests-Only] Adjust expected failures after commit id update Aug 23, 2021
phil-davis
phil-davis previously approved these changes Aug 23, 2021
@phil-davis
Copy link
Contributor

@ishank011 this is ready to merge - gets the tests right up-to-date.

@phil-davis
Copy link
Contributor

@labkode this is an easy merge.

@phil-davis
Copy link
Contributor

@ishank011 @labkode any chance to merge?

@phil-davis
Copy link
Contributor

@ishank011 @labkode ping

@phil-davis
Copy link
Contributor

This has been rebased and is ready to merge.

@labkode labkode merged commit 0dbb301 into cs3org:master Sep 1, 2021
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Sep 1, 2021
@phil-davis phil-davis deleted the bumpCommitId-2021-08-18 branch September 1, 2021 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants