Fix hostname in Apple configuration profile

[Nils160988]

Closes#19656

Signed-off-by: Nils Wittenbrink nilswittenbrink@web.de

[kesselb]

Index: apps/dav/tests/unit/Provisioning/Apple/AppleProvisioningPluginTest.php
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- apps/dav/tests/unit/Provisioning/Apple/AppleProvisioningPluginTest.php	(revision 90c3e3bbeb566f73952f7e4c66b9f98ec84cdde2)
+++ apps/dav/tests/unit/Provisioning/Apple/AppleProvisioningPluginTest.php	(date 1582992422173)
@@ -158,10 +158,9 @@
->method('getServerProtocol')
->wilLReturn('https');
- $this->sabreRequest->expects($this->at(1))
- ->method('getAbsoluteUrl')
- ->with()
- ->willReturn('https://nextcloud.tld/nextcloud/remote.php/dav/provisioning/apple-provisioning.mobileconfig');
+ $this->urlGenerator->expects($this->once())
+ ->method('getBaseUrl')
+ ->willReturn('https://nextcloud.tld/nextcloud');
$this->themingDefaults->expects($this->at(0))
->method('getName')

One test fails due the other method used to get the url. Above patch should mock getBaseUrl properly.

[Nils160988]

Fixed the test as @kesselb suggested

[georgehrke]

@Nils160988 Thx for the contribution!

[welcome]

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22
Most developers hang out on IRC. So join #nextcloud-dev on Freenode for a chat!

[Nils160988]

Thanks for merging :)
Backport to 18? Should not be a critical change I think

[georgehrke]

/backport to stable18

[georgehrke]

/backport to stable17

[backportbot-nextcloud]

backport to stable18 in #19884

[backportbot-nextcloud]

backport to stable17 in #19885