Skip to content

Stable9.1 tests md5 homedir#27003

Closed
SergioBertolinSG wants to merge 10 commits into
stable9.1from
stable9.1-tests-md5-homedir
Closed

Stable9.1 tests md5 homedir#27003
SergioBertolinSG wants to merge 10 commits into
stable9.1from
stable9.1-tests-md5-homedir

Conversation

@SergioBertolinSG

@SergioBertolinSG SergioBertolinSG commented Jan 23, 2017

Copy link
Copy Markdown
Contributor

Backport of #26846, #26865 and #26920

Jenkinsfile has been modified to avoid the use of makefiles.

@mention-bot

Copy link
Copy Markdown

@SergioBertolinSG, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975 and @PVince81 to be potential reviewers.

@PVince81 PVince81 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@SergioBertolinSG

Copy link
Copy Markdown
Contributor Author

Runnning locally tests with alternative home there are some failures:

build/integration/features/external-storage.feature:7
build/integration/features/sharing-v1.feature:59
build/integration/features/sharing-v1.feature:80
build/integration/features/sharing-v1.feature:113
build/integration/features/sharing-v1.feature:144
build/integration/features/sharing-v1.feature:174
build/integration/features/sharing-v1.feature:204
build/integration/features/sharing-v1.feature:857

@SergioBertolinSG

SergioBertolinSG commented Jan 23, 2017

Copy link
Copy Markdown
Contributor Author

The sharing problems are related with public shares, it is not detecting a url in the answer:
url doesn't have value AN_URL

@SergioBertolinSG

Copy link
Copy Markdown
Contributor Author

External storage one is failing because the same reason.

@SergioBertolinSG

SergioBertolinSG commented Jan 23, 2017

Copy link
Copy Markdown
Contributor Author

The problem is the string 'core' is being added to the public link. Should I add it to the expected url?

like in http://localhost:8080/core/index.php/s/c4Wb7qvd0bIFWMQ instead of
http://localhost:8080/index.php/s/c4Wb7qvd0bIFWMQ

@PVince81

Copy link
Copy Markdown
Contributor

@SergioBertolinSG are you sure ? Why didn't we see these failures before this PR ?

I don't think there is any such big change related to public links between master and 9.1 so it might be a false failure.

@PVince81

Copy link
Copy Markdown
Contributor

I could never run these tests due to permission issues:

    And Public shared file "welcome.txt" can be downloaded # FeatureContext::checkPublicSharedFile()
      Unable to open ./welcome.txt using mode w+: fopen(./welcome.txt): failed to open stream: Permission denied (GuzzleHttp\Exception\RequestException)

somehow it seems to be trying to write a local file. We should change that at some point to store the file contents in memory instead.

@PVince81

Copy link
Copy Markdown
Contributor

by the way, I don't see anything with "core":

Config value enable_alt_user_backend for app testing set to yes
Feature: sharing

  Background:                   # /srv/www/htdocs/owncloud/build/integration/features/sharing-v1.feature:2
    Given using api version "1" # FeatureContext::usingApiVersion()
    Given using old dav path    # FeatureContext::usingOldDavPath()

  Scenario: Creating a new public share                    # /srv/www/htdocs/owncloud/build/integration/features/sharing-v1.feature:59
[Mon Jan 23 20:23:02 2017] 127.0.0.1:47250 [404]: /ocs/v2.php/cloud/users/user0
[Mon Jan 23 20:23:03 2017] Successful addUser call with userid: user0
[Mon Jan 23 20:23:03 2017] 127.0.0.1:47254 [200]: /ocs/v1.php/cloud/users
[Mon Jan 23 20:23:03 2017] copying skeleton for user0 from /srv/www/htdocs/owncloud/core/skeleton to /user0/files/
[Mon Jan 23 20:23:04 2017] 127.0.0.1:47258 [200]: /ocs/v1.php/cloud/users/user0
[Mon Jan 23 20:23:04 2017] 127.0.0.1:47262 [200]: /ocs/v2.php/cloud/users/user0
    Given user "user0" exists                              # FeatureContext::assureUserExists()
    And As an "user0"                                      # FeatureContext::asAn()
[Mon Jan 23 20:23:05 2017] 127.0.0.1:47266 [200]: /ocs/v1.php/apps/files_sharing/api/v1/shares
    When creating a share with                             # FeatureContext::creatingShare()
      | path      | welcome.txt |
      | shareType | 3           |                                                                                                                                                                                 
    Then the OCS status code should be "100"               # FeatureContext::theOCSStatusCodeShouldBe()
    And the HTTP status code should be "200"               # FeatureContext::theHTTPStatusCodeShouldBe()
[Mon Jan 23 20:23:05 2017] 127.0.0.1:47270 [200]: /index.php/s/0A5g53VALzvBJAm/download
    And Public shared file "welcome.txt" can be downloaded # FeatureContext::checkPublicSharedFile()
      Unable to open ./welcome.txt using mode w+: fopen(./welcome.txt): failed to open stream: Permission denied (GuzzleHttp\Exception\RequestException)

did you switch dev env maybe ?

@PVince81

Copy link
Copy Markdown
Contributor

I put all these into the other PR #26993 to save time...

@PVince81 PVince81 closed this Jan 23, 2017
@PVince81
PVince81 deleted the stable9.1-tests-md5-homedir branch January 23, 2017 19:32
@SergioBertolinSG

Copy link
Copy Markdown
Contributor Author

Yes, I guess my server is broken because of switching between master and stable9.1. I'll use a new one only for stable9.1.

@lock

lock Bot commented Aug 4, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants