Fix tests for node 11

Author: marudor

.editorconfig

@@ -0,0 +1,7 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+indent_style = space
+indent_size = 2

.gitlab-ci.yml

@@ -1,4 +1,3 @@
-
 variables:
   DOCKER_DRIVER: overlay2
 
@@ -11,13 +10,13 @@ stages:
   artifacts:
     paths:
       - build/stage/marudor/libxmljs/releases/download
-  only: [tags]
 
 .glibc: &glibc
   <<: *baseBuild
   script:
     - npm install -g node-gyp node-pre-gyp
     - npm install --unsafe-perm --build-from-source
+    - npm run test
     - node-pre-gyp package 2>&1
 
 .muslc: &muslc
@@ -27,6 +26,7 @@ stages:
     - apk add python g++ make git
     - npm install --unsafe-perm -g node-gyp node-pre-gyp
     - npm install --unsafe-perm --build-from-source
+    - npm run test
     - node-pre-gyp package 2>&1
 
 node-11:

.prettierrc

@@ -0,0 +1,7 @@
+{
+    "singleQuote": true,
+    "tabWidth": 2,
+    "printWidth": 120,
+    "semi": true,
+    "trailingComma": "es5"
+}

package-lock.json

@@ -38,6 +38,7 @@
   "devDependencies": {
     "github-release-cli": "^1.0.1",
     "nodeunit": "~0.11.2",
+    "prettier": "^1.16.4",
     "semver": "~5.5.0"
   }
 }

package.json

@@ -2,108 +2,118 @@ var libxml = require('../index');
 var semver = require('semver');
 
 if (!global.gc) {
-    throw new Error('must run with --expose_gc for memory management tests');
+  throw new Error('must run with --expose_gc for memory management tests');
 }
 
-var nodeVersion = process.versions.node
-var shouldSkip = semver.satisfies(nodeVersion, '8.x || 9.x || 10.x')
+var nodeVersion = process.versions.node;
+var shouldSkip = semver.satisfies(nodeVersion, '8.x || 9.x || 10.x');
 
 module.exports.setUp = function(done) {
-    collectGarbage();
-    done();
+  collectGarbage();
+  done();
 };
 
 module.exports.inaccessible_document_freed = function(assert) {
-    var xml_memory_before_document = libxml.memoryUsage();
-    for (var i=0; i<10; i++) {
-      makeDocument();
-    }
+  var xml_memory_before_document = libxml.memoryUsage();
+  for (var i = 0; i < 10; i++) {
+    makeDocument();
+  }
+  process.nextTick(() => {
     collectGarbage();
     assert.ok(libxml.memoryUsage() <= xml_memory_before_document);
     assert.done();
+  });
 };
 
 module.exports.inaccessible_document_freed_when_node_freed = function(assert) {
-    if (shouldSkip) {
-        assert.done();
-        console.warn('skipping inaccessible_document_freed_when_node_freed');
-        return;
-    }
-
-    var xml_memory_before_document = libxml.memoryUsage();
-    var nodes = [];
-    for (var i=0; i<10; i++) {
-      nodes.push(makeDocument().get('//center'));
-    }
-    nodes = null;
+  if (shouldSkip) {
+    assert.done();
+    console.warn('skipping inaccessible_document_freed_when_node_freed');
+    return;
+  }
+
+  var xml_memory_before_document = libxml.memoryUsage();
+  var nodes = [];
+  for (var i = 0; i < 10; i++) {
+    nodes.push(makeDocument().get('//center'));
+  }
+  nodes = null;
+  process.nextTick(() => {
     collectGarbage();
     assert.ok(libxml.memoryUsage() <= xml_memory_before_document);
     assert.done();
+  });
 };
 
 module.exports.inaccessible_document_freed_after_middle_nodes_proxied = function(assert) {
-    var xml_memory_before_document = libxml.memoryUsage();
-    var doc = makeDocument();
-    var middle = doc.get('//middle');
-    var inner = doc.get('//inner');
-    inner.remove(); // v0.14.3, v0.15: proxy ref'd parent but can't unref when destroyed
-    doc = middle = inner = null;
+  var xml_memory_before_document = libxml.memoryUsage();
+  var doc = makeDocument();
+  var middle = doc.get('//middle');
+  var inner = doc.get('//inner');
+  inner.remove(); // v0.14.3, v0.15: proxy ref'd parent but can't unref when destroyed
+  doc = middle = inner = null;
+  process.nextTick(() => {
     collectGarbage();
     assert.ok(libxml.memoryUsage() <= xml_memory_before_document);
     assert.done();
+  });
 };
 
 module.exports.inaccessible_tree_freed = function(assert) {
-    if (shouldSkip) {
-        assert.done();
-        console.warn('skipping inaccessible_tree_freed');
-        return;
-    }
-
-    var doc = makeDocument();
-    var xml_memory_after_document = libxml.memoryUsage();
-    doc.get('//middle').remove();;
+  if (shouldSkip) {
+    assert.done();
+    console.warn('skipping inaccessible_tree_freed');
+    return;
+  }
+
+  var doc = makeDocument();
+  var xml_memory_after_document = libxml.memoryUsage();
+  doc.get('//middle').remove();
+  process.nextTick(() => {
     collectGarbage();
-    assert.ok(libxml.memoryUsage() < xml_memory_after_document);
+    assert.ok(libxml.memoryUsage() <= xml_memory_after_document);
     assert.done();
+  });
 };
 
 module.exports.namespace_list_freed = function(assert) {
-    var doc = makeDocument();
-    var el = doc.get('//center');
-    el.namespace("bar", null);
-    var xmlMemBefore = libxml.memoryUsage();
-    for (var i; i<1000; i++) {
-        el.namespaces()
-    }
+  var doc = makeDocument();
+  var el = doc.get('//center');
+  el.namespace('bar', null);
+  var xmlMemBefore = libxml.memoryUsage();
+  for (var i; i < 1000; i++) {
+    el.namespaces();
+  }
+  process.nextTick(() => {
     collectGarbage();
     assert.ok(libxml.memoryUsage() <= xmlMemBefore);
     assert.done();
+  });
 };
 
 function makeDocument() {
-    var body = "<?xml version='1.0' encoding='UTF-8'?>\n" +
-        "<root><outer><middle><inner><center/></inner></middle></outer></root>";
-    return libxml.parseXml(body);
+  var body =
+    "<?xml version='1.0' encoding='UTF-8'?>\n" +
+    '<root><outer><middle><inner><center/></inner></middle></outer></root>';
+  return libxml.parseXml(body);
 }
 
 function collectGarbage(minCycles, maxCycles) {
-    minCycles = minCycles || 3;
-    maxCycles = maxCycles || 10;
+  minCycles = minCycles || 3;
+  maxCycles = maxCycles || 10;
 
-    var cycles = 0;
-    var freedRss = 0;
-    var usage = process.memoryUsage();
-    do {
-        global.gc();
+  var cycles = 0;
+  var freedRss = 0;
+  var usage = process.memoryUsage();
+  do {
+    global.gc();
 
-        var usageAfterGc = process.memoryUsage();
-        freedRss = usage.rss - usageAfterGc.rss;
-        usage = usageAfterGc;
+    var usageAfterGc = process.memoryUsage();
+    freedRss = usage.rss - usageAfterGc.rss;
+    usage = usageAfterGc;
 
-        cycles++;
-    }
-    while ((cycles < minCycles) || ((freedRss !== 0) && (cycles < maxCycles)));
+    cycles++;
+  } while (cycles < minCycles || (freedRss !== 0 && cycles < maxCycles));
 
-    return usage;
+  return usage;
 }