Skip to content

Comments

[gui-tests][full-ci] use XDG_CONFIG_HOME with custom location for AUT during squish tests#11316

Merged
saw-jan merged 3 commits into5from
tests/use-default-config
Oct 27, 2023
Merged

[gui-tests][full-ci] use XDG_CONFIG_HOME with custom location for AUT during squish tests#11316
saw-jan merged 3 commits into5from
tests/use-default-config

Conversation

@saw-jan
Copy link
Member

@saw-jan saw-jan commented Oct 26, 2023

--confdir option is deprecated and about to be removed in #11300.
So, squish tests makes use of XDG_CONFIG_HOME env to set custom config location (i.e. /tmp/owncloudtest/) for owncloud during the test execution

NOTE: env XDG_CONFIG_HOME is only available for the AUT context so not available for the squish and test script

NOTE: we have to come up with something for running tests locally with Squish.
WARN: NOT RECOMMENDED to run gui-tests on local machine (doing so will remove existing client configs)

TheOneRing
TheOneRing previously approved these changes Oct 26, 2023
@saw-jan saw-jan changed the title [gui-tests][full-ci] use default config dir in the tests [gui-tests][full-ci] use XDG_CONFIG_HOME with custom location for AUT during squish tests Oct 26, 2023
@TheOneRing
Copy link
Contributor

TheOneRing commented Oct 26, 2023

Have you thought about a different file format (json/yaml) so its well defined how to handle multiline env vars and escaping

@saw-jan
Copy link
Member Author

saw-jan commented Oct 26, 2023

Have you thought about a different file format that is less picky with escaping etc

AFAIK, ENVVARS= in suite.conf file requires a plain text file with KEY=VALUE pair per line. I don't know if json or yml file works

@saw-jan saw-jan force-pushed the tests/use-default-config branch from 83f37be to 5fd30c1 Compare October 26, 2023 12:49
@saw-jan saw-jan requested a review from TheOneRing October 26, 2023 12:54
@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

@saw-jan saw-jan merged commit 751d9b8 into 5 Oct 27, 2023
@delete-merged-branch delete-merged-branch bot deleted the tests/use-default-config branch October 27, 2023 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants