Auto merge pull request #4 from atomist/sdm-pack-lifecycle

* Modify isChannel to always return true for any MSTeams channel id

Retains current behavior for Slack channel ids.

* Autofix: TypeScript header

[atomist:generated] [atomist:autofix=typescript_header]

Author: Danny Smith

lib/handlers/command/slack/LinkRepo.ts

@@ -1,5 +1,5 @@
 /*
- * Copyright © 2018 Atomist, Inc.
+ * Copyright © 2019 Atomist, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -115,7 +115,7 @@ export class LinkRepo implements HandleCommand {
             return ctx.messageClient.respond(err)
                 .then(() => Success, failure);
         }
-        if (isSlack(this.channelId) && !isChannel(this.channelId)) {
+        if (!isChannel(this.channelId)) {
             const err = "The Atomist Bot can only link repositories to public or private channels. " +
                 "Please try again in a public or private channel.";
             return ctx.messageClient.addressChannels(err, this.channelName)

lib/util/slack.ts

@@ -1,5 +1,5 @@
 /*
- * Copyright © 2018 Atomist, Inc.
+ * Copyright © 2019 Atomist, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,7 +22,10 @@
  * @return true if the channel is a public channel
  */
 export function isChannel(id: string): boolean {
-    return id.indexOf("C") === 0 || id.indexOf("G") === 0;
+    if (isSlack(id)) {
+        return id.indexOf("C") === 0 || id.indexOf("G") === 0;
+    }
+    return true;
 }
 
 /**
@@ -38,4 +41,4 @@ export function isChannel(id: string): boolean {
  */
 export function isSlack(id: string): boolean {
     return !id.includes("skype");
-}
+}

test/util/slack.test.ts

@@ -48,6 +48,10 @@ describe("slack", () => {
             });
         });
 
+        it("should accept any MS Team channel ID", () => {
+            assert(isChannel("12:[email protected]"));
+        });
+
     });
 
     describe("checkIsSlack", () => {