Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-05-07 05:56:34 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: bug in api path existence test

Browse Source 
It was only checking for the path, but not ensuring that the method was also defined
This commit is contained in:
Julian Lam
2020-12-22 10:26:02 -05:00
parent 1cd2689cf6
commit 501a7b7774
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
test/api.js
View File

@@ -235,7 +235,8 @@ describe('API', async () => {
} }
const normalizedPath = pathObj.path.replace(/\/:([^\\/]+)/g, '/{$1}').replace(/\?/g, ''); const normalizedPath = pathObj.path.replace(/\/:([^\\/]+)/g, '/{$1}').replace(/\?/g, '');
assert(schema.paths.hasOwnProperty(normalizedPath)); assert(schema.paths.hasOwnProperty(normalizedPath), `${pathObj.path} is not defined in schema docs`);
assert(schema.paths[normalizedPath].hasOwnProperty(pathObj.method), `${pathObj.path} was found in schema docs, but ${pathObj.method.toUpperCase()} method is not defined`);
}); });
}); });
}); });