From 1fd8ae954b0cd9c155638acb67f57e63a3980367 Mon Sep 17 00:00:00 2001 From: Hornwitser Date: Sun, 15 Sep 2019 17:09:53 +0200 Subject: [PATCH] Add missing files to coverage report Add missing.js test files that requires any files that currently does not have any unit tests defined for them. This should hopefully make it so that when unit tests are added for parts of these files the coverage will increase instead of decreasing. Note that this does not include CLI Tools/ as those are not testable. --- lib/npmPostinstall.js | 5 ++++- sharedPlugins/UPSdisplay/test/missing.js | 4 ++++ sharedPlugins/clusterioMod/test/missing.js | 5 +++++ sharedPlugins/globalChat/test/missing.js | 4 ++++ sharedPlugins/inventoryImports/test/missing.js | 4 ++++ sharedPlugins/remoteCommands/test/missing.js | 4 ++++ sharedPlugins/scenario_loader/test/missing.js | 4 ++++ sharedPlugins/serverManager/test/missing.js | 5 +++++ sharedPlugins/statisticsExporter/test/missing.js | 5 +++++ test/missing.js | 12 ++++++++++++ 10 files changed, 51 insertions(+), 1 deletion(-) create mode 100644 sharedPlugins/UPSdisplay/test/missing.js create mode 100644 sharedPlugins/clusterioMod/test/missing.js create mode 100644 sharedPlugins/globalChat/test/missing.js create mode 100644 sharedPlugins/inventoryImports/test/missing.js create mode 100644 sharedPlugins/remoteCommands/test/missing.js create mode 100644 sharedPlugins/scenario_loader/test/missing.js create mode 100644 sharedPlugins/serverManager/test/missing.js create mode 100644 sharedPlugins/statisticsExporter/test/missing.js create mode 100644 test/missing.js diff --git a/lib/npmPostinstall.js b/lib/npmPostinstall.js index 7e2e3c5e..a57c2f82 100644 --- a/lib/npmPostinstall.js +++ b/lib/npmPostinstall.js @@ -99,7 +99,6 @@ let downloadList = [ ] // initiate folder creations and downloads -processMkdirRecursive(processDownloadRecursive); function processMkdirRecursive(callback){ let item = directories.shift(); makeDirectory(item).then(()=>{ @@ -120,3 +119,7 @@ function processDownloadRecursive(callback){ } }); } + +if (module === require.main) { + processMkdirRecursive(processDownloadRecursive); +} diff --git a/sharedPlugins/UPSdisplay/test/missing.js b/sharedPlugins/UPSdisplay/test/missing.js new file mode 100644 index 00000000..923d82bf --- /dev/null +++ b/sharedPlugins/UPSdisplay/test/missing.js @@ -0,0 +1,4 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); diff --git a/sharedPlugins/clusterioMod/test/missing.js b/sharedPlugins/clusterioMod/test/missing.js new file mode 100644 index 00000000..9993114b --- /dev/null +++ b/sharedPlugins/clusterioMod/test/missing.js @@ -0,0 +1,5 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); +require('../masterPlugin'); diff --git a/sharedPlugins/globalChat/test/missing.js b/sharedPlugins/globalChat/test/missing.js new file mode 100644 index 00000000..923d82bf --- /dev/null +++ b/sharedPlugins/globalChat/test/missing.js @@ -0,0 +1,4 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); diff --git a/sharedPlugins/inventoryImports/test/missing.js b/sharedPlugins/inventoryImports/test/missing.js new file mode 100644 index 00000000..923d82bf --- /dev/null +++ b/sharedPlugins/inventoryImports/test/missing.js @@ -0,0 +1,4 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); diff --git a/sharedPlugins/remoteCommands/test/missing.js b/sharedPlugins/remoteCommands/test/missing.js new file mode 100644 index 00000000..923d82bf --- /dev/null +++ b/sharedPlugins/remoteCommands/test/missing.js @@ -0,0 +1,4 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); diff --git a/sharedPlugins/scenario_loader/test/missing.js b/sharedPlugins/scenario_loader/test/missing.js new file mode 100644 index 00000000..923d82bf --- /dev/null +++ b/sharedPlugins/scenario_loader/test/missing.js @@ -0,0 +1,4 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); diff --git a/sharedPlugins/serverManager/test/missing.js b/sharedPlugins/serverManager/test/missing.js new file mode 100644 index 00000000..9993114b --- /dev/null +++ b/sharedPlugins/serverManager/test/missing.js @@ -0,0 +1,5 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); +require('../masterPlugin'); diff --git a/sharedPlugins/statisticsExporter/test/missing.js b/sharedPlugins/statisticsExporter/test/missing.js new file mode 100644 index 00000000..9993114b --- /dev/null +++ b/sharedPlugins/statisticsExporter/test/missing.js @@ -0,0 +1,5 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('../index'); +require('../masterPlugin'); diff --git a/test/missing.js b/test/missing.js new file mode 100644 index 00000000..3a34f174 --- /dev/null +++ b/test/missing.js @@ -0,0 +1,12 @@ +// These are libraries that does not have any units test, they are +// required here in order to track coverage for them. + +require('lib/clusterTools'); +require('lib/dole_nn_base'); +require('lib/getInstances'); +require('lib/hash'); +require('lib/manager/instanceManager'); +require('lib/manager/modManager'); +require('lib/manager/pluginManager'); +require('lib/npmPostinstall.js'); +require('lib/typedef');