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