owncloud / data_exporter

Export/Import for ownCloud user data
GNU General Public License v2.0
7 stars 5 forks source link

Nightly failed on cliDataExporter/export.feature #205

Closed grgprarup closed 1 year ago

grgprarup commented 1 year ago

Failed on acceptance tests in different pipelines. Build: https://drone.owncloud.com/owncloud/data_exporter/2127/15/11

Scenario: An uploaded file should be contained in an export.                                  # /var/www/owncloud/testrunner/apps/data_exporter/tests/acceptance/features/cliDataExporter/export.feature:9
    Given user "Alice" uploads file with content "hello" to "testfile.txt" using the WebDAV API # FeatureContext::userUploadsAFileWithContentTo()
    When user "Alice" is exported to path "/tmp/fooSomething" using the occ command             # DataExporterContext::exportUserUsingTheOccCommand()
    Then the last export should contain file "testfile.txt" with content "hello"                # DataExporterContext::theLastExportShouldContainFileWithContent()
      Failed to read the file /dataexp_acc_test/ae3e881156e5/tmp/fooSomething/Alice/files.jsonl
      Failed asserting that 404 is identical to 200.

Failed scenarios:

cliDataExporter/export.feature:9
cliDataExporter/export.feature:14
cliDataExporter/export.feature:39
cliDataExporter/export.feature:40
cliDataExporter/export.feature:41
cliDataExporter/export.feature:50
cliDataExporter/export.feature:51
cliDataExporter/export.feature:52
cliDataExporter/export.feature:53
cliDataExporter/export.feature:54
cliDataExporter/export.feature:62
cliDataExporter/export.feature:63
cliDataExporter/export.feature:64
cliDataExporter/export.feature:65
phil-davis commented 1 year ago

Note: it failed against core "master" and passed against core "latest" (10.10). There is probably a problem with something that was merged in core yesterday.

phil-davis commented 1 year ago

There is a latent bug in the data_exporter code, which has been exposed by changes merged in core PR https://github.com/owncloud/core/pull/40257

See PR #206 for the fix.