chat-platform 1.3.0 → 2.0.0

package/.eslintrc

@@ -1,17 +1,24 @@
 {
-    "globals": {
-        "Promise": true
-    },
-    "env": {
-        "node": true,
-        "es6": true
-    },
-    "extends": "eslint:recommended",
-    "rules": {
-        "quotes": ["error", "single"],
-        "no-useless-escape": 0
-    },
-    "parserOptions": {
-        "ecmaVersion": 8
-    }
-}
+  "globals": {
+    "Promise": true
+  },
+  "env": {
+    "node": true,
+    "es6": true,
+    "jest/globals": true
+  },
+  "extends": "eslint:recommended",
+  "rules": {
+    "quotes": [
+      "error",
+      "single"
+    ],
+    "no-useless-escape": 0
+  },
+  "parserOptions": {
+    "ecmaVersion": 8
+  },
+  "plugins": [
+    "jest"
+  ]
+}

package/__tests__/context-provider-memory.js

@@ -1,222 +1,181 @@
 const _ = require('underscore');
 const assert = require('chai').assert;
+const fs = require('fs');
+const os = require('os');
+
 const RED = require('../lib/red-stub')();
 const ContextProviders = require('../chat-context-factory');
-const { when } = require('../lib/utils');
 
 describe('Chat context provider memory', () => {
-
   const contextProviders = ContextProviders(RED);
+  const path = os.tmpdir();
+  const getProvider = () => contextProviders.getProvider('memory', {
+    chatbotId: 'my-bot'
+  });
 
-  beforeEach(() => {
-    const provider = contextProviders.getProvider('memory');
-    provider.reset();
+  beforeAll(async () => {
+    const provider = getProvider();
+    await provider.start();
   });
 
-  it('should create a context provider with some default params with chatId', () => {
+  afterAll(async () => {
+  });
+
+  it('should create a context provider with some default params with userId', async () => {
 
     assert.isTrue(contextProviders.hasProvider('memory'));
-    const provider = contextProviders.getProvider('memory');
+    const provider = getProvider();
     assert.isFunction(provider.getOrCreate);
     assert.isFunction(provider.get);
+    const chatContext = await provider.getOrCreateContext(43, { chatId: 43 });
+    await chatContext.set({ myVariable: 'initial value' })
+    assert.isFunction(chatContext.get);
+    assert.isFunction(chatContext.set);
+    assert.isFunction(chatContext.all);
+    assert.isFunction(chatContext.remove);
+    assert.isFunction(chatContext.clear);
+    const myVariable = await  chatContext.get('myVariable');
+
+    assert.equal(myVariable, 'initial value');
+  });
 
-    return when(provider.getOrCreate(43, null, { myVariable: 'initial value'}))
-      .then(chatContext => {
-        assert.isFunction(chatContext.get);
-        assert.isFunction(chatContext.set);
-        assert.isFunction(chatContext.all);
-        assert.isFunction(chatContext.remove);
-        assert.isFunction(chatContext.clear);
-        return chatContext.get('myVariable');
-      }).then(myVariable => {
-        assert.equal(myVariable, 'initial value');
-      });
+  it('should create a userId given the chatId and transport', async () => {
+    const provider = getProvider();
+
+    const userId = await provider.getOrCreateUserId('42', 'telegram');
+    assert.equal(userId, '42');
+    const context42 = await provider.getOrCreateContext('42', { });
+    await context42.set('myVar', 'guido');
+    await context42.set('anotherVar', 'guidone');
+    assert.equal(await context42.get('myVar'), 'guido');
+    assert.equal(await context42.get('anotherVar'), 'guidone');
+
+    const anotherUserId = await provider.getOrCreateUserId('43', 'facebook');
+    assert.equal(anotherUserId, '43');
+    const context43 = await provider.getOrCreateContext('43', { });
+    await context43.set('againAnotherVar', 'something');
+    assert.equal(await context43.get('againAnotherVar'), 'something');
+
+    await provider.mergeUserId('43', '42');
+
+    assert.equal(await provider.getUserId('42', 'telegram'), '42');
+    assert.equal(await provider.getUserId('43', 'facebook'), '42');
+    assert.equal(await provider.getChatId('42', 'telegram'), '42');
+    assert.equal(await provider.getChatId('42', 'facebook'), '43');
+
+    const newContext = await provider.getOrCreateContext('42');
+    assert.equal(await newContext.get('myVar'), 'guido');
+    assert.equal(await newContext.get('anotherVar'), 'guidone');
   });
 
-  it('should create a context provider with some default params with userId', () => {
+});
 
-    assert.isTrue(contextProviders.hasProvider('memory'));
-    const provider = contextProviders.getProvider('memory');
-    assert.isFunction(provider.getOrCreate);
-    assert.isFunction(provider.get);
-
-    return when(provider.getOrCreate(null, 44, { myVariable: 'initial value'}))
-      .then(chatContext => {
-        assert.isFunction(chatContext.get);
-        assert.isFunction(chatContext.set);
-        assert.isFunction(chatContext.all);
-        assert.isFunction(chatContext.remove);
-        assert.isFunction(chatContext.clear);
-        return chatContext.get('myVariable');
-      }).then(myVariable => {
-        assert.equal(myVariable, 'initial value');
-      });
-  });
+describe('Chat context provider memory', () => {
+  const contextProviders = ContextProviders(RED);
+  const getProvider = () => contextProviders.getProvider('memory', {
+    chatbotId: 'my-bot'
+  });
 
+  beforeAll(async () => {
+    const provider = getProvider();
+    await provider.start();
+  });
 
-  it('should set some value and then get and remove it with chatId', () => {
+  afterAll(async () => {
+  });
 
-    const provider = contextProviders.getProvider('memory');
+  beforeEach(async () => {
+    const provider = getProvider();
+    await provider.reset();
+  });
 
-    return when(provider.getOrCreate(42, null, {}))
-      .then(chatContext => chatContext.set('firstName', 'Guidone'))
-      .then(() => when(provider.get(42).get('firstName')))
-      .then(firstName => assert.equal(firstName, 'Guidone'))
-      .then(() => when(provider.get(42).remove('firstName')))
-      .then(() => when(provider.get(42).get('firstName')))
-      .then(
-        () => {},
-        firstName => {
-          assert.isUndefined(firstName);
-        });
+  it('should create a context provider with some default params with userId', async () => {
+    assert.isTrue(contextProviders.hasProvider('memory'));
+    const provider = getProvider();
+    assert.isFunction(provider.getOrCreate);
+    assert.isFunction(provider.get);
+    const chatContext = await provider.getOrCreateContext(44, { chatId: 43 });
+    await chatContext.set({ myVariable: 'initial value' })
+    assert.isFunction(chatContext.get);
+    assert.isFunction(chatContext.set);
+    assert.isFunction(chatContext.all);
+    assert.isFunction(chatContext.remove);
+    assert.isFunction(chatContext.clear);
+    const myVariable = await chatContext.get('myVariable');
+    assert.equal(myVariable, 'initial value');
   });
 
-  it('should set some value and then get and remove it with userId', () => {
+  it('should set some value and then get and remove it with userId', async () => {
+    const provider = getProvider();
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
+    await chatContext.set('firstName', 'Guidone');
+    let firstName = await chatContext.get('firstName');
+    assert.equal(firstName, 'Guidone');
+    await chatContext.remove('firstName')
+    firstName = await chatContext.get('firstName');
+    assert.isNull(firstName);
+  });
 
-    const provider = contextProviders.getProvider('memory');
+  it('should set some values and then get and remove it with userId', async () => {
+    const provider = getProvider();
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
+    await chatContext.set({firstName: 'Guido', lastName: 'Bellomo'});
+    const firstName = await chatContext.get('firstName');
+    assert.equal(firstName, 'Guido');
+    const lastName = await chatContext.get('lastName');
+    assert.equal(lastName, 'Bellomo');
+    const json = await chatContext.get('firstName', 'lastName');
+    assert.isObject(json);
+    assert.equal(json.firstName, 'Guido');
+    assert.equal(json.lastName, 'Bellomo');
+  });
 
-    return when(provider.getOrCreate(null, 43, {}))
-      .then(chatContext => chatContext.set('firstName', 'Guidone'))
-      .then(() => when(provider.get(null, 43).get('firstName')))
-      .then(firstName => assert.equal(firstName, 'Guidone'))
-      .then(() => when(provider.get(null, 43).remove('firstName')))
-      .then(() => when(provider.get(null, 43).get('firstName')))
-      .then(
-        () => {},
-        firstName => {
-          assert.isUndefined(firstName);
-        });
+  it('should set some values and get the dump with userId', async () => {
+    const provider = getProvider();
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
+    await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo', email: 'spam@gmail.com' })
+    const json = await chatContext.all();
+    assert.isObject(json);
+    assert.equal(json.firstName, 'Guido');
+    assert.equal(json.lastName, 'Bellomo');
+    assert.equal(json.email, 'spam@gmail.com');
   });
 
-  it('should set some values and then get and remove it with chatId', () => {
+  it('should set some values and remove all with userId', async () => {
+    const provider = getProvider();
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
+    await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo' });
+    await chatContext.clear();
+    const json = await chatContext.all();
+    assert.isObject(json);
+    assert.isTrue(_.isEmpty(json));
+  });
 
-    const provider = contextProviders.getProvider('memory');
+  it('should set some value the remove with multiple arguments', async () => {
+    const provider = getProvider();
+    const chatContext = await provider.getOrCreateContext(44, { chatId: 42 });
+    await chatContext.set({ firstName: 'Guidone', lastName: 'Bellomo', email: 'some@email' });
+    const firstName = await chatContext.get('firstName');
+    assert.equal(firstName, 'Guidone');
+    await chatContext.remove('firstName', 'lastName', 'email');
+    const json = await chatContext.all();
+    assert.isUndefined(json.firstName);
+    assert.isUndefined(json.lastName);
+    assert.isUndefined(json.email);
+  });
 
-    return when(provider.getOrCreate(42, null, {}))
-      .then(chatContext => chatContext.set({firstName: 'Guido', lastName: 'Bellomo'}))
-      .then(() => when(provider.get(42).get('firstName')))
-      .then(firstName => assert.equal(firstName, 'Guido'))
-      .then(() =>when(provider.get(42).get('lastName')))
-      .then(lastName => assert.equal(lastName, 'Bellomo'))
-      .then(() => when(provider.get(42).get('firstName', 'lastName')))
-      .then(json => {
-        assert.isObject(json);
-        assert.equal(json.firstName, 'Guido');
-        assert.equal(json.lastName, 'Bellomo');
-      });
-  });
-
-  it('should set some values and then get and remove it with userId', () => {
-
-    const provider = contextProviders.getProvider('memory');
-
-    return when(provider.getOrCreate(null, 43, {}))
-      .then(chatContext => chatContext.set({firstName: 'Guido', lastName: 'Bellomo'}))
-      .then(() => when(provider.get(null, 43).get('firstName')))
-      .then(firstName => assert.equal(firstName, 'Guido'))
-      .then(() =>when(provider.get(null, 43).get('lastName')))
-      .then(lastName => assert.equal(lastName, 'Bellomo'))
-      .then(() => when(provider.get(null, 43).get('firstName', 'lastName')))
-      .then(json => {
-        assert.isObject(json);
-        assert.equal(json.firstName, 'Guido');
-        assert.equal(json.lastName, 'Bellomo');
-      });
-  });
-
-  it('should set some values and get the dump with chatId', () => {
-
-    const provider = contextProviders.getProvider('memory');
-
-    return when(provider.getOrCreate(42, null, {}))
-      .then(chatContext => chatContext.set({firstName: 'Guido', lastName: 'Bellomo', email: 'spam@gmail.com'}))
-      .then(() => when(provider.get(42).all()))
-      .then(json => {
-        assert.isObject(json);
-        assert.equal(json.firstName, 'Guido');
-        assert.equal(json.lastName, 'Bellomo');
-        assert.equal(json.email, 'spam@gmail.com');
-      });
-  });
-
-  it('should set some values and get the dump with userId', () => {
-
-    const provider = contextProviders.getProvider('memory');
-
-    return when(provider.getOrCreate(null, 43, {}))
-      .then(chatContext => chatContext.set({firstName: 'Guido', lastName: 'Bellomo', email: 'spam@gmail.com'}))
-      .then(() => when(provider.get(null, 43).all()))
-      .then(json => {
-        assert.isObject(json);
-        assert.equal(json.firstName, 'Guido');
-        assert.equal(json.lastName, 'Bellomo');
-        assert.equal(json.email, 'spam@gmail.com');
-      });
-  });
-
-  it('should set some values and remove all with chatId', () => {
-
-    const provider = contextProviders.getProvider('memory');
-
-    return when(provider.getOrCreate(42, null, {}))
-      .then(chatContext => chatContext.set({firstName: 'Guido', lastName: 'Bellomo'}))
-      .then(() => when(provider.get(42).clear()))
-      .then(() => when(provider.get(42).all()))
-      .then(json => {
-        assert.isObject(json);
-        assert.isTrue(_.isEmpty(json));
-      });
-  });
-
-  it('should set some values and remove all with userId', () => {
-
-    const provider = contextProviders.getProvider('memory');
-
-    return when(provider.getOrCreate(null, 43, {}))
-      .then(chatContext => chatContext.set({firstName: 'Guido', lastName: 'Bellomo'}))
-      .then(() => when(provider.get(null, 43).clear()))
-      .then(() => when(provider.get(null, 43).all()))
-      .then(json => {
-        assert.isObject(json);
-        assert.isTrue(_.isEmpty(json));
-      });
-  });
-
-  it('should set some value the remove with multiple arguments', () => {
-
-    const provider = contextProviders.getProvider('memory');
-
-    return when(provider.getOrCreate(42, null, {}))
-      .then(chatContext =>chatContext.set({firstName: 'Guidone', lastName: 'Bellomo', email: 'some@email'}))
-      .then(() => when(provider.get(42).get('firstName')))
-      .then(firstName => assert.equal(firstName, 'Guidone'))
-      .then(() => when(provider.get(42).remove('firstName', 'lastName', 'email')))
-      .then(() => when(provider.get(42).all()))
-      .then(json => {
-        assert.isUndefined(json.firstName);
-        assert.isUndefined(json.lastName);
-        assert.isUndefined(json.email);
-      });
-  });
-
-  it('should set some value with chatId and userId', () => {
-    const provider = contextProviders.getProvider('memory');
-    return when(provider.getOrCreate(42, 44, { myVar: 123 }))
-      .then(chatContext => chatContext.set('firstName', 'Guidone'))
-      .then(() => when(provider.get(null, 44).get('firstName')))
-      .then(firstName => assert.equal(firstName, 'Guidone'))
-      .then(() => when(provider.get(null, 44, { myVar: 123 })))
-      .then(chatContext => {
-        assert.equal(chatContext.get('firstName'), 'Guidone');
-        assert.equal(chatContext.get('myVar'), 123);
-      })
-      .then(() => when(provider.get(null, 44).remove('firstName')))
-      .then(() => when(provider.get(null, 44).get('firstName')))
-      /*.then(
-        () => {},
-        firstName => {
-          assert.isUndefined(firstName);
-        });*/
+  it('should set some value with chatId and userId', async () => {
+    const provider = getProvider();
+    let chatContext = await provider.getOrCreateContext(44, { userId: 44, chatId: 42 });
+    await chatContext.set('firstName', 'Guidone');
+    let firstName = await chatContext.get('firstName');
+    assert.equal(firstName, 'Guidone');
+    chatContext = await provider.getOrCreateContext(44, { userId: 44, chatId: 42 });
+    assert.equal(await chatContext.get('userId'), 44);
+    assert.equal(await chatContext.get('firstName'), 'Guidone');
+    await chatContext.remove('firstName');
+    firstName = await chatContext.get('firstName');
+    assert.isNull(firstName);
   });
 
 });

package/__tests__/context-provider-sqlite.js

@@ -1,10 +1,10 @@
 const _ = require('underscore');
 const assert = require('chai').assert;
 const fs = require('fs');
+const os = require('os');
+
 const RED = require('../lib/red-stub')();
 const ContextProviders = require('../chat-context-factory');
-const { when } = require('../lib/utils');
-const os = require('os');
 
 const copyFile = (source, destination) => new Promise((resolve, reject) => {
   fs.copyFile(source, destination, error => {
@@ -29,7 +29,10 @@ const removeFile = file => new Promise((resolve, reject) => {
 describe('Chat context provider sqlite - missing file', () => {
   const contextProviders = ContextProviders(RED);
   const path = os.tmpdir();
-  const getProvider = () => contextProviders.getProvider('sqlite', { dbPath: path + '/mission-control.sqlite' });
+  const getProvider = () => contextProviders.getProvider('sqlite', {
+    dbPath: path + '/mission-control.sqlite',
+    chatbotId: 'my-bot'
+  });
 
   beforeAll(async () => {
     const provider = getProvider();
@@ -40,13 +43,13 @@ describe('Chat context provider sqlite - missing file', () => {
     await removeFile(path + '/mission-control.sqlite');
   });
 
-  it('should create a context provider with some default params with chatId', async () => {
+  it('should create a context provider with some default params with userId', async () => {
 
     assert.isTrue(contextProviders.hasProvider('sqlite'));
     const provider = getProvider();
     assert.isFunction(provider.getOrCreate);
     assert.isFunction(provider.get);
-    const chatContext = await provider.getOrCreate(43, null, { chatId: 43 });
+    const chatContext = await provider.getOrCreateContext(43, { chatId: 43 });
     await chatContext.set({ myVariable: 'initial value' })
     assert.isFunction(chatContext.get);
     assert.isFunction(chatContext.set);
@@ -58,13 +61,43 @@ describe('Chat context provider sqlite - missing file', () => {
     assert.equal(myVariable, 'initial value');
   });
 
+  it('should create a userId given the chatId and transport', async () => {
+    const provider = getProvider();
 
-});
+    const userId = await provider.getOrCreateUserId('42', 'telegram');
+    assert.equal(userId, '42');
+    const context42 = await provider.getOrCreateContext('42', { });
+    await context42.set('myVar', 'guido');
+    await context42.set('anotherVar', 'guidone');
+    assert.equal(await context42.get('myVar'), 'guido');
+    assert.equal(await context42.get('anotherVar'), 'guidone');
+
+    const anotherUserId = await provider.getOrCreateUserId('43', 'facebook');
+    assert.equal(anotherUserId, '43');
+    const context43 = await provider.getOrCreateContext('43', { });
+    await context43.set('againAnotherVar', 'something');
+    assert.equal(await context43.get('againAnotherVar'), 'something');
+
+    await provider.mergeUserId('43', '42');
 
+    assert.equal(await provider.getUserId('42', 'telegram'), '42');
+    assert.equal(await provider.getUserId('43', 'facebook'), '42');
+    assert.equal(await provider.getChatId('42', 'telegram'), '42');
+    assert.equal(await provider.getChatId('42', 'facebook'), '43');
+
+    const newContext = await provider.getOrCreateContext('42');
+    assert.equal(await newContext.get('myVar'), 'guido');
+    assert.equal(await newContext.get('anotherVar'), 'guidone');
+  });
+
+});
 
 describe('Chat context provider sqlite', () => {
   const contextProviders = ContextProviders(RED);
-  const getProvider = () => contextProviders.getProvider('sqlite', { dbPath: __dirname + '/dummy/mission-control.sqlite' });
+  const getProvider = () => contextProviders.getProvider('sqlite', {
+    dbPath: __dirname + '/dummy/mission-control.sqlite',
+    chatbotId: 'my-bot'
+  });
 
   beforeAll(async () => {
     await copyFile(__dirname + '/dummy/mission-control.backup', __dirname + '/dummy/mission-control.sqlite');
@@ -81,30 +114,12 @@ describe('Chat context provider sqlite', () => {
     await provider.reset();
   });
 
-  it('should create a context provider with some default params with chatId', async () => {
-
-    assert.isTrue(contextProviders.hasProvider('sqlite'));
-    const provider = getProvider();
-    assert.isFunction(provider.getOrCreate);
-    assert.isFunction(provider.get);
-    const chatContext = await provider.getOrCreate(43, null, { chatId: 43 });
-    await chatContext.set({ myVariable: 'initial value' })
-    assert.isFunction(chatContext.get);
-    assert.isFunction(chatContext.set);
-    assert.isFunction(chatContext.all);
-    assert.isFunction(chatContext.remove);
-    assert.isFunction(chatContext.clear);
-    const myVariable = await  chatContext.get('myVariable');
-
-    assert.equal(myVariable, 'initial value');
-  });
-
   it('should create a context provider with some default params with userId', async () => {
     assert.isTrue(contextProviders.hasProvider('sqlite'));
     const provider = getProvider();
     assert.isFunction(provider.getOrCreate);
     assert.isFunction(provider.get);
-    const chatContext = await provider.getOrCreate(null, 44, { userId: 44 });
+    const chatContext = await provider.getOrCreateContext(44, { chatId: 43 });
     await chatContext.set({ myVariable: 'initial value' })
     assert.isFunction(chatContext.get);
     assert.isFunction(chatContext.set);
@@ -115,21 +130,9 @@ describe('Chat context provider sqlite', () => {
     assert.equal(myVariable, 'initial value');
   });
 
-
-  it('should set some value and then get and remove it with chatId', async () => {
-    const provider = getProvider();
-    const chatContext = await provider.getOrCreate(42, null, { chatId: 42 });
-    await chatContext.set('firstName', 'Guidone');
-    let firstName = await chatContext.get('firstName');
-    assert.equal(firstName, 'Guidone')
-    await chatContext.remove('firstName')
-    firstName = await chatContext.get('firstName')
-    assert.isNull(firstName);
-  });
-
   it('should set some value and then get and remove it with userId', async () => {
     const provider = getProvider();
-    const chatContext = await provider.getOrCreate(null, 43, { userId: 43 });
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
     await chatContext.set('firstName', 'Guidone');
     let firstName = await chatContext.get('firstName');
     assert.equal(firstName, 'Guidone');
@@ -138,24 +141,9 @@ describe('Chat context provider sqlite', () => {
     assert.isNull(firstName);
   });
 
-
-  it('should set some values and then get and remove it with chatId', async () => {
-    const provider = getProvider();
-    const chatContext = await provider.getOrCreate(42, null, { chatId: 42 });
-    await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo' });
-    let firstName = await chatContext.get('firstName');
-    assert.equal(firstName, 'Guido');
-    let lastName = await chatContext.get('lastName');
-    assert.equal(lastName, 'Bellomo')
-    const json = await chatContext.get('firstName', 'lastName');
-    assert.isObject(json);
-    assert.equal(json.firstName, 'Guido');
-    assert.equal(json.lastName, 'Bellomo');
-  });
-
   it('should set some values and then get and remove it with userId', async () => {
     const provider = getProvider();
-    const chatContext = await provider.getOrCreate(null, 43, { userId: 43 });
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
     await chatContext.set({firstName: 'Guido', lastName: 'Bellomo'});
     const firstName = await chatContext.get('firstName');
     assert.equal(firstName, 'Guido');
@@ -167,20 +155,9 @@ describe('Chat context provider sqlite', () => {
     assert.equal(json.lastName, 'Bellomo');
   });
 
-  it('should set some values and get the dump with chatId', async () => {
-    const provider = getProvider();
-    const chatContext = await provider.getOrCreate(42, null, { chatId: 42 });
-    await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo', email: 'spam@gmail.com' });
-    const json = await chatContext.all();
-    assert.isObject(json);
-    assert.equal(json.firstName, 'Guido');
-    assert.equal(json.lastName, 'Bellomo');
-    assert.equal(json.email, 'spam@gmail.com');
-  });
-
   it('should set some values and get the dump with userId', async () => {
     const provider = getProvider();
-    const chatContext = await provider.getOrCreate(null, 43, { userId: 43 });
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
     await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo', email: 'spam@gmail.com' })
     const json = await chatContext.all();
     assert.isObject(json);
@@ -189,19 +166,9 @@ describe('Chat context provider sqlite', () => {
     assert.equal(json.email, 'spam@gmail.com');
   });
 
-  it('should set some values and remove all with chatId', async () => {
-    const provider = getProvider();
-    const chatContext = await provider.getOrCreate(42, null, { userId: 42 });
-    await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo' });
-    await chatContext.clear();
-    const json = await chatContext.all();
-    assert.isObject(json);
-    assert.isTrue(_.isEmpty(json));
-  });
-
   it('should set some values and remove all with userId', async () => {
     const provider = getProvider();
-    const chatContext = await provider.getOrCreate(null, 43, { userId: 43 });
+    const chatContext = await provider.getOrCreateContext(43, { userId: 43 });
     await chatContext.set({ firstName: 'Guido', lastName: 'Bellomo' });
     await chatContext.clear();
     const json = await chatContext.all();
@@ -211,7 +178,7 @@ describe('Chat context provider sqlite', () => {
 
   it('should set some value the remove with multiple arguments', async () => {
     const provider = getProvider();
-    const chatContext = await provider.getOrCreate(42, null, { chatId: 42 });
+    const chatContext = await provider.getOrCreateContext(44, { chatId: 42 });
     await chatContext.set({ firstName: 'Guidone', lastName: 'Bellomo', email: 'some@email' });
     const firstName = await chatContext.get('firstName');
     assert.equal(firstName, 'Guidone');
@@ -224,11 +191,11 @@ describe('Chat context provider sqlite', () => {
 
   it('should set some value with chatId and userId', async () => {
     const provider = getProvider();
-    let chatContext = await provider.getOrCreate(42, 44, { userId: 44, chatId: 42 });
+    let chatContext = await provider.getOrCreateContext(44, { userId: 44, chatId: 42 });
     await chatContext.set('firstName', 'Guidone');
     let firstName = await chatContext.get('firstName');
     assert.equal(firstName, 'Guidone');
-    chatContext = await provider.getOrCreate(null, 44, { userId: 44, chatId: 42 });
+    chatContext = await provider.getOrCreateContext(44, { userId: 44, chatId: 42 });
     assert.equal(await chatContext.get('userId'), 44);
     assert.equal(await chatContext.get('firstName'), 'Guidone');
     await chatContext.remove('firstName');
@@ -236,4 +203,4 @@ describe('Chat context provider sqlite', () => {
     assert.isNull(firstName);
   });
 
-});
+});