Skip to content

chore(ci): run acceptance tests in GitHub workflows#1237

Draft
phil-davis wants to merge 41 commits intomasterfrom
acceptance-tests-http-1
Draft

chore(ci): run acceptance tests in GitHub workflows#1237
phil-davis wants to merge 41 commits intomasterfrom
acceptance-tests-http-1

Conversation

@phil-davis
Copy link
Copy Markdown
Contributor

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.

@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch 2 times, most recently from 5c29c2a to e4ba790 Compare April 13, 2026 04:34
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.
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from e4ba790 to 1d1bf4a Compare April 13, 2026 04:53
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from 093b1f6 to f6501f6 Compare April 13, 2026 08:10
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.
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from f6501f6 to f3b134e Compare April 13, 2026 08:53
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from 4f959f1 to 59b77fa Compare April 14, 2026 04:56
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from c6ef169 to 1e8e4a0 Compare April 15, 2026 05:09
@phil-davis
Copy link
Copy Markdown
Contributor Author

I finally go it to pass! Next is to look through this and tidy it up.

@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch 6 times, most recently from 4528b0c to 1fb7474 Compare April 15, 2026 12:19
The Apache site conf files are now stored in core
tests/acceptance/setupApache

And sort out the display of settings and log on failure()
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from 37088af to 1e8d102 Compare April 20, 2026 09:59
@phil-davis
Copy link
Copy Markdown
Contributor Author

webUIActivitySharingExternal/publicLinkSharingFederation.feature:8
passed a few commits ago, but on the retry:
https://github.com/owncloud/activity/actions/runs/24455471545/job/71454950256

When the public adds the public link to "%local_server%" as user "Brian" using the webUI        # WebUISharingContext::thePublicAddsThePublicLinkToUsingTheWebUI()
      element click intercepted: Element <a id="change-server" href="#">...</a> is not clickable at point (875, 73). Other element would receive the click: <a id="current-server" href="#">...</a>

But on the latest commit:
https://github.com/owncloud/activity/actions/runs/24660347349/job/72104570289?pr=1237

When the public adds the public link to "%local_server%" as user "Brian" using the webUI        # WebUISharingContext::thePublicAddsThePublicLinkToUsingTheWebUI()
      element not interactable
        (Session info: chrome=74.0.3729.169)
        (Driver info: chromedriver=74.0.3729.6 (255758eccf3d244491b8a1317aa76e1ce10d57e9-refs/branch-heads/3729@{#29}),platform=Linux 6.17.0-1010-azure x86_64) (WebDriver\Exception\ElementNotVisible)

Need to understand what is happening, and why it used to sometimes pass.

@phil-davis
Copy link
Copy Markdown
Contributor Author

owncloud log has multiple:

{"reqId":"0zVqhYrdijE2aSrEiTiq","level":3,"time":"2026-04-20T10:10:42+00:00","remoteAddr":"::1","user":"Brian","app":"PHP","method":"GET","url":"\/server\/apps\/activity\/api\/v2\/activity\/all?format=json&previews=true&since=0","message":"Array to string conversion at \/var\/www\/html\/server\/apps\/activity\/lib\/Formatter\/BaseFormatter.php#34"}

First investigate that.

@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch 2 times, most recently from c4bb180 to 48fe26b Compare April 21, 2026 01:41
@phil-davis phil-davis force-pushed the acceptance-tests-http-1 branch from 48fe26b to da09667 Compare April 21, 2026 01:47
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.

1 participant