chore(ci): run acceptance tests in GitHub workflows#1237
chore(ci): run acceptance tests in GitHub workflows#1237phil-davis wants to merge 44 commits intomasterfrom
Conversation
5c29c2a to
e4ba790
Compare
This allows settings like: AllowOverride All to happen. The final way to do this needs to be thought about, bwecause we want to put the potential site conf files somewhere that will be a common place for all app CI to get them. And there should be flexibility about the names "server" and "federated".
When deleting files, the activity message about the delete is not always at gthe top of the list. So the existing check sometimes fails, but passes when the test scenario is run again. So check for the expected activity message anywhere in the list, it is not essential that it be right at the top.
e4ba790 to
1d1bf4a
Compare
093b1f6 to
f6501f6
Compare
This is code from core for processing a trashbin restore. It did not account for when the server is installed in a subfolder, and the href of items has the subfolder at the front. "/server/remote.php/dav/trash-bin/Alice/" The code here is copied from core TrashbinContext and adjusted to expect the subfolder at the front of the href. If it works, then it will need to go in the core code.
f6501f6 to
f3b134e
Compare
to get the fix for the trashbin restore acceptance test.
4f959f1 to
59b77fa
Compare
1fb7474 to
eb8a887
Compare
The Apache site conf files are now stored in core tests/acceptance/setupApache And sort out the display of settings and log on failure()
and set permissions and validate app-repository
37088af to
1e8d102
Compare
|
But on the latest commit: Need to understand what is happening, and why it used to sometimes pass. |
|
owncloud log has multiple: First investigate that. Edit: |
c4bb180 to
48fe26b
Compare
48fe26b to
da09667
Compare
That webUI scenario passes locally for me, running the selenium-chrome docker image, watching the web interaction with So there is something in the workflow environment/setup that is causing "element not interactable" |
|
Some days ago we had: So there was a more detailed message, and the automatic rerun of the scenario passed. So the "other element" did not always obscure the needed UI element. |
|
See #1242 for the compressed version. |
This is where I got up to earlier today, before I started trying to get it all to work with https
The drone pipelines did work without https being set up, so probably best to continue working here to debug why some acceptance tests are failing.