lib,test: add freelist deprecation and test

As per the dicussion in #569, this patch issues a deprecation warning when freelist module is required. A test file for freelist is also added. PR-URL: #2176 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Brendan Ashworth <[email protected]>

Author: thefourtheye

lib/freelist.js

@@ -1,3 +1,6 @@
 'use strict';
 
+constutil=require('internal/util');
+
 module.exports=require('internal/freelist');
+util.printDeprecationMessage('freelist module is deprecated.');

test/parallel/test-freelist.js

@@ -0,0 +1,33 @@
+'use strict';
+
+// Flags: --expose-internals
+
+constassert=require('assert');
+constfreelist=require('freelist');
+constinternalFreelist=require('internal/freelist');
+
+assert.equal(typeoffreelist,'object');
+assert.equal(typeoffreelist.FreeList,'function');
+assert.strictEqual(freelist,internalFreelist);
+
+constflist1=newfreelist.FreeList('flist1',3,String);
+
+// Allocating when empty, should not change the list size
+varresult=flist1.alloc('test');
+assert.strictEqual(typeofresult,'string');
+assert.strictEqual(result,'test');
+assert.strictEqual(flist1.list.length,0);
+
+// Exhaust the free list
+assert(flist1.free('test1'));
+assert(flist1.free('test2'));
+assert(flist1.free('test3'));
+
+// Now it should not return 'true', as max length is exceeded
+assert.strictEqual(flist1.free('test4'),false);
+assert.strictEqual(flist1.free('test5'),false);
+
+// At this point 'alloc' should just return the stored values
+assert.strictEqual(flist1.alloc(),'test1');
+assert.strictEqual(flist1.alloc(),'test2');
+assert.strictEqual(flist1.alloc(),'test3');