Feature/clear dialogflow interface

functions/.eslintrc.yml

@@ -13,4 +13,3 @@ rules:
   comma-dangle: 0
   mocha/no-exclusive-tests: error
   no-unused-expressions: 0
-  no-extend-native: ["error", { "exceptions": ["Array"] }]

functions/index.js

@@ -24,11 +24,11 @@ logAppStart(actionsMap);
 setup();
 
 /**
- * Action Endpoint
+ * Action of Google Endpoint
  *
  * @type {HttpsFunction}
  */
-exports.playMedia = functions.https.onRequest(bst.Logless.capture(functions.config().bespoken.key, function (req, res) {
+exports.assistant = functions.https.onRequest(bst.Logless.capture(functions.config().bespoken.key, function (req, res) {
   const app = new DialogflowApp({request: req, response: res});
 
   logRequest(app, req);

functions/package.json

@@ -11,8 +11,8 @@
   "scripts": {
     "lint": "eslint ./",
     "mocha": "mocha --opts ./tests/mocha.opts",
-    "start": "firebase serve --only functions:playMedia",
-    "deploy": "firebase deploy --only functions:playMedia",
+    "start": "firebase serve --only functions:assistant",
+    "deploy": "firebase deploy --only functions:assistant",
     "test": "npm run lint && npm run mocha"
   },
   "dependencies": {

functions/tests/index.spec.js

@@ -7,7 +7,7 @@ const {wait} = require('./_utils/wait');
 
 let index, configStub, adminInitStub, functions, admin;
 
-describe('playMedia', () => {
+describe('assistant', () => {
   let res;
 
   before(function () {
@@ -24,11 +24,11 @@ describe('playMedia', () => {
   });
 
   it('should be defined', () => {
-    expect(index.playMedia).to.be.ok;
+    expect(index.assistant).to.be.ok;
   });
 
   it('should store last used action', () => {
-    index.playMedia(buildIntentRequest({
+    index.assistant(buildIntentRequest({
       action: 'welcome',
       lastSeen: null,
     }), res);
@@ -39,7 +39,7 @@ describe('playMedia', () => {
     const action = 'on-definitely-uncovered-action';
     let warning = sinon.spy();
     index.__set__('warning', warning);
-    index.playMedia(buildIntentRequest({
+    index.assistant(buildIntentRequest({
       action,
       lastSeen: null,
     }), res);

functions/tests/integration/input-unknown.spec.js

@@ -22,7 +22,7 @@ describe('integration', () => {
   describe('input-unknown', () => {
     it('should 1st time', () => {
       const res = new MockResponse();
-      index.playMedia(buildIntentRequest({
+      index.assistant(buildIntentRequest({
         action: 'input-unknown',
       }), res);
       expect(res.statusCode).to.be.equal(200);
@@ -46,7 +46,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.true;
       expect(res.speech()).to.contain(
@@ -68,7 +68,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.false;
       expect(res.speech()).to.contain(strings.intents.unknown[2].speech);
@@ -85,7 +85,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.true;
       expect(res.speech()).to.contain(strings.intents.unknown[0].speech);

functions/tests/integration/no-input.spec.js

@@ -17,7 +17,7 @@ describe('integration', () => {
   describe('no-input', () => {
     it('should 1st time', () => {
       const res = new MockResponse();
-      index.playMedia(buildIntentRequest({
+      index.assistant(buildIntentRequest({
         action: 'no-input',
       }), res);
       expect(res.statusCode).to.be.equal(200);
@@ -41,7 +41,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.true;
       expect(res.speech()).to.contain(
@@ -60,7 +60,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.false;
       expect(res.speech()).to.contain(strings.intents.noInput[2].speech);
@@ -77,7 +77,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.true;
       expect(res.speech()).to.contain(strings.intents.noInput[0].speech);

functions/tests/integration/repeat.spec.js

@@ -32,7 +32,7 @@ describe('integration', () => {
           },
         },
       });
-      index.playMedia(req, res);
+      index.assistant(req, res);
       expect(res.statusCode).to.be.equal(200);
       expect(res.userResponse()).to.be.true;
       expect(res.speech()).to.contain('Where to go?');

functions/tests/integration/welcome.spec.js

@@ -20,7 +20,7 @@ describe('integration', () => {
   describe('welcome', () => {
     it('should handle for a new user', () => {
       const res = new MockResponse();
-      index.playMedia(buildIntentRequest({
+      index.assistant(buildIntentRequest({
         action: 'welcome',
         lastSeen: null,
       }), res);
@@ -30,7 +30,7 @@ describe('integration', () => {
 
     it('should handle for return user', () => {
       const res = new MockResponse();
-      index.playMedia(buildIntentRequest({
+      index.assistant(buildIntentRequest({
         action: 'welcome',
       }), res);
       expect(res.speech()).to.contain('Welcome to music at the Internet Archive.');