[tests-only] [full-ci] transfers occ command test scenarios to cli test suites
Description
Transfers occ command test scenarios to cli test suites
Related Issue
- From https://github.com/owncloud/core/issues/39173?
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Database schema changes (next release will require increase of minor version instead of patch)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] Technical debt
- [x] Tests only (no source changes)
Checklist:
- [ ] Code changes
- [ ] Unit tests added
- [ ] Acceptance tests added
- [ ] Documentation ticket raised:
- [ ] Changelog item, see TEMPLATE
@phil-davis @kiranparajuli589 occ command scenarios in listed files needs to run with make test-acceptance-webui command which means I can't run them under make test-acceptance-cli command. Should I leave these tests?
- webUIPersonalSettings/accessToChangeFullNameThroughConfig.feature
- webUIPersonalSettings/personalGeneralSettings.feature
@phil-davis @kiranparajuli589
occ commandscenarios in listed files needs to run withmake test-acceptance-webuicommand which means I can't run them undermake test-acceptance-clicommand. Should I leave these tests?
- webUIPersonalSettings/accessToChangeFullNameThroughConfig.feature
- webUIPersonalSettings/personalGeneralSettings.feature
@sakshamgurung, since the tests uses occ commands as when step, I think all these transferred tests should be moved inside cli suites, that should allow you to use make test-acceptance-cli too
New webUIWithCli folder is created to contain feature files that use both webui and cli contexts. Putting these files inside cli test suite don't allow the use of webui contexts and needs changes in drone and make files, so creating a separate folder inside webui test suite makes more ease.
CI fails in a clone step :-(
@sakshamgurung does this solve #39173 ?
I converted to draft because we do not want to merge acceptance test refactorings to master while the 10.9.0 release process is in progress.
Note: PR #39514 has the other pending test changes waiting for oC10 master.
I will look at this PR later in the week and sort out some refactoring related to having the new sysconfig endpoint in the testing app: https://github.com/owncloud/testing/pull/190
We can use that for system settings changes where the occ config:system:set command is being used just as a Given step in scenario setup.
This PR is no longer in a current project, because the code is "essentially done". I assigned myself so that I find it easily.
Note: there is no rush for this. I would like to sort out some things. I am away this coming week and will look at it the first week of 2022.
I'll see what we can do on this.
CI failes with
+ php occ security:certificates:import /drone/server.crt
certificate not found
also there is more than only moving the tests happening here, please adjust original description







