[tests-only]Bump CORE_COMMITID

[SagarGi]

Description

This PR bumps core commit id to latest for tests

Part of: owncloud/QA#753

• Removes tests from expected to fail linked with issue Various sharing settings cannot be set owncloud/ocis#1328 that were skipped by PR [tests-only] Skip tests related to issue-ocis-1328 on ocis owncloud/core#40213
• Add a BEHAT_FILTER_TAGS in drone.star to skip the tests tagged with skipOnReva introduced by PR [tests-only] Add skipOnReva tag to tests not applicable for reva owncloud/core#40219

And also This PR skips the caldav and carddav endpoint request
owncloud/ocis#4130

[phil-davis]

https://drone.cernbox.cern.ch/cs3org/reva/8027/15/6
It ran for a full hour - maybe hanging after:

Scenario Outline: Copy a folder into another one                                                    # /drone/src/tmp/testrunner/tests/acceptance/features/apiWebdavProperties1/copyFile.feature:889
    Given using <dav_version> DAV path                                                                # FeatureContext::usingOldOrNewDavPath()
    And user "Alice" has created folder "/testshare"                                                  # FeatureContext::userHasCreatedFolder()
    And user "Alice" has created folder "/an-other-folder"                                            # FeatureContext::userHasCreatedFolder()
    When user "Alice" copies folder "/testshare" to "/an-other-folder/testshare" using the WebDAV API # FeatureContext::userCopiesFileUsingTheAPI()
    Then the HTTP status code should be "201"                                                         # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And user "Alice" should see the following elements                                                # FeatureContext::userShouldSeeTheElements()
      | /testshare/ |
    And user "Alice" should see the following elements                                                # FeatureContext::userShouldSeeTheElements()
      | /an-other-folder/testshare/ |

    Examples:
      | dav_version |
      | old         |
      | new         |

I will restart and see if the same thing happens.

s3ng storage also got to the same point:
https://drone.cernbox.cern.ch/cs3org/reva/8027/27/7

The next scenario is:
Scenario Outline: Copying a folder into a sub-folder of itself

I suspect that that scenario has the recursive problem in reva master branch, so we will need to skip it.

[phil-davis]

Needs core PR owncloud/core#40212 approved and merged, then bump the core commit id again.

[phil-davis]

@labkode @gmgigi96 or anyone with enough privs - please approve.
We touched .drone.star to skip the caldav/carddav test scenarios, so GitHub won't let me merge.

[SwikritiT]

Following tests failed in CI

apiSharePublicLink1/createPublicLinkShare.feature:477
 apiSharePublicLink1/createPublicLinkShare.feature:478

Scenario Outline: Allow public sharing of the root on OCIS when the default permission is read and access using the public WebDAV API                                     # /drone/src/tmp/testrunner/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature:445
    Given using OCS API version "<ocs_api_version>"                                                                                                                         # FeatureContext::usingOcsApiVersion()
    And user "Alice" has uploaded file with content "Random data" to "/randomfile.txt"                                                                                      # FeatureContext::userHasUploadedAFileWithContentTo()
    When user "Alice" creates a public link share using the sharing API with settings                                                                                       # FeatureContext::userCreatesAPublicLinkShareWithSettings()
      | path | / |
    Then the OCS status code should be "<ocs_status_code>"                                                                                                                  # OCSContext::theOCSStatusCodeShouldBe()
    And the HTTP status code should be "200"                                                                                                                                # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And the fields of the last response to user "Alice" should include                                                                                                      # FeatureContext::checkFields()
      | item_type              | folder               |
      | mimetype               | httpd/unix-directory |
      | file_target            | /                    |
      | path                   | /                    |
      | permissions            | read                 |
      | share_type             | public_link          |
      | displayname_file_owner | %displayname%        |
      | displayname_owner      | %displayname%        |
      | uid_file_owner         | %username%           |
      | uid_owner              | %username%           |
      | name                   |                      |
    And the public should be able to download file "/randomfile.txt" from inside the last public link shared folder using the <public-webdav-api-version> public WebDAV API # PublicWebDavContext::shouldBeAbleToDownloadFileInsidePublicSharedFolder()
    And the downloaded content should be "Random data"                                                                                                                      # FeatureContext::downloadedContentShouldBe()
    And the public upload to the last publicly shared folder using the <public-webdav-api-version> public WebDAV API should fail with HTTP status code "403"                # PublicWebDavContext::publiclyUploadingShouldNotWork()

    Examples:
      | ocs_api_version | ocs_status_code | public-webdav-api-version |
      | 1               | 100             | new                       |
        │ file_target has unexpected value '/home'
        │ 
        file_target doesn't have value '/'
        Failed asserting that false is true.
      | 2               | 200             | new                       |
        │ file_target has unexpected value '/home'
        │ 
        file_target doesn't have value '/'
        Failed asserting that false is true.

Passes locally in OCIS

[phil-davis]

reva master could be different from reva edge and oCIS - fixes (and changes/differences to behavior) are not often backported from edge to master.

If these tests pass on reva edge branch, then we just need to keep/put them in expected-failures on reva master and a note about it in the expected-failures file. We don't need to raise issues for a difference between reva master and reva edge.

What to do about reva master tests for the future needs to be discussed separately.

[SwikritiT]

reva master could be different from reva edge and oCIS - fixes (and changes/differences to behavior) are not often backported from edge to master.

If these tests pass on reva edge branch, then we just need to keep/put them in expected-failures on reva master and a note about it in the expected-failures file. We don't need to raise issues for a difference between reva master and reva edge.

What to do about reva master tests for the future needs to be discussed separately.

Okay I'll add them to expected-failures with a note.

[phil-davis]

@labkode or anyone who can - this has been updated to the latest core commit id. Please review and merge.

[grgprarup]

The tests are failing on newly added MOVE requests with body, https://drone.cernbox.cern.ch/cs3org/reva/8201/16/7#:~:text=Scenario%3A%20send%20MOVE%20requests%20to%20webDav%20endpoints%20with,1512
Since, implementation for http 415 is done in reva-edge only.

[phil-davis]

The tests are failing on newly added MOVE requests with body, https://drone.cernbox.cern.ch/cs3org/reva/8201/16/7#:~:text=Scenario%3A%20send%20MOVE%20requests%20to%20webDav%20endpoints%20with,1512 Since, implementation for http 415 is done in reva-edge only.

I will add to expected-failures and make a general issue for "things that fail in reva master".

[phil-davis]

@labkode @glpatcern up-to-date again - please merge.