Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

feat: remove sync hooks support

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2021-04-22 11:35:36 -04:00
parent 4e490f6058
commit 01956af43a
2 changed files with 1 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/plugins/hooks.js
View File

@@ -137,10 +137,6 @@ async function fireFilterHook(hook, hookList, params) {
payload => resolve(payload),
err => reject(err)
);
return;
}
if (returned) {
resolve(returned);
}
});
}

7
test/plugins.js
View File

@@ -47,7 +47,7 @@ describe('Plugins', () => {
});
});
it('should register and fire a filter hook having 3 methods, one returning a promise, one calling the callback and one just returning', async () => {
it('should register and fire a filter hook having 2 methods, one returning a promise, one calling the callback', async () => {
function method1(data, callback) {
data.foo += 1;
callback(null, data);
@@ -58,14 +58,9 @@ describe('Plugins', () => {
resolve(data);
});
}
function method3(data) {
data.foo += 1;
return data;
}
plugins.hooks.register('test-plugin', { hook: 'filter:test.hook2', method: method1 });
plugins.hooks.register('test-plugin', { hook: 'filter:test.hook2', method: method2 });
plugins.hooks.register('test-plugin', { hook: 'filter:test.hook2', method: method3 });
const data = await plugins.hooks.fire('filter:test.hook2', { foo: 1 });
assert.strictEqual(data.foo, 8);