17415: Fixed port and hostname, added proper test

Arvados-DCO-1.1-Signed-off-by: Daniel Kutyła <daniel.kutyla@contractors.roche.com>

diff --git a/cypress/integration/collection.spec.js b/cypress/integration/collection.spec.js
index fa3ad8e6a3b2815ebea4c57dc77813e7fe29e3ba..b3f06d10c8f71f673fbf9902c7e52042443b0969 100644 (file)
--- a/cypress/integration/collection.spec.js
+++ b/cypress/integration/collection.spec.js
@@ -46,10 +46,13 @@ describe('Collection panel tests', function () {
             cy.get('[data-cy=context-menu]').contains('Open as network folder or S3 bucket').click();
             cy.get('[data-cy=download-button').click();
 
-            const filename = path.join(downloadsFolder, `${testCollection.name}.duck`)
+            const filename = path.join(downloadsFolder, `${testCollection.name}.duck`);
+            const expectedValue = '<?xmlversion="1.0"encoding="UTF-8"?><!DOCTYPEplistPUBLIC"-//Apple//DTDPLIST1.0//EN""http://www.apple.com/DTDs/PropertyList-1.0.dtd"><plistversion="1.0"><dict><key>Protocol</key><string>davs</string><key>Provider</key><string>iterateGmbH</string><key>UUID</key><string>zzzzz-4zz18-oehuaangyo2timv</string><key>Hostname</key><string>0.0.0.0</string><key>Port</key><string>40041</string><key>Username</key><string>collectionuser1</string><key>Labels</key><array></array></dict></plist>';
 
             cy.readFile(filename, { timeout: 15000 })
-                .should('have.length.gt', 50)
+                .then((str) => {
+                    expect(str.replaceAll(' ', '').replaceAll('\n', ''), expectedValue);
+                })
                 .then(() => cy.task('clearDownload', { filename }));
         });
     });

diff --git a/src/views-components/webdav-s3-dialog/webdav-s3-dialog.tsx b/src/views-components/webdav-s3-dialog/webdav-s3-dialog.tsx
index eec15c49ee842038ac4d3d75afd78f1b355ff6d9..c4d2ede00de2d3349145fda317c7ebf32a1c2306 100644 (file)
--- a/src/views-components/webdav-s3-dialog/webdav-s3-dialog.tsx
+++ b/src/views-components/webdav-s3-dialog/webdav-s3-dialog.tsx
@@ -59,9 +59,9 @@ const mountainduckTemplate = ({
       <key>UUID</key>
       <string>${uuid}</string>
       <key>Hostname</key>
-      <string>${collectionsUrl.replace('https://', ``).replace('*', uuid)}</string>
+      <string>${collectionsUrl.replace('https://', ``).replace('*', uuid).split(':')[0]}</string>
       <key>Port</key>
-      <string>443</string>
+      <string>${collectionsUrl.split(':')[2] || 443}</string>
       <key>Username</key>
       <string>${username}</string>
       <key>Labels</key>