ti2 1.0.114 → 1.0.115

package/controllers/__tests__/bookings.js

@@ -106,10 +106,10 @@ describe('user: bookings controller', () => {
         token: userToken,
         payload,
       });
-      expect(plugins[0].searchHotelBooking).toHaveBeenCalled();
+      expect(plugins[0].searchItineraries).toHaveBeenCalled();
       expect(Array.isArray(bookings)).toBeTruthy();
-      expect(plugins[0].searchHotelBooking.mock.calls[0][0].payload).toEqual(payload);
-      expect(plugins[0].searchHotelBooking.mock.calls[0][0].token).toEqual(token);
+      expect(plugins[0].searchItineraries.mock.calls[0][0].payload).toEqual(payload);
+      expect(plugins[0].searchItineraries.mock.calls[0][0].token).toEqual(token);
     });
 
   });

package/controllers/__tests__/user.js

@@ -222,6 +222,19 @@ describe('user', () => {
     expect(plugins[0].validateToken.mock.calls[0][0].payload).toEqual({
       tokenHint: apiKey.split('-')[0],
     });
+    expect(plugins[0].validateToken.mock.calls[0][0].userId).toBe(userId);
+    expect(plugins[0].validateToken.mock.calls[0][0].hint).toBe(apiKey.split('-')[0]);
+  });
+  it('testing the user token should not infer a hint when no hint is provided', async () => {
+    const { valid } = await doApiPost({
+      url: `/${appName}/${userId}/validate`,
+      token: userKey,
+      payload: {},
+    });
+    expect(plugins[0].validateToken).toHaveBeenCalled();
+    expect(valid).toBe(true);
+    expect(plugins[0].validateToken.mock.calls[0][0].userId).toBe(userId);
+    expect(plugins[0].validateToken.mock.calls[0][0].hint).toBeUndefined();
   });
   it('should be able to delete an app settings', async () => {
     const returnValue = await doApiDelete({

package/controllers/app.js

@@ -261,6 +261,8 @@ const validateAppToken = plugins => async (req, res, next) => {
       axios,
       token,
       payload: req.body,
+      userId,
+      hint,
       requestId: req.requestId,
     });
     return res.json({ valid });

package/controllers/bookings.js

@@ -52,7 +52,8 @@ const bookingsSearch = plugins => async (req, res, next) => {
   try {
     const { app, token } = await getAppAndToken({ plugins, appKey, userId, hint });
     assert(app.searchItineraries || app.searchHotelBooking || app.searchBooking, `searchItineraries or searchHotelBooking or searchBooking is not available for ${appKey}`);
-    const search = (app.searchHotelBooking || app.searchBooking || app.searchItineraries).bind(app);
+    // Prefer the dedicated itinerary endpoint so date/filter payloads used by mining are preserved.
+    const search = (app.searchItineraries || app.searchHotelBooking || app.searchBooking).bind(app);
     const results = await search({
       axios,
       token,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ti2",
-  "version": "1.0.114",
+  "version": "1.0.115",
   "description": "Tourist Industry Exchange (TI2)",
   "main": "index.js",
   "scripts": {