Merge pull request #1035 from microsoft/Zerryth/QnAMakerServiceHostname

QnAMakerService correctly builds hostname URL & throws error w/o URL

Author: cleemullins

libraries/botframework-config/src/models/qnaMakerService.ts

@@ -39,8 +39,14 @@ export class QnaMakerService extends ConnectedService implements IQnAService {
      */
     constructor(source: IQnAService = {} as IQnAService) {
         super(source, ServiceTypes.QnA);
+        
+        if (!source.hostname) {
+            throw TypeError('QnAMakerService requires source parameter to have a hostname.')
+        }
 
-        this.hostname = new URL('/qnamaker', this.hostname).href;
+        if (!this.hostname.endsWith('/qnamaker')) {
+            this.hostname = new URL('/qnamaker', this.hostname).href;
+        }
     }
 
     // encrypt keys in service

libraries/botframework-config/tests/service.test.js

@@ -29,5 +29,22 @@ describe("Service Tests", () => {
         });
         assert.equal(qna.hostname, "https://myservice.azurewebsites.net/qnamaker");
     });
+
+    it("QnAMaker correctly does not add suffix when already hostname endind with \/qnamaker", () => {
+        let qnaWithQnamakerHostname = new bf.QnaMakerService({
+            hostname: "https://MyServiceThatDoesntNeedAppending.azurewebsites.net/qnamaker"
+        });
+        assert.equal(qnaWithQnamakerHostname.hostname, "https://MyServiceThatDoesntNeedAppending.azurewebsites.net/qnamaker");
+    });
+
+    it("QnAMaker should throw error without hostname", () => {
+        function createQnaWithoutHostname() {
+            new bf.QnaMakerService({});
+        }
+
+        let noHostnameError = new TypeError('QnAMakerService requires source parameter to have a hostname.')
+
+        assert.throws(() => createQnaWithoutHostname(), noHostnameError)
+    });
 });