skip puppet that requires token in smoke testing (#1580)

wechaty · Aug 31, 2018 · bb0eb5c · bb0eb5c
1 parent 9a9760f
commit bb0eb5c
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/tests/fixtures/smoke-testing.ts b/tests/fixtures/smoke-testing.ts
@@ -2,16 +2,23 @@
 
 import { Wechaty }    from 'wechaty'
 
-function getBotList () {
-  return [
+function getBotList (): Wechaty[] {
+  const botList = [
     new Wechaty({ puppet: 'wechaty-puppet-mock' }),
     new Wechaty({ puppet: 'wechaty-puppet-wechat4u' }),
     new Wechaty({ puppet: 'wechaty-puppet-puppeteer' }),
-    new Wechaty({
-      puppet: 'wechaty-puppet-padchat',
-      // we use WECHATY_PUPPET_PADCHAT_TOKEN environment variable at here.
-    }),
   ]
+
+  if (!process.env.TRAVIS_PULL_REQUEST) {
+    botList.push(
+      new Wechaty({
+        puppet: 'wechaty-puppet-padchat',
+        // we use WECHATY_PUPPET_PADCHAT_TOKEN environment variable at here.
+      })
+    )
+  }
+
+  return botList
 }
 
 async function main () {