cartodb/Windshaft-cartodb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make tests to fail if unexpected keys are found in redis

Browse Source 
Make test fail if they try to remove unexistent redis key
This commit is contained in:
Raul Ochoa 2015-09-25 19:23:33 +02:00
parent af1e31fb29
commit 4f4dab143e
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
test/support/test_helper.js
View File

@ -65,8 +65,8 @@ function checkSurrogateKey(res, expectedKey) {
assert.equal(res.headers['surrogate-key'], expectedKey);
}
//global after to capture test suites that leave keys in redis
after(function(done) {
//global afterEach to capture test suites that leave keys in redis
afterEach(function(done) {
// restoring nock globally after each suite
nock.cleanAll();
@ -95,7 +95,7 @@ after(function(done) {
});
if (Object.keys(databasesTasks).length === 0) {
//assert.equal(keysFound.length, 0, 'Unexpected keys found in redis: ' + keysFound.join(', '));
assert.equal(keysFound.length, 0, 'Unexpected keys found in redis: ' + keysFound.join(', '));
done();
}
}
@ -127,7 +127,8 @@ function deleteRedisKeys(keysToDelete, callback) {
Object.keys(keysToDelete).forEach(function(k) {
var redisClient = redis.createClient(global.environment.redis.port);
redisClient.select(keysToDelete[k], function() {
redisClient.del(k, function() {
redisClient.del(k, function(err, deletedKeysCount) {
assert.notStrictEqual(deletedKeysCount, 0, 'No KEYS deleted for: [db=' + keysToDelete[k] + ']' + k);
taskDone(k);
});
});