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

Investigate test fails cliMain with encryption and user_ldap #509

Closed
phil-davis opened this issue Feb 7, 2020 · 5 comments
Closed

Investigate test fails cliMain with encryption and user_ldap #509

phil-davis opened this issue Feb 7, 2020 · 5 comments

Comments

@phil-davis
Copy link
Contributor

phil-davis commented Feb 7, 2020

https://drone.owncloud.com/owncloud/user_ldap/2232/200/13

--- Failed scenarios:

    /var/www/owncloud/testrunner/tests/acceptance/features/cliMain/filesChecksum.feature:4
    /var/www/owncloud/testrunner/tests/acceptance/features/cliMain/filesChecksum.feature:8
    /var/www/owncloud/testrunner/tests/acceptance/features/cliMain/filesChecksum.feature:12
    /var/www/owncloud/testrunner/tests/acceptance/features/cliMain/filesChecksum.feature:17

24 scenarios (20 passed, 4 failed)
128 steps (124 passed, 4 failed)
Feature: files checksum command

  Scenario: Administrator verifies the checksum of all the files       # /var/www/owncloud/testrunner/tests/acceptance/features/cliMain/filesChecksum.feature:4
    When the administrator invokes occ command "files:checksum:verify" # OccContext::theAdministratorInvokesOccCommand()
    Then the command should have been successful                       # OccContext::theCommandShouldHaveBeenSuccessful()
      The command was not successful, exit code was 1.
      stdOut was: 'This operation might take quite some time.
      '
      stdErr was: '
      In Encryption.php line 359:
                                                                                     
        Can not decrypt this file, probably this is a shared file. Please ask the f  
        ile owner to reshare the file with you.                                      
                                                                                     
      
      files:checksums:verify [-r|--repair] [-u|--user USER] [-p|--path PATH]
      
      '
       (Exception)

It seems to have started the last 2 nights. Might be due to some change on about Wednesday 2020-02-05?

@phil-davis
Copy link
Contributor Author

Note: the fail is with user-key encryption.
It passes with master-key encryption: https://drone.owncloud.com/owncloud/user_ldap/2232/235/13

So maybe files:checksum:verify is not supported for user-key encryption.

@phil-davis
Copy link
Contributor Author

Happens in each nightly, e.g.: https://drone.owncloud.com/owncloud/user_ldap/2239/200/13

@phil-davis
Copy link
Contributor Author

The problem is in encryption. See these issues and linked PRs for the tests that are now skipped:
owncloud/core#36985
owncloud/encryption#181
owncloud/encryption#182

The user_ldap CLI acceptance tests should pass tonight, because the pipelines have the correct skipOnEncryption* filter tags, so they will ignore the skipped scenarios.

@phil-davis
Copy link
Contributor Author

The necessary scenarios should now be skipped in core.
We will see the result tonight, then if all is well then this issue can be closed.

@phil-davis
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant