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

Add composer allow-plugins #325

Merged
merged 1 commit into from
Feb 18, 2022
Merged

Conversation

kiranparajuli589
Copy link
Contributor

Description

This PR commits the allow-plugins section for composer.json so that it is there for anyone who updates to composer 2.2.

Related

Part of owncloud/QA#723

Note: This PR is created with a script. If there is any unexpected case in it, I will update manually.

@ownclouders
Copy link
Contributor

💥 Acceptance tests pipeline cliCoreMKey-7-3-master-mysql8.0-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/encryption/2314/70/1

@kiranparajuli589
Copy link
Contributor Author

total unexpected failed scenarios

runsh: Total unexpected failed scenarios throughout the test run:
cliExternalStorage/filesExternalWebdavOwncloud.feature:251
cliExternalStorage/filesExternalWebdavOwncloud.feature:270
cliExternalStorage/filesExternalWebdavOwncloud.feature:290
cliExternalStorage/filesExternalWebdavOwncloud.feature:309
cliExternalStorage/filesExternalWebdavOwncloud.feature:329
cliExternalStorage/filesExternalWebdavOwncloud.feature:354
cliExternalStorage/filesExternalWebdavOwncloud.feature:379
cliExternalStorage/filesExternalWebdavOwncloud.feature:401

@kiranparajuli589
Copy link
Contributor Author

 Scenario: user moves their own folder to the external storage                                                                         # /var/www/owncloud/testrunner/tests/acceptance/features/cliExternalStorage/filesExternalWebdavOwncloud.feature:251
    Given user "Alice" has been created with default attributes and without skeleton files                                              # FeatureContext::userHasBeenCreatedWithDefaultAttributesAndWithoutSkeletonFiles()
      Provisioning::usersHaveBeenCreated Unexpected failure when creating the user 'Alice': HTTP status 400 HTTP reason Bad Request OCS status 400 OCS message User already exists (Exception)

how the user already exists? 🤔

@ownclouders
Copy link
Contributor

💥 Acceptance tests pipeline cliCoreMKey-7-3-master-mysql8.0-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/encryption/2315/70/1

@kiranparajuli589
Copy link
Contributor Author

kiranparajuli589 commented Feb 17, 2022

same errors with the rerun :(

@kiranparajuli589
Copy link
Contributor Author

kiranparajuli589 commented Feb 17, 2022

hmmm...these are recently added tests and are failing,,,i'm digging further 🗡️

@kiranparajuli589
Copy link
Contributor Author

kiranparajuli589 commented Feb 17, 2022

omg, user alice is already created in the background but these scenarios tries to create the same user again.
https://github.com/owncloud/core/blob/master/tests/acceptance/features/cliExternalStorage/filesExternalWebdavOwncloud.feature#L10

@phil-davis
Copy link
Contributor

I am fixing in #324 already. The problem was a defect in the existing CI pipelines. I need to see what other things are still failing.

@phil-davis
Copy link
Contributor

#324 is ready to review. It should pass CI. Then we can rebase here...

@kiranparajuli589
Copy link
Contributor Author

#324 is merged and here rebased.

Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

LGTM if CI passes.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kiranparajuli589 kiranparajuli589 merged commit 85654e4 into master Feb 18, 2022
@delete-merged-branch delete-merged-branch bot deleted the composer-allow-plugins branch February 18, 2022 09:13
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