Throw error when SP URL is null. Closes #6012

pnp · May 21, 2024 · d6e7ed1 · d6e7ed1
1 parent 884c943
commit d6e7ed1
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/src/m365/spo/commands/folder/folder-set.spec.ts b/src/m365/spo/commands/folder/folder-set.spec.ts
@@ -247,7 +247,7 @@ describe(commands.FOLDER_SET, () => {
 
   it('fails validation if the webUrl option is not valid', async () => {
     const actual = await command.validate({ options: { webUrl: 'abc', url: folderRelSiteUrl, name: newFolderName } }, commandInfo);
-    assert.strictEqual(actual, "abc is not a valid SharePoint Online site URL");
+    assert.strictEqual(actual, "'abc' is not a valid SharePoint Online site URL.");
   });
 
   it('passes validation when the url option specified', async () => {

diff --git a/src/m365/spo/commands/storageentity/storageentity-list.spec.ts b/src/m365/spo/commands/storageentity/storageentity-list.spec.ts
@@ -235,7 +235,7 @@ describe(commands.STORAGEENTITY_LIST, () => {
   it('rejects invalid SharePoint Online URL', async () => {
     const url = 'http://contoso';
     const actual = await command.validate({ options: { appCatalogUrl: url } }, commandInfo);
-    assert.strictEqual(actual, `${url} is not a valid SharePoint Online site URL`);
+    assert.strictEqual(actual, `'${url}' is not a valid SharePoint Online site URL.`);
   });
 
   it('fails validation when no SharePoint Online app catalog URL specified', async () => {

diff --git a/src/m365/spo/commands/storageentity/storageentity-remove.spec.ts b/src/m365/spo/commands/storageentity/storageentity-remove.spec.ts
@@ -194,7 +194,7 @@ describe(commands.STORAGEENTITY_REMOVE, () => {
   it('rejects invalid SharePoint Online URL', async () => {
     const url = 'http://contoso';
     const actual = await command.validate({ options: { appCatalogUrl: url, key: 'prop' } }, commandInfo);
-    assert.strictEqual(actual, `${url} is not a valid SharePoint Online site URL`);
+    assert.strictEqual(actual, `'${url}' is not a valid SharePoint Online site URL.`);
   });
 
   it('fails validation when no SharePoint Online app catalog URL specified', async () => {

diff --git a/src/m365/spo/commands/storageentity/storageentity-set.spec.ts b/src/m365/spo/commands/storageentity/storageentity-set.spec.ts
@@ -266,7 +266,7 @@ describe(commands.STORAGEENTITY_SET, () => {
   it('rejects invalid SharePoint Online URL', async () => {
     const url = 'http://contoso';
     const actual = await command.validate({ options: { appCatalogUrl: url, key: 'prop', value: 'val' } }, commandInfo);
-    assert.strictEqual(actual, `${url} is not a valid SharePoint Online site URL`);
+    assert.strictEqual(actual, `'${url}' is not a valid SharePoint Online site URL.`);
   });
 
   it('handles promise rejection', async () => {

diff --git a/src/utils/validation.ts b/src/utils/validation.ts
@@ -352,12 +352,12 @@ export const validation = {
   },
 
   isValidSharePointUrl(url: string): boolean | string {
-    if (!url) {
-      return false;
+    if (typeof url !== 'string') {
+      return 'SharePoint Online site URL must be a string.';
     }
 
     if (url.indexOf('https://') !== 0) {
-      return `${url} is not a valid SharePoint Online site URL`;
+      return `'${url}' is not a valid SharePoint Online site URL.`;
     }
     else {
       return true;