perf: use Map instead of Object for fields

Since the log object's properties get set, updated, deleted a lot
the object qualifies as hot object/hash table mode, making it less
performant. Maps fit in this situation as they enable the usage of
hash maps efficiently.

.eslintrc

@@ -1 +1,13 @@
-{ "extends": "scality" }
+{
+    "extends": "airbnb",
+    "env": {
+        "node": true,
+        "mocha": true
+    },
+    "rules": {
+        "indent": [2,4],
+        "no-multi-spaces": [2, { exceptions: { "SwitchCase": true, "CallExpression": true } } ],
+        "valid-jsdoc": 2,
+        "strict": 0
+    }
+}

.github/workflows/tests.yaml

@@ -1,31 +0,0 @@
-name: Tests
-
-on:
-  push:
-    branches-ignore:
-      - development/**
-      - q/*/**
-
-jobs:
-  tests:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkokut
-        uses: actions/checkout@v4
-      - name: Install deps
-        run: sudo apt-get update -q
-      - uses: actions/setup-node@v4
-        with:
-          node-version: '16'
-      - name: Install Yarn
-        run: npm install -g yarn
-      - name: install dependencies
-        run: yarn install --frozen-lockfile
-      - name: run lint
-        run: echo "linter is disabled temporarily ()" || yarn run --silent lint -- --max-warnings 0      
-      - name: run lint_md
-        run: yarn --silent lint_md
-      - name: run test
-        run: yarn test
-      - name: run coverage
-        run: yarn coverage

CONTRIBUTING.md

@@ -1,5 +1,53 @@
-# Contributing rules
+# Contributing to WereLogs
 
-Please follow the
+Contributing to WereLogs can take multiple shapes:
-[Contributing Guidelines](
+ * Reporting a bug
-https://github.com/scality/Guidelines/blob/master/CONTRIBUTING.md).
+ * Requesting/Suggesting/Discussing a feature
+ * Providing Code:
+   - a bug fix
+   - a new feature
+   - improving the documentation
+
+Each of these shapes of contribution must comply to a given set of rules, to
+facilitate cooperation.
+
+## Discussing the project
+
+This project is an opensource project, provided to you by other developers, and
+they too, are human. Thus, please try to explain your point of view when
+discussing a feature or bug, with examples, ascii art drawings, in an organized
+discourse. This should help everyone understand your point better, as well as
+it creates a context and cordial exchange between people. At any moment, try to
+keep your calm, and avoid flaming, as it never helps any discussion, and surely
+repulses some of the other contributors.
+
+Summarizing:
+ * Keep calm and stay cordial
+ * Explain your point, using the help of samples of code, ascii art diagrams
+ etc.
+
+## Reporting an Issue
+
+In order to report a bug, here are a few rules that can help both the
+reporter as well as the developer of the project:
+ * A bug report's name must shortly describe the user-side effect of the bug
+ * A bug report's body must provide the following informations:
+   - Describe in detail the effect of the bug in an expected usage situation
+   - Provide information about the setup in which werelogs is used, if you feel
+   it is relevant.
+Some additional useful information for a bug report can be:
+   - Running environment
+   - A sample of code that triggers the bug (REALLY helpful and appreciated)
+   - Describe any potential work-around found for the bug
+
+## Requesting or Suggesting a feature
+
+In order for any feature suggestion to be clear, please comply to the following
+basic rules:
+ * Issue title must be clear about the feature (a clear and semantic name might
+ help) and what value it adds to the project
+ * The description of the suggestion must provide insight into:
+   - What the feature provides
+   - At least one use-case of the feature (to show the value of it for the
+   project)
+   - If thought of, one suggestion as to how technically implement the feature

DESIGN.md

@@ -41,10 +41,9 @@ as it means that whenever the errors happen, depending on your log level, you
 might have already lost quite a bit of priceless information about the error
 encountered, and the code path the request went through. To address this, we
 offer multiple features:
-
+ * [Request ID namespacing](###request-id-namespacing)
-* [Request ID namespacing](###request-id-namespacing)
+ * [Request unit Logs](###request-unit-logs)
-* [Request unit Logs](###request-unit-logs)
+ 
-
 ### Request ID namespacing
 
 Usually, in networked systems, it is considered good practice to generate an
@@ -88,3 +87,5 @@ error (or higher level logging operation) is logged before the log context is
 freed, then the full set of buffered logging messages is freed, not taking
 any logging resources for the log entries not considered 'useless' by a given
 log level configuration.
+
+

LICENSE

@@ -1,191 +0,0 @@
-
-                                 Apache License
-                           Version 2.0, January 2004
-                        http://www.apache.org/licenses/
-
-   TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
-
-   1. Definitions.
-
-      "License" shall mean the terms and conditions for use, reproduction,
-      and distribution as defined by Sections 1 through 9 of this document.
-
-      "Licensor" shall mean the copyright owner or entity authorized by
-      the copyright owner that is granting the License.
-
-      "Legal Entity" shall mean the union of the acting entity and all
-      other entities that control, are controlled by, or are under common
-      control with that entity. For the purposes of this definition,
-      "control" means (i) the power, direct or indirect, to cause the
-      direction or management of such entity, whether by contract or
-      otherwise, or (ii) ownership of fifty percent (50%) or more of the
-      outstanding shares, or (iii) beneficial ownership of such entity.
-
-      "You" (or "Your") shall mean an individual or Legal Entity
-      exercising permissions granted by this License.
-
-      "Source" form shall mean the preferred form for making modifications,
-      including but not limited to software source code, documentation
-      source, and configuration files.
-
-      "Object" form shall mean any form resulting from mechanical
-      transformation or translation of a Source form, including but
-      not limited to compiled object code, generated documentation,
-      and conversions to other media types.
-
-      "Work" shall mean the work of authorship, whether in Source or
-      Object form, made available under the License, as indicated by a
-      copyright notice that is included in or attached to the work
-      (an example is provided in the Appendix below).
-
-      "Derivative Works" shall mean any work, whether in Source or Object
-      form, that is based on (or derived from) the Work and for which the
-      editorial revisions, annotations, elaborations, or other modifications
-      represent, as a whole, an original work of authorship. For the purposes
-      of this License, Derivative Works shall not include works that remain
-      separable from, or merely link (or bind by name) to the interfaces of,
-      the Work and Derivative Works thereof.
-
-      "Contribution" shall mean any work of authorship, including
-      the original version of the Work and any modifications or additions
-      to that Work or Derivative Works thereof, that is intentionally
-      submitted to Licensor for inclusion in the Work by the copyright owner
-      or by an individual or Legal Entity authorized to submit on behalf of
-      the copyright owner. For the purposes of this definition, "submitted"
-      means any form of electronic, verbal, or written communication sent
-      to the Licensor or its representatives, including but not limited to
-      communication on electronic mailing lists, source code control systems,
-      and issue tracking systems that are managed by, or on behalf of, the
-      Licensor for the purpose of discussing and improving the Work, but
-      excluding communication that is conspicuously marked or otherwise
-      designated in writing by the copyright owner as "Not a Contribution."
-
-      "Contributor" shall mean Licensor and any individual or Legal Entity
-      on behalf of whom a Contribution has been received by Licensor and
-      subsequently incorporated within the Work.
-
-   2. Grant of Copyright License. Subject to the terms and conditions of
-      this License, each Contributor hereby grants to You a perpetual,
-      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
-      copyright license to reproduce, prepare Derivative Works of,
-      publicly display, publicly perform, sublicense, and distribute the
-      Work and such Derivative Works in Source or Object form.
-
-   3. Grant of Patent License. Subject to the terms and conditions of
-      this License, each Contributor hereby grants to You a perpetual,
-      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
-      (except as stated in this section) patent license to make, have made,
-      use, offer to sell, sell, import, and otherwise transfer the Work,
-      where such license applies only to those patent claims licensable
-      by such Contributor that are necessarily infringed by their
-      Contribution(s) alone or by combination of their Contribution(s)
-      with the Work to which such Contribution(s) was submitted. If You
-      institute patent litigation against any entity (including a
-      cross-claim or counterclaim in a lawsuit) alleging that the Work
-      or a Contribution incorporated within the Work constitutes direct
-      or contributory patent infringement, then any patent licenses
-      granted to You under this License for that Work shall terminate
-      as of the date such litigation is filed.
-
-   4. Redistribution. You may reproduce and distribute copies of the
-      Work or Derivative Works thereof in any medium, with or without
-      modifications, and in Source or Object form, provided that You
-      meet the following conditions:
-
-      (a) You must give any other recipients of the Work or
-          Derivative Works a copy of this License; and
-
-      (b) You must cause any modified files to carry prominent notices
-          stating that You changed the files; and
-
-      (c) You must retain, in the Source form of any Derivative Works
-          that You distribute, all copyright, patent, trademark, and
-          attribution notices from the Source form of the Work,
-          excluding those notices that do not pertain to any part of
-          the Derivative Works; and
-
-      (d) If the Work includes a "NOTICE" text file as part of its
-          distribution, then any Derivative Works that You distribute must
-          include a readable copy of the attribution notices contained
-          within such NOTICE file, excluding those notices that do not
-          pertain to any part of the Derivative Works, in at least one
-          of the following places: within a NOTICE text file distributed
-          as part of the Derivative Works; within the Source form or
-          documentation, if provided along with the Derivative Works; or,
-          within a display generated by the Derivative Works, if and
-          wherever such third-party notices normally appear. The contents
-          of the NOTICE file are for informational purposes only and
-          do not modify the License. You may add Your own attribution
-          notices within Derivative Works that You distribute, alongside
-          or as an addendum to the NOTICE text from the Work, provided
-          that such additional attribution notices cannot be construed
-          as modifying the License.
-
-      You may add Your own copyright statement to Your modifications and
-      may provide additional or different license terms and conditions
-      for use, reproduction, or distribution of Your modifications, or
-      for any such Derivative Works as a whole, provided Your use,
-      reproduction, and distribution of the Work otherwise complies with
-      the conditions stated in this License.
-
-   5. Submission of Contributions. Unless You explicitly state otherwise,
-      any Contribution intentionally submitted for inclusion in the Work
-      by You to the Licensor shall be under the terms and conditions of
-      this License, without any additional terms or conditions.
-      Notwithstanding the above, nothing herein shall supersede or modify
-      the terms of any separate license agreement you may have executed
-      with Licensor regarding such Contributions.
-
-   6. Trademarks. This License does not grant permission to use the trade
-      names, trademarks, service marks, or product names of the Licensor,
-      except as required for reasonable and customary use in describing the
-      origin of the Work and reproducing the content of the NOTICE file.
-
-   7. Disclaimer of Warranty. Unless required by applicable law or
-      agreed to in writing, Licensor provides the Work (and each
-      Contributor provides its Contributions) on an "AS IS" BASIS,
-      WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
-      implied, including, without limitation, any warranties or conditions
-      of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
-      PARTICULAR PURPOSE. You are solely responsible for determining the
-      appropriateness of using or redistributing the Work and assume any
-      risks associated with Your exercise of permissions under this License.
-
-   8. Limitation of Liability. In no event and under no legal theory,
-      whether in tort (including negligence), contract, or otherwise,
-      unless required by applicable law (such as deliberate and grossly
-      negligent acts) or agreed to in writing, shall any Contributor be
-      liable to You for damages, including any direct, indirect, special,
-      incidental, or consequential damages of any character arising as a
-      result of this License or out of the use or inability to use the
-      Work (including but not limited to damages for loss of goodwill,
-      work stoppage, computer failure or malfunction, or any and all
-      other commercial damages or losses), even if such Contributor
-      has been advised of the possibility of such damages.
-
-   9. Accepting Warranty or Additional Liability. While redistributing
-      the Work or Derivative Works thereof, You may choose to offer,
-      and charge a fee for, acceptance of support, warranty, indemnity,
-      or other liability obligations and/or rights consistent with this
-      License. However, in accepting such obligations, You may act only
-      on Your own behalf and on Your sole responsibility, not on behalf
-      of any other Contributor, and only if You agree to indemnify,
-      defend, and hold each Contributor harmless for any liability
-      incurred by, or claims asserted against, such Contributor by reason
-      of your accepting any such warranty or additional liability.
-
-   END OF TERMS AND CONDITIONS
-
-   Copyright 2016 Scality
-
-   Licensed under the Apache License, Version 2.0 (the "License");
-   you may not use this file except in compliance with the License.
-   You may obtain a copy of the License at
-
-       http://www.apache.org/licenses/LICENSE-2.0
-
-   Unless required by applicable law or agreed to in writing, software
-   distributed under the License is distributed on an "AS IS" BASIS,
-   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-   See the License for the specific language governing permissions and
-   limitations under the License.

README.md

@@ -1,25 +1,16 @@
 # WereLogs
 
-[![Scality CI][badgepriv]](http://ci.ironmann.io/gh/scality/werelogs)
-
 This repository provides a NodeJS Library that aims to be an efficient logging
 library, reducing as much as possible the need to compute anything in NodeJS,
 and focusing on a simple I/O scheme. The goal here is to make the most of
 NodeJS's strengths, but relying on its I/O capacities, and avoiding any form of
 computation that is known to not be advantageous in Node.
 
-## Contributing
-
-In order to contribute, please follow the
-[Contributing Guidelines](
-https://github.com/scality/Guidelines/blob/master/CONTRIBUTING.md).
-
 ## Installing the Library
 
 In order to install WereLogs, you can use NPM with github's HTTP url, and save
 it in your own package.json:
-
+```
-```sh
 $> npm i --save scality/werelogs
 ```
 
@@ -71,9 +62,13 @@ import Logger from 'werelogs';
  * Object.
  *
  * This config options object contains a log level called 'level', a log
- * dumping threshold called 'dump'. The only unnecessary
+ * dumping threshold called 'dump', and an array of stream called 'streams'.
+ * The 'streams' option shall follow bunyan's configuration needs, as werelogs
+ * acts almost as a passthrough for this specific option. The only unnecessary
  * field is the 'level' of each individual stream, as werelogs is managing
- * that on its own.
+ * that on its own. For the reference about how to configure bunyan's streams,
+ * please refer to its repository's readme:
+ * https://github.com/trentm/node-bunyan
  *
  * All request loggers instantiated through this Logger will inherit its
  * configuration.
@@ -83,6 +78,19 @@ const log = new Logger(
     {
         level: 'debug',
         dump: 'error',
+        streams: [
+            // A simple, usual logging stream
+            { stream: process.stdout},
+            // A more complex logging stream provided by a bunyan plugin
+            // that will upload the logs directly to an ELK's logstash service
+            {
+                type: 'raw',
+                stream: require('bunyan-logstash').createStream({
+                    host: '127.0.0.1',
+                    port: 5505,
+                }),
+            }
+        ],
     }
 );
 
@@ -157,3 +165,8 @@ In order to find out the known issues, it is advised to take a look at the
 [project's github page](http://github.com/scality/werelogs). There, you should
 be able to find the issues, tagged with the releases they are impacting,
 whether they're open or closed.
+
+## Contributing
+
+The contributing rules for this project are defined in the associated
+CONTRIBUTING.md file.

circle.yml

@@ -0,0 +1,23 @@
+general:
+    branches:
+        ignore:
+            - /^ultron\/.*/ # Ignore ultron/* branches
+    artifacts:
+        - coverage/
+        - doc/
+
+machine:
+    node:
+        version: 4.1.0
+test:
+    override:
+        - npm run lint_md
+        - npm run lint
+        - npm run gendoc
+        - npm run coverage
+        # running ft_test packs werelogs and installs it + deps into
+        # tests/functional. Pack is like publishing werelogs in a local tgz
+        # archive that can be installed.
+        # This step shall ensure that no issue is encountered when installing
+        # the package, and allows to functionally test werelogs.
+        - npm run ft_test

index.d.ts

@@ -1,65 +0,0 @@
-interface WerelogsConfigOptions {
-    level?: 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal';
-    dump?: 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal';
-    streams?: object[];
-}
-
-declare class WerelogsConfig {
-    constructor(config?: WerelogsConfigOptions);
-    reset(): WerelogsConfig;
-    update(config: WerelogsConfig): WerelogsConfig;
-    logger: any;
-    level: 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal';
-    dump: 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal';
-    end: 'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal';
-}
-
-interface LogDictionary {
-    httpMethod?: string;
-    httpURL?: string;
-    [field: string]: any;
-}
-
-declare module 'werelogs' {
-    export class RequestLogger {
-        constructor(
-            logger: any,
-            logLevel: string,
-            dumpThreshold: string,
-            endLevel: string,
-            uids?: string|Array<string>
-        );
-        getUids(): Array<string>;
-        getSerializedUids(): string;
-        addDefaultFields(fields: LogDictionary): LogDictionary;
-        trace(msg: string, data?: LogDictionary): void;
-        debug(msg: string, data?: LogDictionary): void;
-        info(msg: string, data?: LogDictionary): void;
-        warn(msg: string, data?: LogDictionary): void;
-        error(msg: string, data?: LogDictionary): void;
-        fatal(msg: string, data?: LogDictionary): void;
-        end(msg: string, data?: LogDictionary): void;
-        errorEnd(msg: string, data?:LogDictionary): void;
-    }
-
-    export class Logger {
-        name: string;
-        constructor(name: string);
-        newRequestLogger(uids?: string|Array<string>): RequestLogger;
-        newRequestLoggerFromSerializedUids(uids: string): RequestLogger;
-        trace(msg: string, data?: LogDictionary): void;
-        debug(msg: string, data?: LogDictionary): void;
-        info(msg: string, data?: LogDictionary): void;
-        warn(msg: string, data?: LogDictionary): void;
-        error(msg: string, data?: LogDictionary): void;
-        fatal(msg: string, data?: LogDictionary): void;
-    }
-
-    export function configure(config: WerelogsConfigOptions): void;
-
-    export class API {
-        constructor(config: WerelogsConfigOptions);
-        reconfigure(config: WerelogsConfigOptions): void;
-        Logger: Logger;
-    }
-}

index.js

@@ -1,51 +1 @@
-const API = require('./lib/api.js');
+module.exports = require('./lib/Logger.js');
-const stderrUtils = require('./lib/stderrUtils');
-
-/*
- * For convenience purposes, we provide an already instanciated API; so that
- * old uses of the imported Logger class can be kept as-is. For quick logging,
- * this also provides a hassle-free way to log using werelogs.
- */
-const werelogs = new API();
-
-module.exports = {
-    Logger: werelogs.Logger,
-    configure: werelogs.reconfigure.bind(werelogs),
-    Werelogs: API,
-    /**
-     * Timestamp logs going to stderr
-     *
-     * @example <caption>Simplest usage</caption>
-     * ```
-     * const { stderrUtils } = require('werelogs');
-     * stderrUtils.catchAndTimestampStderr();
-     * ```
-     *
-     * @example <caption>Manage process exit</caption>
-     * ```
-     * const { stderrUtils } = require('werelogs');
-     * // set exitCode to null to keep process running on uncaughtException
-     * stderrUtils.catchAndTimestampStderr(undefined, null);
-     * // application init
-     * process.on('uncaughtException', (err) => {
-     *     // custom handling, close connections, files
-     *     this.worker.kill(); // or process.exit(1);
-     * });
-     * // Note you could use prependListener to execute your callback first
-     * // and then let stderrUtils exit the process.
-     * ```
-     *
-     * @example <caption>Custom listener</caption>
-     * ```
-     * const { stderrUtils } = require('werelogs');
-     * stderrUtils.catchAndTimestampWarning();
-     * // application init
-     * process.on('uncaughtException', (err, origin) => {
-     *     stderrUtils.printErrorWithTimestamp(err, origin);
-     *     // close and stop everything
-     *     process.exit(1);
-     * });
-     * ```
-     */
-    stderrUtils,
-};

lib/Config.js

@@ -1,8 +1,9 @@
+'use strict';
 
-// eslint-disable-line strict
+const bunyan = require('bunyan');
 
 const LogLevel = require('./LogLevel.js');
-const SimpleLogger = require('./SimpleLogger.js');
+
 /**
  * This class is the central point of the once-only configuration system for
  * werelogs. It is instanciated once to be exported, and all configuration
@@ -15,31 +16,10 @@ class Config {
      * This is the default constructor of the Config Object, and the only way
      * to instanciate it (with default parameters).
      *
-     * @param {object} conf             - A configuration object for werelogs.
-     * @param {string} conf.level       - The string name of the logging level
-     *                                    ('trace', 'debug', 'info', 'warn',
-     *                                    'error' and 'fatal' in order of
-     *                                    importance.)
-     * @param {string} conf.dump        - The string name of the log dumping
-     *                                    level ('trace', 'debug', 'info',
-     *                                    'warn', 'error' and 'fatal' in order
-     *                                    of importance.)
-     * @param {object[]} conf.streams   - The array of streams into which to
-     *                                    log. This is an Array of objects
-     *                                    which have a field named 'stream',
-     *                                    which is writeable.
-     *
      * @returns {undefined}
      */
-    constructor(conf) {
+    constructor() {
-        this.logLevel = 'info';
+        this.reset();
-        this.dumpThreshold = 'error';
-        this.endLevel = 'info';
-        this.streams = [{ level: 'trace', stream: process.stdout }];
-        this.simpleLogger = new SimpleLogger('werelogs', this.streams);
-        if (conf) {
-            this.update(conf);
-        }
     }
 
     /**
@@ -71,10 +51,12 @@ class Config {
      *                                  ('trace', 'debug', 'info', 'warn',
      *                                  'error' and 'fatal' in order of
      *                                  importance.)
-     * @param {object[]} config.streams - The array of streams into which to
+     * @param {object[]} config.streams   - The array of streams into which to log.
-     *                                    log. This is an Array of objects
+     *                                      Their configuration is directly
-     *                                    which have a field named 'stream',
+     *                                      related to the expected bunyan
-     *                                    which is writeable.
+     *                                      streams array, for compatibility
+     *                                      purposes (except that the 'level'
+     *                                      field is not accounted for)
      *
      * @see [Bunyan's documentation]{@link
      * https://github.com/trentm/node-bunyan/blob/master/README.md#streams} for
@@ -89,50 +71,37 @@ class Config {
 
         const checkedConfig = config || {};
 
-        if (Object.prototype.hasOwnProperty.call(checkedConfig, 'level')) {
+        if (checkedConfig.hasOwnProperty('level')) {
             LogLevel.throwIfInvalid(checkedConfig.level);
-        }
-        if (Object.prototype.hasOwnProperty.call(checkedConfig, 'dump')) {
-            LogLevel.throwIfInvalid(checkedConfig.dump);
-        }
-        // for check log level vs. log dump level
-        const newLogLevel = checkedConfig.level || this.logLevel;
-        const newLogDumpLevel = checkedConfig.dump || this.dumpThreshold;
-        if (newLogDumpLevel
-            && !LogLevel.shouldLog(newLogDumpLevel, newLogLevel)) {
-            throw new Error(
-                'Logging level should be at most logging dump level',
-            );
-        }
-
-        if (Object.prototype.hasOwnProperty.call(checkedConfig, 'level')) {
             this.logLevel = checkedConfig.level;
         }
 
-        if (Object.prototype.hasOwnProperty.call(checkedConfig, 'dump')) {
+        if (checkedConfig.hasOwnProperty('dump')) {
+            LogLevel.throwIfInvalid(checkedConfig.dump);
             this.dumpThreshold = checkedConfig.dump;
         }
 
-        if (Object.prototype.hasOwnProperty.call(checkedConfig, 'end')) {
+        if (checkedConfig.hasOwnProperty('end')) {
             LogLevel.throwIfInvalid(checkedConfig.end);
             this.endLevel = checkedConfig.end;
         }
 
-        if (Object.prototype.hasOwnProperty.call(checkedConfig, 'streams')) {
+        if (checkedConfig.hasOwnProperty('streams')) {
             if (!Array.isArray(checkedConfig.streams)) {
-                throw new TypeError('WereLogs config.streams must be an Array '
+                throw new Error('WereLogs config.streams must be an Array of Writeable Streams.');
-                                    + 'of Writeable Streams.');
             }
             if (!checkedConfig.streams.length) {
-                throw new Error('Werelogs config.streams must contain at '
+                throw new Error('Werelogs config.streams must contain at least one stream.');
-                                + 'least one stream.');
             }
-            this.streams = checkedConfig.streams.map(stream => {
+            this.streams = checkedConfig.streams.map((stream) => {
-                stream.level = 'trace'; // eslint-disable-line no-param-reassign
+                stream.level = 'trace';
                 return stream;
             });
 
-            this.simpleLogger = new SimpleLogger('werelogs', this.streams);
+            this.bLogger = bunyan.createLogger({
+                name: 'werelogs',
+                streams: this.streams,
+            });
         }
 
         return this;
@@ -148,7 +117,7 @@ class Config {
      *                          operations by the user code.
      */
     get logger() {
-        return this.simpleLogger;
+        return this.bLogger;
     }
 
     /**
@@ -188,4 +157,4 @@ class Config {
     }
 }
 
-module.exports = Config;
+module.exports = new Config();

lib/LogLevel.js

@@ -1,5 +1,4 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const logLevels = [
     'trace',
@@ -19,15 +18,10 @@ const logLevels = [
  *
  * @throw {Error}           A human-readable message that tells which log
  *                          levels are supported.
- * @throw {TypeError}       A human-readable message indicating that the
- *                          provided logging level was not a string
  *
  * @returns {undefined}
  */
 function throwIfInvalid(level) {
-    if (typeof level !== 'string') {
-        throw new TypeError('Logging level should be a string');
-    }
     if (logLevels.indexOf(level) === -1) {
         throw new RangeError(`Invalid logging level: ${level} is neither`
                              + ` ${logLevels.join(', ')}.`);

lib/Logger.js

@@ -1,32 +1,52 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const LogLevel = require('./LogLevel.js');
 const RequestLogger = require('./RequestLogger.js');
-const { unserializeUids } = require('./Utils.js');
+const unserializeUids = require('./Utils.js').unserializeUids;
 const Config = require('./Config.js');
 
 class Logger {
+
     /**
      * This is the constructor of the Logger class. It takes optional
      * configuration parameters, that allow to modify its behavior.
      *
-     * @param {Werelogs.Config} config  - An instanciated Werelogs Config object
+     * @param {string} name  - The name of the Logger. It can be found later on in
+     *                         the log entries.
      *
-     * @param {string} name  - The name of the Logger. It can be found later on
+     * @param {object} config         - A configuration object for werelogs.
-     *                         in the log entries.
+     * @param {string} config.level   - The string name of the logging level
+     *                                  ('trace', 'debug', 'info', 'warn',
+     *                                  'error' and 'fatal' in order of
+     *                                  importance.)
+     * @param {string} config.dump    - The string name of the log dumping level
+     *                                  ('trace', 'debug', 'info', 'warn',
+     *                                  'error' and 'fatal' in order of
+     *                                  importance.)
+     * @param {object[]} config.streams   - The array of streams into which to log.
+     *                                      Their configuration is directly
+     *                                      related to the expected bunyan
+     *                                      streams array, for compatibility
+     *                                      purposes (except that the 'level'
+     *                                      field is not accounted for)
+     *
+     * @see [Bunyan's documentation]{@link
+     * https://github.com/trentm/node-bunyan/blob/master/README.md#streams} for
+     * a more detailed description of the streams array configuration.
      *
      * @returns {undefined}
      */
-    constructor(config, name) {
+    constructor(name, config) {
-        if (!(config instanceof Config)) {
-            throw new TypeError('Invalid parameter Type for "config".');
-        }
-        if (!(typeof name === 'string' || name instanceof String)) {
-            throw new TypeError('Invalid parameter Type for "name".');
-        }
-        this.config = config;
         this.name = name;
+        Config.update(config);
+    }
+
+    setLevel(levelName) {
+        Config.update({ level: levelName });
+    }
+
+    setDumpThreshold(levelName) {
+        Config.update({ dump: levelName });
     }
 
     /**
@@ -38,10 +58,10 @@ class Logger {
      * @returns {RequestLogger}                 A Valid Request Logger
      */
     newRequestLogger(uids) {
-        const rLog = new RequestLogger(this.config.logger,
+        const rLog = new RequestLogger(Config.logger,
-            this.config.level, this.config.dump,
+                                       Config.level, Config.dump, Config.end,
-            this.config.end, uids);
+                                       uids);
-        rLog.addDefaultFields({ name: this.name });
+        rLog.addDefaultFields({name: this.name});
         return rLog;
     }
 
@@ -54,37 +74,36 @@ class Logger {
      * @returns {RequestLogger}         A Valid Request Logger
      */
     newRequestLoggerFromSerializedUids(serializedUids) {
-        const rLog = new RequestLogger(this.config.logger,
+        const rLog = new RequestLogger(Config.logger,
-            this.config.level, this.config.dump,
+                                       Config.level, Config.dump, Config.end,
-            this.config.end,
+                                       unserializeUids(serializedUids));
-            unserializeUids(serializedUids));
+        rLog.addDefaultFields({name: this.name});
-        rLog.addDefaultFields({ name: this.name });
         return rLog;
     }
 
     _doLog(levelName, msg, data) {
-        const sLogger = this.config.logger;
+        const bLogger = Config.logger;
-        const finalData = { name: this.name, time: Date.now() };
+        const finalData = new Map();
-        if (!LogLevel.shouldLog(levelName, this.config.level)) {
+        finalData.set('name', this.name);
+        if (!LogLevel.shouldLog(levelName, Config.level)) {
             return;
         }
         if (data !== undefined && typeof data !== 'object') {
-            sLogger.fatal(
+            bLogger.fatal(
                 {
-                    callparams: [msg, data],
+                    'callparams': [ msg, data ],
                 },
                 'Werelogs API was mis-used.'
-                + ' This development error should be fixed ASAP.',
+                + ' This development error should be fixed ASAP.');
-            );
             return;
         }
         if (data) {
-            Object.keys(data).forEach(k => {
+            Object.keys(data).forEach((k) => {
-                finalData[k] = data[k];
+                finalData.set(k, data[k]);
             });
         }
-        const args = [finalData, msg];
+        const args = [ finalData, msg ];
-        sLogger[levelName].apply(sLogger, args);
+        bLogger[levelName].apply(bLogger, args);
     }
 
     /**

lib/RequestLogger.js

@@ -1,15 +1,16 @@
+'use strict';
 
-// eslint-disable-line strict
+const assert = require('assert');
 
 const LogLevel = require('./LogLevel.js');
 const Utils = require('./Utils.js');
-
+const serializeUids = Utils.serializeUids;
-const { serializeUids, generateUid, objectCopy } = Utils;
+const generateUid = Utils.generateUid;
+const copyFields = Utils.copyFields;
 
 function ensureUidValidity(uid) {
     if (uid.indexOf(':') !== -1) {
-        throw new Error(`RequestLogger UID "${uid}" contains an illegal `
+        throw new Error(`RequestLogger UID "${uid}" contains an illegal character: ':'.`);
-                        + 'character: \':\'.');
     }
     return uid;
 }
@@ -17,13 +18,18 @@ function ensureUidValidity(uid) {
 class EndLogger {
     constructor(reqLogger) {
         this.logger = reqLogger;
-        this.fields = {};
+        this.fields = Object.create(null);
     }
 
     augmentedLog(level, msg, data) {
-        // We can alter current instance, as it won't be usable after this
+        assert.strictEqual(this.logger.elapsedTime, null, 'The logger\'s'
-        // call.
+                           + 'end() wrapper should not be called more than'
-        this.fields = objectCopy(this.fields, data || {});
+                           + ' once.');
+        if (data) {
+            Object.keys(data).forEach(k => {
+                this.fields[k] = data[k];
+            });
+        }
         return this.logger.log(level, msg, this.fields, true);
     }
 
@@ -42,7 +48,11 @@ class EndLogger {
      */
     addDefaultFields(fields) {
         const oldFields = this.fields;
-        this.fields = objectCopy({}, this.fields, fields);
+        if (fields) {
+            Object.keys(fields).forEach((k) => {
+                this.fields[k] = fields[k];
+            });
+        }
         return oldFields;
     }
 
@@ -138,6 +148,7 @@ class EndLogger {
  * request.
  */
 class RequestLogger {
+
     /**
      * Constructor of the WereLogs Request Logger.
      * This function takes a logger instance, a logging level, and a last
@@ -173,7 +184,7 @@ class RequestLogger {
      * @returns {undefined}
      */
     constructor(logger, logLevel, dumpThreshold, endLevel, uids) {
-        let uidList;
+        let uidList = undefined;
 
         if (!LogLevel.shouldLog(dumpThreshold, logLevel)) {
             throw new Error('Logging Dump level should be equal or'
@@ -183,18 +194,18 @@ class RequestLogger {
         if (uids !== undefined && Array.isArray(uids)) {
             uidList = uids.map(uid => ensureUidValidity(uid));
             uidList.push(generateUid());
-        } else if (uids !== undefined && typeof uids === 'string') {
+        } else if (uids !== undefined && typeof(uids) === 'string') {
-            uidList = [ensureUidValidity(uids)];
+            uidList = [ ensureUidValidity(uids) ];
         }
-        this.uids = uidList || [generateUid()];
+        this.uids = uidList || [ generateUid() ];
 
         this.entries = [];
-        this.fields = {};
+        this.fields = new Map();
         this.logLevel = logLevel;
         this.dumpThreshold = dumpThreshold;
         this.endLevel = endLevel;
         this.endLogger = new EndLogger(this);
-        this.sLogger = logger;
+        this.bLogger = logger;
 
         this.startTime = process.hrtime();
         this.elapsedTime = null;
@@ -244,7 +255,7 @@ class RequestLogger {
      */
     addDefaultFields(fields) {
         const oldFields = this.fields;
-        this.fields = objectCopy({}, this.fields, fields);
+        copyFields(this.fields, fields);
         return oldFields;
     }
 
@@ -365,6 +376,8 @@ class RequestLogger {
         if (msg === undefined && data === undefined) {
             return this.endLogger;
         }
+        assert.strictEqual(this.elapsedTime, null, 'The "end()" logging method '
+                           + 'should not be called more than once.');
         return this.log(this.endLevel, msg, data, true);
     }
 
@@ -381,6 +394,8 @@ class RequestLogger {
      * @returns {undefined}
      */
     errorEnd(msg, data) {
+        assert.strictEqual(this.elapsedTime, null, 'The "end()" logging method '
+                           + 'should not be called more than once.');
         return this.log('error', msg, data, true);
     }
 
@@ -421,36 +436,26 @@ class RequestLogger {
                 'Werelogs API was mis-used.'
                 + ' This development error should be fixed ASAP.',
                 {
-                    callparams: [msg, logFields],
+                    'callparams': [ msg, logFields ],
-                },
+                });
-            );
             return;
         }
-        const fields = objectCopy({}, this.fields, logFields || {});
+        let elapsedMs = 0;
         const endFlag = isEnd || false;
-
-        /*
-        * using Date.now() as it's faster than new Date(). logstash component
-        * uses this field to generate ISO 8601 timestamp
-        */
-        if (fields.time === undefined) {
-            fields.time = Date.now();
-        }
-
-        // eslint-disable-next-line camelcase
-        fields.req_id = serializeUids(this.uids);
         if (endFlag) {
-            if (this.elapsedTime !== null) {
-                // reset elapsedTime to avoid an infinite recursion
-                // while logging the error
-                this.elapsedTime = null;
-                this.error('RequestLogger.end() has been called more than once');
-            }
             this.elapsedTime = process.hrtime(this.startTime);
-            // eslint-disable-next-line camelcase
+            elapsedMs = this.elapsedTime[0] * 1000
-            fields.elapsed_ms = this.elapsedTime[0] * 1000
                 + this.elapsedTime[1] / 1000000;
         }
+
+        const fields = new Map();
+        fields.set('time', new Date().toJSON());
+        fields.set('req_id', serializeUids(this.uids));
+        fields.set('elapsed_ms', elapsedMs);
+        this.fields.forEach((v, k) => {
+            fields.set(k, v);
+        });
+        copyFields(fields, logFields || {});
         const logEntry = {
             level,
             fields,
@@ -459,7 +464,7 @@ class RequestLogger {
         this.entries.push(logEntry);
 
         if (LogLevel.shouldLog(level, this.dumpThreshold)) {
-            this.entries.forEach(entry => {
+            this.entries.forEach((entry) => {
                 this.doLogIO(entry);
             });
             this.entries = [];
@@ -484,28 +489,9 @@ class RequestLogger {
      * @returns {undefined}
      */
     doLogIO(logEntry) {
-        switch (logEntry.level) {
+        LogLevel.throwIfInvalid(logEntry.level);
-        case 'trace':
+
-            this.sLogger.trace(logEntry.fields, logEntry.msg);
+        this.bLogger[logEntry.level](logEntry.fields, logEntry.msg);
-            break;
-        case 'debug':
-            this.sLogger.debug(logEntry.fields, logEntry.msg);
-            break;
-        case 'info':
-            this.sLogger.info(logEntry.fields, logEntry.msg);
-            break;
-        case 'warn':
-            this.sLogger.warn(logEntry.fields, logEntry.msg);
-            break;
-        case 'error':
-            this.sLogger.error(logEntry.fields, logEntry.msg);
-            break;
-        case 'fatal':
-            this.sLogger.fatal(logEntry.fields, logEntry.msg);
-            break;
-        default:
-            throw new Error(`Unexpected log level: ${logEntry.level}`);
-        }
     }
 }
 

lib/SimpleLogger.js

@@ -1,112 +0,0 @@
-
-// eslint-disable-line strict
-
-const os = require('os');
-const safeJSONStringify = require('safe-json-stringify');
-const fastJSONStringify = require('fast-safe-stringify')
-
-function errorStackReplacer(key, value) {
-    if (value instanceof Error) {
-        return value.stack;
-    }
-    return value;
-}
-
-/*
-* This function safely stringifies JSON. If an exception occcurs (due to
-* circular references, exceptions thrown from object getters etc.), the module
-* safe-json-stringify is used to remove the offending property and return a
-* stringified object. This approach is ideal as it does not tax every log entry
-* to use the module and has been tested using Cosbench to verify that it doesn't
-* introduce any regression in performance.
-*/
-function safeStringify(obj) {
-    let str;
-    try {
-        // Try to stringify the object (fast version)
-        str = fastJSONStringify(obj, errorStackReplacer);
-    } catch (e) {
-        // fallback to remove circular object references or other exceptions
-        // eslint-disable-next-line no-param-reassign
-        obj.unsafeJSON = true;
-        return safeJSONStringify(obj, errorStackReplacer);
-    }
-    return str;
-}
-
-function isWriteableStream(s) {
-    if (!s.stream) {
-        return false;
-    }
-    // duck typing to check if the obect is a writeable stream
-    return s.stream.writable;
-}
-
-class SimpleLogger {
-    constructor(name, streams) {
-        this.name = name;
-        this.streams = [{ level: 'trace', stream: process.stdout }];
-        if (streams) {
-            if (!Array.isArray(streams)) {
-                throw new Error('Invalid streams. streams must be an array'
-                                + ' list of writeable streams');
-            }
-            /*
-            * This is for backwards compatibility. current config in projects
-            * create a bunyan-logstash stream which is not a compatible
-            * writeable stream. Any non-writable streams will be ignored.
-            * This will be changed to throw an error if non-writable stream is
-            * encountered.
-            */
-            this.streams = streams.filter(isWriteableStream);
-        }
-        this.hostname = os.hostname();
-    }
-
-    log(level, fields, message) {
-        let logFields;
-        let logMsg;
-        if (message === undefined && typeof fields === 'string') {
-            logMsg = fields;
-            logFields = {};
-        } else {
-            logMsg = message;
-            logFields = fields || {};
-        }
-        // TODO - Protect these fields from being overwritten
-        logFields.level = level;
-        logFields.message = logMsg;
-        logFields.hostname = this.hostname;
-        logFields.pid = process.pid;
-
-        const safeString = safeStringify(logFields);
-        this.streams.forEach(s => s.stream
-            .write(`${safeString}\n`));
-    }
-
-    info(fields, message) {
-        this.log('info', fields, message);
-    }
-
-    debug(fields, message) {
-        this.log('debug', fields, message);
-    }
-
-    trace(fields, message) {
-        this.log('trace', fields, message);
-    }
-
-    warn(fields, message) {
-        this.log('warn', fields, message);
-    }
-
-    error(fields, message) {
-        this.log('error', fields, message);
-    }
-
-    fatal(fields, message) {
-        this.log('fatal', fields, message);
-    }
-}
-
-module.exports = SimpleLogger;

lib/Utils.js

@@ -1,38 +1,18 @@
-
+'use strict';
-// eslint-disable-line strict
-
-/**
- * @constant
- * @type {String[]} - The lookup table to generate the UID
- */
-const lut = [];
-for (let i = 0; i < 256; i++) {
-    lut[i] = (i < 16 ? '0' : '') + (i).toString(16);
-}
 
 /**
  * This function generates a string uid.
  *
- * The base algorithm is taken from here: http://jcward.com/UUID.js
- * And is explained here: http://stackoverflow.com/a/21963136
- *
  * @returns {string}    An hexadecimal string representation of an unique
  *                      id made of 80 bits.of entropy.
  */
 function generateUid() {
-    const d0 = Math.random() * 0xffffffff | 0;
+    function s4() {
-    const d1 = Math.random() * 0xffffffff | 0;
+        return Math.floor((1 + Math.random()) * 0x10000)
-    const d2 = Math.random() * 0xffffffff | 0;
+            .toString(16)
-    return lut[d0 & 0xff]
+            .substring(1);
-        + lut[d0 >> 8 & 0xff]
+    }
-        + lut[d0 >> 16 & 0xff]
+    return s4() + s4() + s4() + s4() + s4();
-        + lut[d1 & 0xff]
-        + lut[d1 >> 8 & 0xff]
-        + lut[d1 >> 16 & 0x0f | 0x40]
-        + lut[d2 & 0x3f | 0x80]
-        + lut[d2 >> 8 & 0xff]
-        + lut[d2 >> 16 & 0xff]
-        + lut[d2 >> 24 & 0xff];
 }
 
 /**
@@ -52,41 +32,30 @@ function serializeUids(uidList) {
  * This function unserializes an array of UIDs from a string and returns the
  * generated Array.
  *
- * @param {string} stringdata - The string data of the serialized array of UIDs
+ * @param {string} stringdata   - The string data of the serialized array of UIDs
  *
- * @returns {string[]}        - The unserialized array of string UIDs
+ * @returns {string[]}            The unserialized array of string UIDs
  */
 function unserializeUids(stringdata) {
     return stringdata.split(':');
 }
 
 /**
-* This function copies the properties from the source object to the target
+* This function copies the properties from the source object to the fields map
-* object.
+* @param {map} fields - Map of log entry fields
-*
+* @param {object} source - object to copy from
-* @param {...object} target - object to be copied to
+* @returns {map} - map object containing the copied fields
-* @returns {object} - target object
 */
-function objectCopy(target) {
+function copyFields(fields, source) {
-    const result = target;
+    Object.keys(source).forEach(f => {
-    /* eslint-disable prefer-rest-params */
+        fields.set(f, source[f]);
-    const nb = arguments.length;
+    });
-    for (let i = 1; i < nb; i++) {
+    return fields;
-        const source = arguments[i];
-        const keys = Object.keys(source);
-        const keysNb = keys.length;
-        for (let j = 0; j < keysNb; j++) {
-            const key = keys[j];
-            result[key] = source[key];
-        }
-    }
-    /* eslint-enable prefer-rest-params */
-    return result;
 }
 
 module.exports = {
     generateUid,
     serializeUids,
     unserializeUids,
-    objectCopy,
+    copyFields,
 };

lib/api.js

@@ -1,72 +0,0 @@
-
-// eslint-disable-line strict
-
-const Config = require('./Config.js');
-const Logger = require('./Logger.js');
-
-class API {
-    /**
-     * This is the constructor of the Logger class. It takes optional
-     * configuration parameters, that allow to modify its behavior.
-     *
-     * @param {object} config         - A configuration object for werelogs.
-     * @param {string} config.level   - The name of the logging level ('trace',
-     *                                  'debug', 'info', 'warn', 'error' and
-     *                                  'fatal' in order of importance.)
-     * @param {string} config.dump    - The name of the log dumping level
-     *                                  ('trace', 'debug', 'info', 'warn',
-     *                                  'error' and 'fatal' in order of
-     *                                  importance.)
-     * @param {object[]} config.streams - The streams into which to log. This
-     *                                    is an Array of objects which have a
-     *                                    field named 'stream', which is
-     *                                    writeable.
-     */
-    constructor(config) {
-        this.config = new Config(config);
-        this.preboundLogger = Logger.bind(null, this.config);
-    }
-
-    /**
-     * This is a thunk function that allows reconfiguring the streams and log
-     * levels of all Logger and future RequestLogger objects. Note that
-     * existing RequestLogger will live their lifespan retaining the old
-     * configuration.
-     * If the provided configuration is erroneous, the function may throw
-     * exceptions depending on the detected configuration error. Please see the
-     * Config class's documentation about that.
-     *
-     * @throws {TypeError}  - One of the provided arguments is not of the
-     *                        expected type
-     * @throws {RangeError} - The logging level provided is not part of the
-     *                        supported logging levels
-     * @throws {Error}      - A human-readable message providing details about
-     *                        a logic error due to the input parameters
-     *                        provided.
-     *
-     * @param {object} config         - A configuration object for werelogs.
-     * @param {string} config.level   - The name of the logging level ('trace',
-     *                                  'debug', 'info', 'warn', 'error' and
-     *                                  'fatal' in order of importance.)
-     * @param {string} config.dump    - The name of the log dumping level
-     *                                  ('trace', 'debug', 'info', 'warn',
-     *                                  'error' and 'fatal' in order of
-     *                                  importance.)
-     * @param {object[]} config.streams - The streams into which to log. This
-     *                                    is an Array of objects which have a
-     *                                    field named 'stream', which is
-     *                                    writeable.
-     *
-     * @returns {undefined}
-     *
-     */
-    reconfigure(config) {
-        this.config.update(config);
-    }
-
-    get Logger() {
-        return this.preboundLogger;
-    }
-}
-
-module.exports = API;

lib/stderrUtils.js

@@ -1,106 +0,0 @@
-/**
- * @returns {string} a timestamp in ISO format YYYY-MM-DDThh:mm:ss.sssZ
- */
-const defaultTimestamp = () => new Date().toISOString();
-
-/**
- * Prints on stderr a timestamp, the origin and the error
- *
- * If no other instructions are needed on uncaughtException,
- * consider using `catchAndTimestampStderr` directly.
- *
- * @example
- * process.on('uncaughtException', (err, origin) => {
- *   printErrorWithTimestamp(err, origin);
- *   // server.close();
- *   // file.close();
- *   process.nextTick(() => process.exit(1));
- * });
- * // Don't forget to timestamp warning
- * catchAndTimestampWarning();
- * @param {Error} err see process event uncaughtException
- * @param {uncaughtException|unhandledRejection} origin see process event
- * @param {string} [date=`defaultTimestamp()`] Date to print
- * @returns {boolean} see process.stderr.write
- */
-function printErrorWithTimestamp(
-    err, origin, date = defaultTimestamp(),
-) {
-    return process.stderr.write(`${date}: ${origin}:\n${err.stack}\n`);
-}
-
-/**
- * Prefer using `catchAndTimestampStderr` instead of this function.
- *
- * Adds listener for uncaughtException to print with timestamp.
- *
- * If you want to manage the end of the process, you can set exitCode to null.
- * Or use `printErrorWithTimestamp` in your own uncaughtException listener.
- *
- * @param {Function} [dateFct=`defaultTimestamp`] Fct returning a formatted date
- * @param {*} [exitCode=1] On uncaughtException, if not null, `process.exit`
- *                         will be called with this value
- * @returns {undefined}
- */
-function catchAndTimestampUncaughtException(
-    dateFct = defaultTimestamp, exitCode = 1,
-) {
-    process.on('uncaughtException', (err, origin) => {
-        printErrorWithTimestamp(err, origin, dateFct());
-        if (exitCode !== null) {
-            process.nextTick(() => process.exit(exitCode));
-        }
-    });
-}
-
-/**
- * Forces the use of `--trace-warnings` and adds a date in warning.detail
- * The warning will be printed by the default `onWarning`
- *
- * @param {string} [dateFct=`defaultTimestamp`] Fct returning a formatted date
- * @returns {undefined}
- */
-function catchAndTimestampWarning(dateFct = defaultTimestamp) {
-    process.traceProcessWarnings = true;
-    // must be executed first, before the default `onWarning`
-    process.prependListener('warning', warning => {
-        if (warning.detail) {
-            // eslint-disable-next-line no-param-reassign
-            warning.detail += `\nAbove Warning Date: ${dateFct()}`;
-        } else {
-            // eslint-disable-next-line no-param-reassign
-            warning.detail = `Above Warning Date: ${dateFct()}`;
-        }
-    });
-}
-
-/**
- * Adds listener for uncaughtException and warning to print them with timestamp.
- *
- * If you want to manage the end of the process, you can set exitCode to null.
- * Or use `printErrorWithTimestamp` in your own uncaughtException listener.
- *
- * @example
- * const { stderrUtils } = require('werelogs');
- * // first instruction in your index.js or entrypoint
- * stderrUtils.catchAndTimestampStderr();
- *
- * @param {Function} [dateFct=`defaultTimestamp`] Fct returning a formatted date
- * @param {*} [exitCode=1] On uncaughtException, if not null, `process.exit`
- *                         will be called with this value
- * @returns {undefined}
- */
-function catchAndTimestampStderr(
-    dateFct = defaultTimestamp, exitCode = 1,
-) {
-    catchAndTimestampUncaughtException(dateFct, exitCode);
-    catchAndTimestampWarning(dateFct);
-}
-
-module.exports = {
-    defaultTimestamp,
-    printErrorWithTimestamp,
-    catchAndTimestampUncaughtException,
-    catchAndTimestampWarning,
-    catchAndTimestampStderr,
-};

licenses/SAFE-JSON-STRINGIFY

@@ -1,21 +0,0 @@
-The MIT License (MIT)
-
-Copyright (c) 2014 E-conomic
-
-
-Permission is hereby granted, free of charge, to any person obtaining a copy of
-this software and associated documentation files (the “Software”), to deal in
-the Software without restriction, including without limitation the rights to
-use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
-the Software, and to permit persons to whom the Software is furnished to do so,
-subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED “AS IS”, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
-FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
-COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
-IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
-CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package.json

@@ -1,18 +1,15 @@
 {
   "name": "werelogs",
-  "engines": {
+  "version": "1.0.1-beta2",
-    "node": ">=10"
-  },
-  "version": "8.1.5",
   "description": "An efficient raw JSON logging library aimed at micro-services architectures.",
   "main": "index.js",
   "scripts": {
     "gendoc": "jsdoc $(git ls-files 'lib/*.js') -d doc",
     "lint": "eslint $(git ls-files '*.js')",
-    "lint_md": "markdownlint $(git ls-files '*.md')",
+    "lint_md": "mdlint $(git ls-files '*.md')",
     "test": "mocha tests/unit/",
-    "ft_test": "(npm pack && cp werelogs-*.tgz tests/functional && cd tests/functional && cp -R ../../node_modules/ node_modules/ && npm install werelogs-*.tgz && ./node_modules/.bin/mocha . multi-modules/ && rm -rf tests/functional/node_modules tests/functional/werelogs-*.tgz tests/functional/*lock*)",
+    "ft_test": "rm -rf tests/functional/node_modules && npm pack && cp -R node_modules werelogs-*.tgz tests/functional && cd tests/functional && npm install werelogs-*.tgz && ./node_modules/.bin/mocha . multi-modules/ && cd -",
-    "coverage": "nyc ./node_modules/.bin/_mocha tests/unit"
+    "coverage": "istanbul cover ./node_modules/.bin/_mocha tests/unit"
   },
   "repository": {
     "type": "git",
@@ -26,27 +23,23 @@
     "library",
     "JSON"
   ],
-  "author": "Giorgio Regni",
+  "author": "David Pineau",
-  "license": "Apache-2.0",
+  "license": "ISC",
   "bugs": {
     "url": "https://github.com/scality/werelogs/issues"
   },
   "homepage": "https://github.com/scality/werelogs#readme",
   "dependencies": {
-    "fast-safe-stringify": "^2.1.1",
+    "bunyan": "^1.5.1"
-    "safe-json-stringify": "^1.2.0"
   },
   "devDependencies": {
-    "eslint": "^7.32.0",
+    "eslint": "^1.10.1",
-    "eslint-config-airbnb": "^18.2.1",
+    "eslint-config-airbnb": "^1.0.2",
-    "eslint-config-scality": "git+https://git.yourcmc.ru/vitalif/zenko-eslint-config-scality.git",
+    "eslint-plugin-react": "^3.10.0",
-    "eslint-plugin-import": "^2.22.1",
+    "istanbul": "^1.0.0-alpha",
-    "eslint-plugin-jsx-a11y": "^6.4.1",
+    "istanbul-api": "==1.0.0-alpha.9",
-    "eslint-plugin-react": "^7.26.0",
+    "jsdoc": "^3.4.0",
-    "eslint-plugin-react-hooks": "^4.2.0",
+    "mdlint": "^0.1.0",
-    "jsdoc": "^3.4.3",
+    "mocha": "^2.3.4"
-    "markdownlint-cli": "^0.27.1",
-    "mocha": ">=3.1.2",
-    "nyc": "^15.1.0"
   }
 }

tests/Utils.js

@@ -1,49 +1,53 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const assert = require('assert');
 
 const LogLevel = require('../lib/LogLevel.js');
 
 class DummyLogger {
+
     constructor() {
         this.ops = [];
         this.counts = {
-            trace: 0,
+            'trace': 0,
-            debug: 0,
+            'debug': 0,
-            info: 0,
+            'info': 0,
-            warn: 0,
+            'warn': 0,
-            error: 0,
+            'error': 0,
-            fatal: 0,
+            'fatal': 0,
         };
     }
 
     trace(obj, msg) {
-        this._doLog('trace', obj, msg);
+        this.doLog('trace', obj, msg);
     }
 
     debug(obj, msg) {
-        this._doLog('debug', obj, msg);
+        this.doLog('debug', obj, msg);
     }
 
     info(obj, msg) {
-        this._doLog('info', obj, msg);
+        this.doLog('info', obj, msg);
     }
 
     warn(obj, msg) {
-        this._doLog('warn', obj, msg);
+        this.doLog('warn', obj, msg);
     }
 
     error(obj, msg) {
-        this._doLog('error', obj, msg);
+        this.doLog('error', obj, msg);
     }
 
     fatal(obj, msg) {
-        this._doLog('fatal', obj, msg);
+        this.doLog('fatal', obj, msg);
     }
 
-    _doLog(level, obj, msg) {
+    doLog(level, obj, msg) {
-        this.ops.push([level, [obj, msg]]);
+        const fields = Object.create(null);
+        if (obj instanceof Map) {
+            obj.forEach((v, k) => fields[k] = v);
+        }
+        this.ops.push([ level, [ fields, msg]]);
         this.counts[level] += 1;
     }
 }
@@ -57,8 +61,7 @@ function computeBehavior(filterLevel, logLevel, testLevel) {
 
     return {
         value,
-        msg: `Expected ${logLevel} to be called ${value} times with `
+        'msg': `Expected ${logLevel} to be called ${value} times with filter level ${filterLevel}.`,
-             + `filter level ${filterLevel}.`,
     };
 }
 
@@ -69,14 +72,12 @@ function genericFilterGenerator(filterLevel, testLevel, createLogger) {
         const logger = createLogger(dummyLogger, filterLevel);
 
         switch (testLevel) {
-        /* eslint-disable no-multi-spaces */
         case 'trace': logger.trace('test trace'); break;
         case 'debug': logger.debug('test debug'); break;
         case 'info':  logger.info('test info');  break;
         case 'warn':  logger.warn('test warn');  break;
         case 'error': logger.error('test error'); break;
         case 'fatal': logger.fatal('test fatal'); break;
-        /* eslint-enable no-multi-spaces */
         default:
             done(new Error('Unexpected testLevel name: ', testLevel));
         }
@@ -107,10 +108,9 @@ function loggingMisuseGenerator(test, createLogger) {
                 logger.info.apply(logger, test.args);
             },
             Error,
-            `Werelogs should not throw with ${test.desc}`,
+            'Werelogs should not throw with ' + test.desc);
-        );
         assert(dummyLogger.ops[0][0], 'fatal',
-            'Expected the Module Logger to have logged a fatal message.');
+               'Expected the Module Logger to have logged a fatal message.');
         done();
     };
 }

tests/functional/basic_usage.js

@@ -1,116 +1,84 @@
-
-// eslint-disable-line strict
-
 const assert = require('assert');
-const { PassThrough } = require('stream');
+const bunyan = require('bunyan');
 
-const pass = new PassThrough();
+const Logger = require('werelogs');
-
+const logBuffer = new bunyan.RingBuffer({ limit: 1 });
-const werelogs = require('werelogs'); // eslint-disable-line
-
-// With PassThrough, SimpleLogger can use it as Writeable stream and all the
-// data being written can be read into a variable
-const logBuffer = {
-    records: [],
-};
-pass.on('data', data => {
-    logBuffer.records.push(data.toString());
-});
-
-werelogs.configure({
-    level: 'info',
-    dump: 'error',
-    streams: [{
-        stream: pass,
-        type: 'raw',
-    }],
-});
 
 function createModuleLogger() {
-    return new werelogs.Logger('FT-test');
+    return new Logger('FT-test', {
+        level: 'info',
+        dump: 'error',
+        streams: [{
+            type: 'raw',
+            stream: logBuffer,
+        }],
+    });
 }
 
 function checkFields(fields) {
-    const record = JSON.parse(logBuffer.records[0].trim());
+    Object.keys(fields).forEach((k) => {
-    Object.keys(fields).forEach(k => {
+        if (fields.hasOwnProperty(k)) {
-        if (Object.prototype.hasOwnProperty.call(fields, k)) {
+            assert.deepStrictEqual(logBuffer.records[0][k], fields[k]);
-            assert.deepStrictEqual(record[k], fields[k]);
         }
     });
 }
 
-function parseLogEntry() {
-    return JSON.parse(logBuffer.records[0].trim());
-}
-
 describe('Werelogs is usable as a dependency', () => {
     describe('Usage of the ModuleLogger', () => {
-        afterEach(() => {
+        it('Should be able to create a logger', (done) => {
-            logBuffer.records = [];
-        });
-        it('Should be able to create a logger', done => {
             assert.doesNotThrow(
                 createModuleLogger,
                 Error,
-                'Werelogs threw an exception trying to create a ModuleLogger.',
+                'Werelogs threw an exception trying to create a ModuleLogger.'
             );
             done();
         });
 
-        it('Should be able to log a simple message', done => {
+        it('Should be able to log a simple message', (done) => {
             const logger = createModuleLogger();
             const msg = 'This is a simple message';
             logger.info(msg);
-            assert.strictEqual(parseLogEntry().message, msg);
+            assert.strictEqual(logBuffer.records[0].msg, msg);
             done();
         });
 
-        it('Should be able to log a message and additional fields', done => {
+        it('Should be able to log a message and additional fields', (done) => {
             const logger = createModuleLogger();
             const msg = 'This is a message with added fields';
-            const fields = {
+            const fields = { errorCode: 9, description: 'TestError', options: { dump: false } };
-                errorCode: 9,
-                description: 'TestError',
-                options: { dump: false },
-            };
             logger.info(msg, fields);
-            assert.strictEqual(parseLogEntry().message, msg);
+            assert.strictEqual(logBuffer.records[0].msg, msg);
             checkFields(fields);
             done();
         });
     });
 
     describe('Usage of the RequestLogger', () => {
-        afterEach(() => {
+        it('Should be able to create a logger', (done) => {
-            logBuffer.records = [];
-        });
-        it('Should be able to create a logger', done => {
             assert.doesNotThrow(
-                () => createModuleLogger().newRequestLogger(),
+                () => {
+                    return createModuleLogger().newRequestLogger();
+                },
                 Error,
-                'Werelogs threw an exception trying to create a ModuleLogger.',
+                'Werelogs threw an exception trying to create a ModuleLogger.'
             );
             done();
         });
 
-        it('Should be able to log a simple message', done => {
+        it('Should be able to log a simple message', (done) => {
             const logger = createModuleLogger().newRequestLogger();
             const msg = 'This is a simple message';
             logger.info(msg);
-            assert.strictEqual(parseLogEntry().message, msg);
+            assert.strictEqual(logBuffer.records[0].msg, msg);
             done();
         });
 
-        it('Should be able to log a message and additional fields', done => {
+        it('Should be able to log a message and additional fields', (done) => {
             const logger = createModuleLogger().newRequestLogger();
             const msg = 'This is a message with added fields';
-            const fields = {
+            const fields = { errorCode: 9, description: 'TestError', options: { dump: false } };
-                errorCode: 9,
-                description: 'TestError',
-                options: { dump: false },
-            };
             logger.info(msg, fields);
-            assert.strictEqual(parseLogEntry().message, msg);
+            assert.strictEqual(logBuffer.records[0].msg, msg);
             checkFields(fields);
             done();
         });

tests/functional/multi-modules/index.js

@@ -1,57 +1,46 @@
 const assert = require('assert');
-const { PassThrough } = require('stream');
 
-const Werelogs = require('werelogs');  // eslint-disable-line
+const bunyan = require('bunyan');
+
+const Werelogs = require('werelogs');
 const modules = [
     require('./module1.js'),
     require('./module2.js'),
     require('./module3.js'),
 ];
 
-const pass = new PassThrough();
+describe('Config is shared and unique within one program', () => {
-
+    it('should find all log entries in the RingBuffer with the right module name', (done) => {
-const logBuffer = {
+        const rBuffer = new bunyan.RingBuffer({ limit: 15 });
-    records: [],
+        const log = new Werelogs('test-index', {
-};
-pass.on('data', data => {
-    logBuffer.records.push(JSON.parse(data.toString().trim()));
-});
-
-describe('Config is shared and unique within one API', () => {
-    it('should find all log entries in the RingBuffer with the right '
-       + 'module name', done => {
-        Werelogs.configure({
             level: 'debug',
             dump: 'fatal',
             streams: [{
                 type: 'raw',
-                stream: pass,
+                stream: rBuffer,
             }],
         });
-        const log = new Werelogs.Logger('test-index');
+        modules.forEach((mod) => { mod(); });
-        modules.forEach(mod => { mod(); });
         log.warn('Logging as warn');
         const rLog = log.newRequestLogger();
         rLog.info('Logging request as info');
-        /* eslint-disable max-len */
+        assert.deepStrictEqual(rBuffer.records.length, 5, 'Expected to see 5 log entries in the ring buffer.');
-        assert.deepStrictEqual(logBuffer.records.length, 5, 'Expected to see 5 log entries in the ring buffer.');
+        assert.deepStrictEqual(rBuffer.records[0].msg, 'Logging as info');
-        assert.deepStrictEqual(logBuffer.records[0].message, 'Logging as info');
+        assert.deepStrictEqual(rBuffer.records[0].name, 'test-mod1');
-        assert.deepStrictEqual(logBuffer.records[0].name, 'test-mod1');
+        assert.deepStrictEqual(rBuffer.records[0].level, 30);
-        assert.deepStrictEqual(logBuffer.records[0].level, 'info');
+        assert.deepStrictEqual(rBuffer.records[1].msg, 'Logging as debug');
-        assert.deepStrictEqual(logBuffer.records[1].message, 'Logging as debug');
+        assert.deepStrictEqual(rBuffer.records[1].name, 'test-mod2');
-        assert.deepStrictEqual(logBuffer.records[1].name, 'test-mod2');
+        assert.deepStrictEqual(rBuffer.records[1].level, 20);
-        assert.deepStrictEqual(logBuffer.records[1].level, 'debug');
+        assert.deepStrictEqual(rBuffer.records[2].msg, 'Logging as error');
-        assert.deepStrictEqual(logBuffer.records[2].message, 'Logging as error');
+        assert.deepStrictEqual(rBuffer.records[2].name, 'test-mod3');
-        assert.deepStrictEqual(logBuffer.records[2].name, 'test-mod3');
+        assert.deepStrictEqual(rBuffer.records[2].level, 50);
-        assert.deepStrictEqual(logBuffer.records[2].level, 'error');
+        assert.deepStrictEqual(rBuffer.records[3].msg, 'Logging as warn');
-        assert.deepStrictEqual(logBuffer.records[3].message, 'Logging as warn');
+        assert.deepStrictEqual(rBuffer.records[3].name, 'test-index');
-        assert.deepStrictEqual(logBuffer.records[3].name, 'test-index');
+        assert.deepStrictEqual(rBuffer.records[3].level, 40);
-        assert.deepStrictEqual(logBuffer.records[3].level, 'warn');
+        assert.deepStrictEqual(rBuffer.records[4].msg, 'Logging request as info');
-        assert.deepStrictEqual(logBuffer.records[4].message, 'Logging request as info');
+        assert.deepStrictEqual(rBuffer.records[4].name, 'test-index');
-        assert.deepStrictEqual(logBuffer.records[4].name, 'test-index');
+        assert.deepStrictEqual(rBuffer.records[4].level, 30);
-        assert.deepStrictEqual(logBuffer.records[4].level, 'info');
+        assert.notStrictEqual(rBuffer.records[4].req_id, undefined);
-        assert.notStrictEqual(logBuffer.records[4].req_id, undefined);
-        /* eslint-enable max-len */
         done();
     });
 });

tests/functional/multi-modules/module1.js

@@ -1,4 +1,4 @@
-const Werelogs = require('werelogs').Logger;  // eslint-disable-line
+const Werelogs = require('werelogs');
 
 const log = new Werelogs('test-mod1');
 

tests/functional/multi-modules/module2.js

@@ -1,4 +1,4 @@
-const Werelogs = require('werelogs').Logger;  // eslint-disable-line
+const Werelogs = require('werelogs');
 
 const log = new Werelogs('test-mod2');
 

tests/functional/multi-modules/module3.js

@@ -1,4 +1,4 @@
-const Werelogs = require('werelogs').Logger;  // eslint-disable-line
+const Werelogs = require('werelogs');
 
 const log = new Werelogs('test-mod3');
 

tests/unit/Config.js

@@ -1,58 +1,64 @@
-/* eslint-disable max-len */
-
 const assert = require('assert');
+const bunyan = require('bunyan');
 
 const Config = require('../../lib/Config.js');
+const logBuffer = new bunyan.RingBuffer({ limit: 1 });
 
 describe('Config', () => {
-    const config = new Config();
-
     beforeEach(() => {
-        config.reset();
+        Config.reset();
     });
 
-    it('should work with default configuration', done => {
+    it('should work with default configuration', (done) => {
         assert.doesNotThrow(
             () => {
-                config.logger.info('test message');
+                Config.logger.info('test message');
             },
-            Error,
+            Error);
-        );
         done();
     });
 
-    it('log level should be updateable', done => {
+    it('log level should be updateable', (done) => {
-        config.update({ level: 'debug' });
+        Config.update({ level: 'debug' });
-        assert.strictEqual(config.level, 'debug', 'Expected config\'s log level to be updated.');
+        assert.strictEqual(Config.level, 'debug', 'Expected Config\'s log level to be updated.');
         done();
     });
 
-    it('dump threshold should be updateable', done => {
+    it('dump threshold should be updateable', (done) => {
-        const origDump = config.dump;
+        const origDump = Config.dump;
-        assert.notStrictEqual(origDump, 'warn', 'Expected original config.dump to differ from value to update.');
+        assert.notStrictEqual(origDump, 'warn', 'Expected original Config.dump to differ from value to update.');
-        config.update({ dump: 'warn' });
+        Config.update({ dump: 'warn' });
-        assert.strictEqual(config.dump, 'warn', 'Expected config\'s dump threshold to be updated.');
+        assert.strictEqual(Config.dump, 'warn', 'Expected Config\'s dump threshold to be updated.');
         done();
     });
 
-    it('end logging level should be updateable', done => {
+    it('end logging level should be updateable', (done) => {
-        const origEnd = config.end;
+        const origEnd = Config.end;
-        assert.notStrictEqual(origEnd, 'trace', 'Expected original config.end to differ from value to update.');
+        assert.notStrictEqual(origEnd, 'trace', 'Expected original Config.end to differ from value to update.');
-        config.update({ end: 'trace' });
+        Config.update({ end: 'trace' });
-        assert.strictEqual(config.end, 'trace', 'Expected config\'s end log level to be updated.');
+        assert.strictEqual(Config.end, 'trace', 'Expected Config\'s end log level to be updated.');
         done();
     });
 
-    it('should not be modified by an empty config object', done => {
+    it('streams should be updateable', (done) => {
-        const origLevel = config.level;
+        const origStreams = Config.streams;
-        const origDump = config.dump;
+        const origLogger = Config.logger;
-        const origLogger = config.logger;
+        Config.update({ streams: [{ type: 'raw', stream: logBuffer }] });
-        const origStreams = config.streams;
+        assert.notStrictEqual(origStreams, Config.streams, 'Expected Config\'s streams to have been updated.');
-        config.update({});
+        assert.notStrictEqual(origLogger, Config.Logger, 'Expected Config\'s logger to have been replaced by update.');
-        assert.deepStrictEqual(origLevel, config.level, 'Expected logging level not to have changed.');
+        done();
-        assert.deepStrictEqual(origDump, config.dump, 'Expected dump threshold not to have changed.');
+    });
-        assert.strictEqual(origLogger, config.logger, 'Expected logger not to have changed.');
+
-        assert.deepStrictEqual(origStreams, config.streams, 'Expected streams not to have changed.');
+    it('should not be modified by an empty config object', (done) => {
+        const origLevel = Config.level;
+        const origDump = Config.dump;
+        const origLogger = Config.logger;
+        const origStreams = Config.streams;
+        Config.update({});
+        assert.deepStrictEqual(origLevel, Config.level, 'Expected logging level not to have changed.');
+        assert.deepStrictEqual(origDump, Config.dump, 'Expected dump threshold not to have changed.');
+        assert.strictEqual(origLogger, Config.logger, 'Expected logger not to have changed.');
+        assert.deepStrictEqual(origStreams, Config.streams, 'Expected streams not to have changed.');
         done();
     });
 });

tests/unit/LogLevel.js

@@ -1,5 +1,4 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const assert = require('assert');
 
@@ -13,97 +12,89 @@ function generateValidThrowTest(level) {
             },
             Error,
             'Expected level to be valid and '
-            + 'the function not to throw an Error.',
+            + 'the function not to throw an Error.');
-        );
         done();
     };
 }
 
 describe('LogLevel', () => {
     describe('throwIfInvalid(level)', () => {
-        it('should throw on invalid string', done => {
+        it('should throw on invalid string', (done) => {
             assert.throws(
                 () => {
                     LogLevel.throwIfInvalid('invalid');
                 },
                 RangeError,
                 'Expected function to throw an Error instance due to '
-                + 'invalid log level.',
+                + 'invalid log level.');
-            );
             done();
         });
 
         it('should not throw on "trace" level',
-            generateValidThrowTest('trace'));
+           generateValidThrowTest('trace'));
 
         it('should not throw on "debug" level',
-            generateValidThrowTest('debug'));
+           generateValidThrowTest('debug'));
 
         it('should not throw on "info" level',
-            generateValidThrowTest('info'));
+           generateValidThrowTest('info'));
 
         it('should not throw on "warn" level',
-            generateValidThrowTest('warn'));
+           generateValidThrowTest('warn'));
 
         it('should not throw on "error" level',
-            generateValidThrowTest('error'));
+           generateValidThrowTest('error'));
 
         it('should not throw on "fatal" level',
-            generateValidThrowTest('fatal'));
+           generateValidThrowTest('fatal'));
     });
 
     describe('shouldLog(level, floor)', () => {
-        it('should return true on "trace" parameters', done => {
+        it('should return true on "trace" parameters', (done) => {
             assert.strictEqual(
                 LogLevel.shouldLog('trace', 'trace'),
                 true,
-                'Expected trace floor to allow logging trace level.',
+                'Expected trace floor to allow logging trace level.');
-            );
             done();
         });
 
-        it('should return true on "debug" parameters', done => {
+        it('should return true on "debug" parameters', (done) => {
             assert.strictEqual(
                 LogLevel.shouldLog('debug', 'debug'),
                 true,
-                'Expected debug floor to allow logging debug level.',
+                'Expected debug floor to allow logging debug level.');
-            );
             done();
         });
 
-        it('should return true on "info" parameters', done => {
+        it('should return true on "info" parameters', (done) => {
             assert.strictEqual(
                 LogLevel.shouldLog('info', 'info'),
                 true,
-                'Expected info floor to allow logging info level.',
+                'Expected info floor to allow logging info level.');
-            );
             done();
         });
 
-        it('should return true on "warn" parameters', done => {
+        it('should return true on "warn" parameters', (done) => {
             assert.strictEqual(
                 LogLevel.shouldLog('warn', 'warn'),
                 true,
-                'Expected warn floor to allow logging warn level.',
+                'Expected warn floor to allow logging warn level.');
-            );
             done();
         });
 
-        it('should return true on "error" parameters', done => {
+        it('should return true on "error" parameters', (done) => {
             assert.strictEqual(
                 LogLevel.shouldLog('error', 'error'),
                 true,
-                'Expected error floor to allow logging error level.',
+                'Expected error floor to allow logging error level.');
-            );
             done();
         });
 
-        it('should return true on "fatal" parameters', done => {
+        it('should return true on "fatal" parameters', (done) => {
             assert.strictEqual(
                 LogLevel.shouldLog('fatal', 'fatal'),
                 true,
-                'Expected fatal floor to allow logging fatal level.',
+                'Expected fatal floor to allow logging fatal level.');
-            );
             done();
         });
     });

tests/unit/Logger.js

@@ -1,15 +1,15 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const assert = require('assert');
 
-const { genericFilterGenerator, loggingMisuseGenerator, DummyLogger } = require('../Utils');
+const Utils = require('../Utils.js');
+const genericFilterGenerator = Utils.genericFilterGenerator;
+const loggingMisuseGenerator = Utils.loggingMisuseGenerator;
+const DummyLogger = Utils.DummyLogger;
 
 const Config = require('../../lib/Config.js');
 const RequestLogger = require('../../lib/RequestLogger.js');
-const Logger = require('../../lib/Logger.js');
+const Logger = require('../../index.js');
-
-const config = new Config();
 
 /*
  * This function is a thunk-function calling the Utils'  filterGenerator with
@@ -18,107 +18,163 @@ const config = new Config();
  */
 function filterGenerator(logLevel, callLevel) {
     function createModuleLogger(dummyLogger, filterLevel) {
+        const logger = new Logger('TestModuleLogger',
+            {
+                level: filterLevel,
+                dump: 'fatal',
+            });
         /*
-         * Here, patch the config by setting a specifically designed dummyLogger
+         * Here, patch the Config by setting a specificly designed dummyLogger
          * for testing purposes that will help us collect runtime data.
          */
-        const testConfig = new Config({ level: filterLevel, dump: 'fatal' });
+        Config.bLogger = dummyLogger;
-        testConfig.simpleLogger = dummyLogger;
 
-        return new Logger(testConfig, 'TestModuleLogger');
+        return logger;
     }
 
     return genericFilterGenerator(logLevel, callLevel, createModuleLogger);
 }
 
 function checkFields(src, result) {
-    Object.keys(src).forEach(k => {
+    Object.keys(src).forEach((k) => {
-        if (Object.prototype.hasOwnProperty.call(src, k)) {
+        if (src.hasOwnProperty(k)) {
             assert.deepStrictEqual(result[k], src[k]);
         }
     });
-    assert.ok(Object.prototype.hasOwnProperty.call(result, 'time'));
-    // Time field should be current give or take 1s
-    assert.ok((Date.now() - result.time) < 1000);
 }
 
 
-describe('Logger is usable:', () => {
+describe('WereLogs Logger is usable:', () => {
     beforeEach(() => {
-        config.reset();
+        Config.reset();
     });
 
-    it('Cannot be instanciated without parameters', done => {
+    it('Can be instanciated with only a name', (done) => {
-        assert.throws(
+        assert.doesNotThrow(
-            () => new Logger(),
+            () => {
-            TypeError,
+                return new Logger('WereLogsTest');
-            'Logger Instanciation should not succeed without parameter.',
+            },
-        );
+            Error,
+            'WereLogs Instanciation should not throw any kind of error.');
         done();
     });
 
-    it('Cannot be instanciated with only a config', done => {
+    it('Cannot be instanciated with invalid log level', (done) => {
         assert.throws(
-            () => new Logger(config),
+            () => {
-            TypeError,
+                return new Logger('test', {level: 'invalidlevel'});
-            'Logger Instanciation should not be succeed without a name.',
+            },
-        );
+            RangeError,
+            'WereLogs should not be instanciable without the proper logging levels.');
         done();
     });
 
-    it('Cannot be instanciated with a bad config type', done => {
+    it('Cannot be instanciated with invalid dump threshold level', (done) => {
         assert.throws(
-            () => new Logger({ level: 'info' }, 'WereLogsTest'),
+            () => {
-            TypeError,
+                return new Logger('test', {level: 'trace', dump: 'invalidlevel'});
-            'Logger Instanciation should not succeed with a bad config type.',
+            },
-        );
+            RangeError,
+            'WereLogs should not be instanciable without the proper dumping threshold levels.');
+
         done();
     });
 
-    it('Cannot be instanciated with only a name', done => {
+    it('Cannot be instanciated with a non-Array in config.streams', (done) => {
         assert.throws(
-            () => new Logger('WereLogsTest'),
+            () => {
-            TypeError,
+                return new Logger('test', {streams: process.stdout});
-            'Logger Instanciation should not succeed with only a name.',
+            },
-        );
+            Error,
+            'Werelogs should not be instanciable with a stream option that is not an array.');
         done();
     });
 
-    it('Can create Per-Request Loggers', done => {
+    it('Cannot be instanciated with an empty Array in config.streams', (done) => {
-        const logger = new Logger(config, 'test');
+        assert.throws(
+            () => {
+                return new Logger('test', {streams: []});
+            },
+            Error,
+            'Werelogs should not be instanciable with an empty array for the streams option.');
+        done();
+    });
+
+    it('Cannot set logging level to invalid level at runtime', (done) => {
+        const logger = new Logger('test');
+        assert.throws(
+            () => {
+                logger.setLevel('invalidLevel');
+            },
+            RangeError,
+            'WereLogs should not be able to set log level to an invalid level.');
+        done();
+    });
+
+    it('Can set logging level at runtime', (done) => {
+        const logger = new Logger('test');
+        assert.doesNotThrow(
+            () => {
+                logger.setLevel('fatal');
+            },
+            RangeError,
+            'WereLogs should be able to set log level at runtime.');
+        done();
+    });
+
+    it('Cannot set dump threshold to invalid level at runtime', (done) => {
+        const logger = new Logger('test');
+        assert.throws(
+            () => {
+                logger.setDumpThreshold('invalidLevel');
+            },
+            RangeError,
+            'WereLogs should not be able to set dump threshold to an invalid level.');
+        done();
+    });
+
+    it('Can set dump threshold at runtime', (done) => {
+        const logger = new Logger('test');
+        assert.doesNotThrow(
+            () => {
+                logger.setDumpThreshold('fatal');
+            },
+            RangeError,
+            'WereLogs should be able to set dump threshold at runtime.');
+        done();
+    });
+
+    it('Can create Per-Request Loggers', (done) => {
+        const logger = new Logger('test');
         assert.doesNotThrow(
             () => {
                 logger.newRequestLogger();
             },
             Error,
-            'Werelogs should not throw when creating a request logger.',
+            'Werelogs should not throw when creating a request logger.');
-        );
+        const reqLogger = logger.newRequestLogger();
-        done();
-    });
-
-    it('Can create Per-Request Loggers from a Serialized UID Array', done => {
-        const logger = new Logger(config, 'test');
-        assert.doesNotThrow(
-            () => {
-                logger.newRequestLogger();
-            },
-            Error,
-            // eslint-disable-next-line max-len
-            'Werelogs should not throw when creating a request logger from a Serialized UID Array.',
-        );
-        const reqLogger = logger.newRequestLoggerFromSerializedUids(
-            'OneUID:SecondUID:TestUID:YouWinUID',
-        );
         assert(reqLogger instanceof RequestLogger, 'RequestLogger');
-        assert.deepStrictEqual(reqLogger.getUids().slice(0, -1),
-            ['OneUID', 'SecondUID', 'TestUID', 'YouWinUID']);
         done();
     });
 
-    it('Uses the additional fields as expected', done => {
+    it('Can create Per-Request Loggers from a Serialized UID Array', (done) => {
+        const logger = new Logger('test');
+        assert.doesNotThrow(
+            () => {
+                logger.newRequestLogger();
+            },
+            Error,
+            'Werelogs should not throw when creating a request logger from a Serialized UID Array.');
+        const reqLogger = logger.newRequestLoggerFromSerializedUids('OneUID:SecondUID:TestUID:YouWinUID');
+        assert(reqLogger instanceof RequestLogger, 'RequestLogger');
+        assert.deepStrictEqual(reqLogger.getUids().slice(0, -1), ['OneUID', 'SecondUID', 'TestUID', 'YouWinUID']);
+        done();
+    });
+
+    it('Uses the additional fields as expected', (done) => {
         const dummyLogger = new DummyLogger();
-        config.simpleLogger = dummyLogger;
+        const logger = new Logger('test');
-        const logger = new Logger(config, 'test');
+        Config.bLogger = dummyLogger;
         const fields = {
             ip: '127.0.0.1',
             method: 'GET',
@@ -130,31 +186,28 @@ describe('Logger is usable:', () => {
         done();
     });
 
-    /* eslint-disable max-len */
     describe('Does not crash and logs a fatal message when mis-using its logging API', () => {
         const testValues = [
-            { desc: 'a string as second argument', args: ['test', 'second-param-string'] },
+            { desc: 'a string as second argument', args: [ 'test', 'second-param-string' ] },
-            { desc: 'a function as second argument', args: ['test', () => { }] }, // eslint-disable-line arrow-body-style
+            { desc: 'a function as second argument', args: [ 'test', () => { return; } ] },
-            { desc: 'a Number as second argument', args: ['test', 1] },
+            { desc: 'a Number as second argument', args: [ 'test', 1 ] },
-            { desc: 'more than 2 arguments', args: ['test', 2, 3, 4] },
+            { desc: 'more than 2 arguments', args: [ 'test', 2, 3, 4 ] },
         ];
-        /* eslint-enable max-len */
         function createMisusableLogger(dummyLogger) {
-            config.simpleLogger = dummyLogger;
+            const logger = new Logger('test');
-            const logger = new Logger(config, 'test');
+            Config.bLogger = dummyLogger;
             return logger;
         }
 
         for (let i = 0; i < testValues.length; ++i) {
             const test = testValues[i];
-            it(`Does not crash with ${test.desc}`,
+            it('Does not crash with ' + test.desc,
-                loggingMisuseGenerator(test, createMisusableLogger));
+               loggingMisuseGenerator(test, createMisusableLogger));
         }
     });
 });
 
-/* eslint-disable no-multi-spaces, max-len */
+describe('Werelogs Module-level Logger can log as specified by the log level', () => {
-describe('Logger can log as specified by the log level', () => {
     it('Trace level does not filter trace level out',   filterGenerator('trace', 'trace'));
     it('Trace level does not filter debug level out',   filterGenerator('trace', 'debug'));
     it('Trace level does not filter info level out',    filterGenerator('trace', 'info'));
@@ -197,4 +250,3 @@ describe('Logger can log as specified by the log level', () => {
     it('Fatal level filters error level out',           filterGenerator('fatal', 'error'));
     it('Fatal level does not filter fatal level out',   filterGenerator('fatal', 'fatal'));
 });
-/* eslint-enable no-multi-spaces, max-len */

tests/unit/RequestLogger.js

@@ -1,9 +1,11 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const assert = require('assert');
 
-const { DummyLogger, genericFilterGenerator, loggingMisuseGenerator } = require('../Utils.js');
+const Utils = require('../Utils.js');
+const DummyLogger = Utils.DummyLogger;
+const genericFilterGenerator = Utils.genericFilterGenerator;
+const loggingMisuseGenerator = Utils.loggingMisuseGenerator;
 
 const RequestLogger = require('../../lib/RequestLogger.js');
 
@@ -21,21 +23,18 @@ function filterGenerator(logLevel, callLevel) {
 }
 
 
-function runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts,
+function runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts, done) {
-    done) {
     const dummyLogger = new DummyLogger();
     const reqLogger = new RequestLogger(dummyLogger, 'trace', 'error', 'info');
 
-    commandHistory.every((val, index) => {
+    commandHistory.every(function doLogWithLevel(val, index) {
         switch (val) {
-        /* eslint-disable no-multi-spaces */
         case 'trace': reqLogger.trace(index); break;
         case 'debug': reqLogger.debug(index); break;
         case 'info': reqLogger.info(index);   break;
         case 'warn': reqLogger.warn(index);   break;
         case 'error': reqLogger.error(index); break;
         case 'fatal': reqLogger.fatal(index); break;
-        /* eslint-enable no-multi-spaces */
         default:
             done(new Error('Unexpected logging level name: ', val));
         }
@@ -43,40 +42,40 @@ function runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts,
     });
 
     expectedHistory.every((val, index) => {
-        assert.strictEqual(dummyLogger.ops[index][0], val[0],
+        assert.strictEqual(dummyLogger.ops[index][0], val[0], 'Expected log entry levels to match.');
-            'Expected log entry levels to match.');
+        assert.strictEqual(dummyLogger.ops[index][1][1], val[1], 'Expected log entry values to match.');
-        assert.strictEqual(dummyLogger.ops[index][1][1], val[1],
-            'Expected log entry values to match.');
         return true;
     });
     assert.deepEqual(dummyLogger.counts, expectedCounts);
 }
 
-/* eslint-disable no-multi-spaces, max-len */
+
 describe('RequestLogger', () => {
     describe('Object Instanciation', () => {
         describe('Logging Levels Initialization', () => {
-            it('Throws if LogLevel is higher than dumpThreshold', done => {
+            it('Throws if LogLevel is higher than dumpThreshold', (done) => {
                 assert.throws(
-                    () => new RequestLogger(undefined, 'fatal', 'debug', 'info'),
+                    () => {
+                        return new RequestLogger(undefined, 'fatal', 'debug', 'info');
+                    },
                     Error,
-                    'Dump level "debug" should not be valid with logging level "fatal".',
+                    'Dump level "debug" should not be valid with logging level "fatal".');
-                );
                 done();
             });
 
-            it('Works with LogLevel lesser or equal to DumpLevel', done => {
+            it('Works with LogLevel lesser or equal to DumpLevel', (done) => {
                 assert.doesNotThrow(
-                    () => new RequestLogger(undefined, 'debug', 'fatal', 'info'),
+                    () => {
+                        return new RequestLogger(undefined, 'debug', 'fatal', 'info');
+                    },
                     Error,
-                    'Dump level "fatal" should be valid with logging level "debug".',
+                    'Dump level "fatal" should be valid with logging level "debug".');
-                );
                 done();
             });
         });
 
         describe('UID Initialization', () => {
-            it('defines an UID when none provided', done => {
+            it('defines an UID when none provided', (done) => {
                 const dummyLogger = new DummyLogger();
                 const reqLogger = new RequestLogger(dummyLogger, 'debug', 'fatal', 'info');
                 assert.strictEqual(Array.isArray(reqLogger.uids), true, 'Expected uid list to be an Array.');
@@ -84,7 +83,7 @@ describe('RequestLogger', () => {
                 done();
             });
 
-            it('creates an UID array out of the provided UID string', done => {
+            it('creates an UID array out of the provided UID string', (done) => {
                 const dummyLogger = new DummyLogger();
                 const uids = 'BasicUid';
                 const reqLogger = new RequestLogger(dummyLogger, 'debug', 'fatal', 'info', uids);
@@ -94,16 +93,17 @@ describe('RequestLogger', () => {
                 done();
             });
 
-            it('throws when UID string provided contains a colon', done => {
+            it('throws when UID string provided contains a colon', (done) => {
                 assert.throws(
-                    () => new RequestLogger(undefined, 'debug', 'fatal', 'info', 'pouet:tata'),
+                    () => {
+                        return new RequestLogger(undefined, 'debug', 'fatal', 'info', 'pouet:tata');
+                    },
                     Error,
-                    'UID string "pouet:tata" should be rejected by the RequestLogger constructor.',
+                    'UID string "pouet:tata" should be rejected by the RequestLogger constructor.');
-                );
                 done();
             });
 
-            it('expands the UID array when one is provided', done => {
+            it('expands the UID array when one is provided', (done) => {
                 const dummyLogger = new DummyLogger();
                 const uids = ['oneuid', 'twouid', 'threeuids'];
                 const reqLogger = new RequestLogger(dummyLogger, 'debug', 'fatal', 'info', uids);
@@ -113,18 +113,19 @@ describe('RequestLogger', () => {
                 done();
             });
 
-            it('throws when UID string Array provided contains an UID that contains a colon', done => {
+            it('throws when UID string Array provided contains an UID that contains a colon', (done) => {
                 assert.throws(
-                    () => new RequestLogger(undefined, 'debug', 'fatal', 'info', ['OneUID', 'SecondUID', 'Test:DashUID']),
+                    () => {
+                        return new RequestLogger(undefined, 'debug', 'fatal', 'info', ['OneUID', 'SecondUID', 'Test:DashUID']);
+                    },
                     Error,
-                    'UID string "Test:DashUID" should be rejected by the RequestLogger constructor.',
+                    'UID string "Test:DashUID" should be rejected by the RequestLogger constructor.');
-                );
                 done();
             });
         });
 
         describe('getUids() method', () => {
-            it('retrieves a list of string UID', done => {
+            it('retrieves a list of string UID', (done) => {
                 const dummyLogger = new DummyLogger();
                 const reqLogger = new RequestLogger(dummyLogger, 'info', 'error', 'info');
                 const uidlist = reqLogger.getUids();
@@ -134,7 +135,7 @@ describe('RequestLogger', () => {
             });
 
             describe('Length of the UIDs array', () => {
-                it('default constructor yields a one-item UID list', done => {
+                it('default constructor yields a one-item UID list', (done) => {
                     const dummyLogger = new DummyLogger();
                     const reqLogger = new RequestLogger(dummyLogger, 'info', 'error', 'info');
                     const uidlist = reqLogger.getUids();
@@ -142,7 +143,7 @@ describe('RequestLogger', () => {
                     done();
                 });
 
-                it('manually-set UID constructor yields a one-item UID list', done => {
+                it('manually-set UID constructor yields a one-item UID list', (done) => {
                     const dummyLogger = new DummyLogger();
                     const myUid = 'ThisIsMyUid';
                     const reqLogger = new RequestLogger(dummyLogger, 'info', 'error', 'info', myUid);
@@ -152,9 +153,9 @@ describe('RequestLogger', () => {
                     done();
                 });
 
-                it('manually-set parent UID List constructor yields a n+1 item UID list', done => {
+                it('manually-set parent UID List constructor yields a n+1 item UID list', (done) => {
                     const dummyLogger = new DummyLogger();
-                    const myParentUidList = ['ThisIsMyOriginUid', 'ThisIsMySecondGenUid', 'ThisIsMyThirdGenUid'];
+                    const myParentUidList = [ 'ThisIsMyOriginUid', 'ThisIsMySecondGenUid', 'ThisIsMyThirdGenUid' ];
                     const reqLogger = new RequestLogger(dummyLogger, 'info', 'error', 'info', myParentUidList);
                     const uidlist = reqLogger.getUids();
                     assert.strictEqual(uidlist.length, myParentUidList.length + 1, 'Expected n+1 item in UID Array compared to set UID List array');
@@ -163,7 +164,7 @@ describe('RequestLogger', () => {
                 });
             });
 
-            it('internal data cannot be set through returned UID List', done => {
+            it('internal data cannot be set through returned UID List', (done) => {
                 const dummyLogger = new DummyLogger();
                 const reqLogger = new RequestLogger(dummyLogger, 'info', 'error', 'info');
                 const uidlist = reqLogger.getUids();
@@ -174,11 +175,11 @@ describe('RequestLogger', () => {
         });
 
         describe('getSerializedUids()', () => {
-            it('Should return a properly serialized UID Array', done => {
+            it('Should return a properly serialized UID Array', (done) => {
                 const dummyLogger = new DummyLogger();
-                const uidList = ['FirstUID', 'SecondUID', 'ThirdUID', 'TestUID'];
+                const uidList = [ 'FirstUID', 'SecondUID', 'ThirdUID', 'TestUID' ];
                 const reqLogger = new RequestLogger(dummyLogger, 'info', 'error', 'info', uidList);
-                const expectedString = `FirstUID:SecondUID:ThirdUID:TestUID:${reqLogger.getUids()[4]}`;
+                const expectedString = 'FirstUID:SecondUID:ThirdUID:TestUID:' + reqLogger.getUids()[4];
                 assert.strictEqual(reqLogger.getSerializedUids(), expectedString, 'Expected serialized UID List to match expected data.');
                 done();
             });
@@ -187,10 +188,10 @@ describe('RequestLogger', () => {
 
     describe('Does not crash when mis-using its logging API', () => {
         const testValues = [
-            { desc: 'a string as second argument', args: ['test', 'second-param-string'] },
+            { desc: 'a string as second argument', args: [ 'test', 'second-param-string' ] },
-            { desc: 'a function as second argument', args: ['test', function f() {  }] },
+            { desc: 'a function as second argument', args: [ 'test', () => { return; } ] },
-            { desc: 'a Number as second argument', args: ['test', 1] },
+            { desc: 'a Number as second argument', args: [ 'test', 1 ] },
-            { desc: 'more than 2 arguments', args: ['test', 2, 3, 4] },
+            { desc: 'more than 2 arguments', args: [ 'test', 2, 3, 4 ] },
         ];
         function createMisusableRequestLogger(dummyLogger) {
             return new RequestLogger(dummyLogger, 'info', 'error', 'info');
@@ -198,8 +199,8 @@ describe('RequestLogger', () => {
 
         for (let i = 0; i < testValues.length; ++i) {
             const test = testValues[i];
-            it(`Does not crash with ${test.desc}`,
+            it('Does not crash with ' + test.desc,
-                loggingMisuseGenerator(test, createMisusableRequestLogger));
+               loggingMisuseGenerator(test, createMisusableRequestLogger));
         }
     });
 
@@ -246,39 +247,33 @@ describe('RequestLogger', () => {
         it('Fatal level filters error level out',           filterGenerator('fatal', 'error'));
         it('Fatal level does not filter fatal level out',   filterGenerator('fatal', 'fatal'));
     });
-    /* eslint-enable no-multi-spaces, max-len */
 
     describe('Logging API regression testing', () => {
-        it('Should not alter the input fields when not actually logging',
+        it('Should not alter the input fields when not actually logging', (done) => {
-            done => {
-                const dummyLogger = new DummyLogger();
-                const reqLogger = new RequestLogger(dummyLogger,
-                    'info', 'fatal', 'info');
-                const refFields = { hits: 45, count: 32 };
-                const usedFields = { ...refFields };
-                reqLogger.debug('test', usedFields);
-                assert.deepStrictEqual(usedFields, refFields);
-                done();
-            });
-
-        it('Should not alter the input fields when actually logging',
-            done => {
-                const dummyLogger = new DummyLogger();
-                const reqLogger = new RequestLogger(dummyLogger,
-                    'info', 'fatal', 'info');
-                const refFields = { hits: 45, count: 32 };
-                const usedFields = { ...refFields };
-                reqLogger.info('test', usedFields);
-                assert.deepStrictEqual(usedFields, refFields);
-                done();
-            });
-
-        it('Should not alter the input fields when dumping', done => {
             const dummyLogger = new DummyLogger();
-            const reqLogger = new RequestLogger(dummyLogger,
+            const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
-                'info', 'fatal', 'info');
+            const refFields = { 'hits': 45, 'count': 32 };
-            const refFields = { hits: 45, count: 32 };
+            const usedFields = Object.assign({}, refFields);
-            const usedFields = { ...refFields };
+            reqLogger.debug('test', usedFields);
+            assert.deepStrictEqual(usedFields, refFields);
+            done();
+        });
+
+        it('Should not alter the input fields when actually logging', (done) => {
+            const dummyLogger = new DummyLogger();
+            const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
+            const refFields = { 'hits': 45, 'count': 32 };
+            const usedFields = Object.assign({}, refFields);
+            reqLogger.info('test', usedFields);
+            assert.deepStrictEqual(usedFields, refFields);
+            done();
+        });
+
+        it('Should not alter the input fields when dumping', (done) => {
+            const dummyLogger = new DummyLogger();
+            const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
+            const refFields = { 'hits': 45, 'count': 32 };
+            const usedFields = Object.assign({}, refFields);
             reqLogger.error('test', usedFields);
             assert.deepStrictEqual(usedFields, refFields);
             done();
@@ -286,7 +281,7 @@ describe('RequestLogger', () => {
     });
 
     describe('Default Fields', () => {
-        it('should not modify the object passed as a parameter', done => {
+        it('should not modify the object passed as a parameter', (done) => {
             const add1 = {
                 attr1: 0,
             };
@@ -294,8 +289,7 @@ describe('RequestLogger', () => {
                 attr2: 'string',
             };
             const dummyLogger = new DummyLogger();
-            const reqLogger = new RequestLogger(dummyLogger,
+            const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
-                'info', 'fatal', 'info');
             reqLogger.addDefaultFields(add1);
             reqLogger.addDefaultFields(add2);
             assert.deepStrictEqual(add1, { attr1: 0 });
@@ -303,192 +297,129 @@ describe('RequestLogger', () => {
             done();
         });
 
-        it('should add one added default field to the log entries', done => {
+        it('should add one added default field to the log entries', (done) => {
             const clientInfo = {
                 clientIP: '127.0.0.1',
             };
             const dummyLogger = new DummyLogger();
-            const reqLogger = new RequestLogger(dummyLogger,
+            const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
-                'info', 'fatal', 'info');
             reqLogger.addDefaultFields(clientInfo);
             reqLogger.info('test message');
-            assert.strictEqual(clientInfo.clientIP,
+            assert.strictEqual(clientInfo.clientIP, dummyLogger.ops[0][1][0].clientIP);
-                dummyLogger.ops[0][1][0].clientIP);
             done();
         });
 
-        it('should add multiple added default fields to the log entries',
+        it('should add multiple added default fields to the log entries', (done) => {
-            done => {
+            const clientInfo = {
-                const clientInfo = {
+                clientIP: '127.0.0.1',
-                    clientIP: '127.0.0.1',
+                clientPort: '1337',
-                    clientPort: '1337',
+            };
-                };
+            const requestInfo = {
-                const requestInfo = {
+                object: '/tata/self.txt',
-                    object: '/tata/self.txt',
+                creator: 'Joddy',
-                    creator: 'Joddy',
+            };
-                };
+            const dummyLogger = new DummyLogger();
-                const dummyLogger = new DummyLogger();
+            const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
-                const reqLogger = new RequestLogger(dummyLogger,
+            reqLogger.addDefaultFields(clientInfo);
-                    'info', 'fatal', 'info');
+            reqLogger.addDefaultFields(requestInfo);
-                reqLogger.addDefaultFields(clientInfo);
+            reqLogger.info('test message');
-                reqLogger.addDefaultFields(requestInfo);
+            assert.strictEqual(clientInfo.clientIP, dummyLogger.ops[0][1][0].clientIP);
-                reqLogger.info('test message');
+            assert.strictEqual(clientInfo.clientPort, dummyLogger.ops[0][1][0].clientPort);
-                assert.strictEqual(clientInfo.clientIP,
+            assert.strictEqual(requestInfo.object, dummyLogger.ops[0][1][0].object);
-                    dummyLogger.ops[0][1][0].clientIP);
+            assert.strictEqual(requestInfo.creator, dummyLogger.ops[0][1][0].creator);
-                assert.strictEqual(clientInfo.clientPort,
+            done();
-                    dummyLogger.ops[0][1][0].clientPort);
+        });
-                assert.strictEqual(requestInfo.object,
-                    dummyLogger.ops[0][1][0].object);
-                assert.strictEqual(requestInfo.creator,
-                    dummyLogger.ops[0][1][0].creator);
-                done();
-            });
     });
 
     describe('Automatic Elapsed Time computation', () => {
         describe('Deprecated API:', () => {
-            it('should include an "elapsed_ms" field in the last log entry',
+            it('should include an "elapsed_ms" field in the last log entry', (done) => {
-                done => {
+                const dummyLogger = new DummyLogger();
-                    const dummyLogger = new DummyLogger();
+                const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
-                    const reqLogger = new RequestLogger(dummyLogger,
+                reqLogger.end('Last message');
-                        'info', 'fatal', 'info');
+                assert.strictEqual(dummyLogger.ops[0][1][1], 'Last message');
-                    reqLogger.end('Last message');
+                assert.notStrictEqual(dummyLogger.ops[0][1][0].elapsed_ms, undefined);
-                    assert.strictEqual(dummyLogger.ops[0][1][1], 'Last message');
+                assert.strictEqual(typeof(dummyLogger.ops[0][1][0].elapsed_ms), 'number');
-                    assert.notStrictEqual(dummyLogger.ops[0][1][0].elapsed_ms,
+                done();
-                        undefined);
+            });
-                    assert.strictEqual(typeof dummyLogger.ops[0][1][0]
-                        .elapsed_ms, 'number');
-                    done();
-                });
 
-            // eslint-disable-next-line max-len
             it('should include an "elapsed_ms" field in the last log entry and be error level', () => {
                 const dummyLogger = new DummyLogger();
-                const reqLogger = new RequestLogger(dummyLogger,
+                const reqLogger = new RequestLogger(dummyLogger, 'info', 'fatal', 'info');
-                    'info', 'fatal', 'info');
                 reqLogger.errorEnd('Last message failed');
-                assert.strictEqual(dummyLogger.ops[0][1][1],
+                assert.strictEqual(dummyLogger.ops[0][1][1], 'Last message failed');
-                    'Last message failed');
+                assert.notStrictEqual(dummyLogger.ops[0][1][0].elapsed_ms, undefined);
-                assert.notStrictEqual(dummyLogger.ops[0][1][0].elapsed_ms,
+                assert.strictEqual(typeof(dummyLogger.ops[0][1][0].elapsed_ms), 'number');
-                    undefined);
-                assert.strictEqual(typeof dummyLogger.ops[0][1][0].elapsed_ms,
-                    'number');
                 assert.strictEqual(dummyLogger.ops[0][0], 'error');
             });
         });
 
         const endLogging = {
-            trace: endLogger => endLogger.trace.bind(endLogger),
+            trace: (endLogger) => { return endLogger.trace.bind(endLogger); },
-            debug: endLogger => endLogger.debug.bind(endLogger),
+            debug: (endLogger) => { return endLogger.debug.bind(endLogger); },
-            info: endLogger => endLogger.info.bind(endLogger),
+            info: (endLogger) => { return endLogger.info.bind(endLogger); },
-            warn: endLogger => endLogger.warn.bind(endLogger),
+            warn: (endLogger) => { return endLogger.warn.bind(endLogger); },
-            error: endLogger => endLogger.error.bind(endLogger),
+            error: (endLogger) => { return endLogger.error.bind(endLogger); },
-            fatal: endLogger => endLogger.fatal.bind(endLogger),
+            fatal: (endLogger) => { return endLogger.fatal.bind(endLogger); },
         };
-        /* eslint-disable max-len */
+        Object.keys(endLogging).forEach((level) => {
-        Object.keys(endLogging).forEach(level => {
+            it(`should include an "elapsed_ms" field in the last log entry with level ${level}`, (done) => {
-            it(`should include an "elapsed_ms" field in the last log entry with level ${level}`, done => {
                 const dummyLogger = new DummyLogger();
                 const reqLogger = new RequestLogger(dummyLogger, 'trace', 'fatal');
                 endLogging[level](reqLogger.end())('Last message');
                 assert.strictEqual(dummyLogger.ops[0][1][1], 'Last message');
                 assert.notStrictEqual(dummyLogger.ops[0][1][0].elapsed_ms, undefined);
-                assert.strictEqual(typeof dummyLogger.ops[0][1][0].elapsed_ms, 'number');
+                assert.strictEqual(typeof(dummyLogger.ops[0][1][0].elapsed_ms), 'number');
                 assert.strictEqual(dummyLogger.ops[0][0], level);
                 done();
             });
         });
-        /* eslint-enable max-len */
 
-        it('should be augmentable through addDefaultFields', done => {
+        it('should be augmentable through addDefaultFields', (done) => {
             const dummyLogger = new DummyLogger();
             const reqLogger = new RequestLogger(dummyLogger, 'trace', 'fatal');
-            reqLogger.end().addDefaultFields({ endFlag: true });
+            reqLogger.end().addDefaultFields({endFlag: true});
             // Someone could do multiple operations in the meantime before
             // end() logging
-            reqLogger.end().error('Test Augmented END', { endValue: 42 });
+            reqLogger.end().error('Test Augmented END', {endValue: 42});
             assert.strictEqual(dummyLogger.ops[0][1][1], 'Test Augmented END');
-            assert.strictEqual(typeof dummyLogger.ops[0][1][0].elapsed_ms,
+            assert.strictEqual(typeof(dummyLogger.ops[0][1][0].elapsed_ms), 'number');
-                'number');
             assert.strictEqual(dummyLogger.ops[0][1][0].endFlag, true);
             assert.strictEqual(dummyLogger.ops[0][1][0].endValue, 42);
             done();
         });
-
-        it('should log an error in addition to request logs when end() called more than once',
-        done => {
-            const dummyLogger = new DummyLogger();
-            const reqLogger = new RequestLogger(dummyLogger, 'trace', 'fatal');
-            reqLogger.end().info('after first call to end()');
-            reqLogger.end().debug('after second call to end()');
-            assert.strictEqual(dummyLogger.ops.length, 3);
-            assert.strictEqual(dummyLogger.ops[0][0], 'info');
-            assert.strictEqual(dummyLogger.ops[0][1][1], 'after first call to end()');
-            assert.strictEqual(dummyLogger.ops[1][0], 'error');
-            assert.strictEqual(dummyLogger.ops[2][0], 'debug');
-            assert.strictEqual(dummyLogger.ops[2][1][1], 'after second call to end()');
-            done();
-        });
     });
 
     describe('Log History dumped when logging floor level reached', () => {
-        it('Dumping duplicates log entries', done => {
+        it('Dumping duplicates log entries', (done) => {
             const commandHistory = ['info', 'error'];
             const expectedHistory = [['info', 0], ['info', 0], ['error', 1]];
-            const expectedCounts = {
+            const expectedCounts = { trace: 0, debug: 0, info: 2, warn: 0, error: 1, fatal: 0 };
-                trace: 0,
-                debug: 0,
-                info: 2,
-                warn: 0,
-                error: 1,
-                fatal: 0,
-            };
 
-            runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts,
+            runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts, done);
-                done);
             done();
         });
 
-        it('Dumping Keeps logging history order', done => {
+        it('Dumping Keeps logging history order', (done) => {
             const commandHistory = ['trace', 'info', 'debug', 'error'];
-            const expectedHistory = [['trace', 0], ['info', 1], ['debug', 2],
+            const expectedHistory = [['trace', 0], ['info', 1], ['debug', 2], ['trace', 0], ['info', 1], ['debug', 2], ['error', 3]];
-                ['trace', 0], ['info', 1], ['debug', 2],
+            const expectedCounts = { trace: 2, debug: 2, info: 2, warn: 0, error: 1, fatal: 0 };
-                ['error', 3]];
-            const expectedCounts = {
-                trace: 2,
-                debug: 2,
-                info: 2,
-                warn: 0,
-                error: 1,
-                fatal: 0,
-            };
 
-            runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts,
+            runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts, done);
-                done);
             done();
         });
 
-        it('Dumping multiple times does not re-dump already-dumped entries',
+        it('Dumping multiple times does not re-dump already-dumped entries', (done) => {
-            done => {
+            const commandHistory = ['trace', 'info', 'debug', 'error',
-                const commandHistory = ['trace', 'info', 'debug', 'error',
+                                    'warn', 'debug', 'fatal'];
-                    'warn', 'debug', 'fatal'];
+            const expectedHistory = [['trace', 0], ['info', 1], ['debug', 2],
-                const expectedHistory = [['trace', 0], ['info', 1], ['debug', 2],
+                                     ['trace', 0], ['info', 1], ['debug', 2], ['error', 3],
-                    ['trace', 0], ['info', 1], ['debug', 2],
+                                     ['warn', 4], ['debug', 5],
-                    ['error', 3], ['warn', 4], ['debug', 5],
+                                     ['warn', 4], ['debug', 5], ['fatal', 6]];
-                    ['warn', 4], ['debug', 5],
+            const expectedCounts = { trace: 2, debug: 4, info: 2, warn: 2, error: 1, fatal: 1 };
-                    ['fatal', 6]];
-                const expectedCounts = {
-                    trace: 2,
-                    debug: 4,
-                    info: 2,
-                    warn: 2,
-                    error: 1,
-                    fatal: 1,
-                };
 
-                runLoggingDumpTest(commandHistory, expectedHistory,
+            runLoggingDumpTest(commandHistory, expectedHistory, expectedCounts, done);
-                    expectedCounts, done);
+            done();
-                done();
+        });
-            });
     });
 });

tests/unit/Utils.js

@@ -1,20 +1,19 @@
-
+'use strict';
-// eslint-disable-line strict
 
 const assert = require('assert');
-
+const Utils = require('../../lib/Utils.js');
-const {
+const generateUid = Utils.generateUid;
-    generateUid, serializeUids, unserializeUids, objectCopy,
+const serializeUids = Utils.serializeUids;
-} = require('../../lib/Utils');
+const unserializeUids = Utils.unserializeUids;
+const copyFields = Utils.copyFields;
 
 describe('Utils: generateUid', () => {
-    it('generates a string-typed ID', done => {
+    it('generates a string-typed ID', (done) => {
         const uid = generateUid();
-        assert.strictEqual(typeof uid, 'string',
+        assert.strictEqual(typeof(uid), 'string', 'The generated ID is not a String (' + typeof(uid) + ')');
-            `The generated ID is not a String (${typeof uid})`);
         done();
     });
-    it('generate roughly unique IDs', done => {
+    it('generate roughly unique IDs', (done) => {
         const generated = {};
         let count = 0;
         for (let i = 0; i < 10000; ++i) {
@@ -22,77 +21,41 @@ describe('Utils: generateUid', () => {
             count = generated[uid] ? generated[uid] + 1 : 1;
             generated[uid] = count;
         }
-        Object.keys(generated).every(uid => {
+        Object.keys(generated).every((uid) => {
-            assert.strictEqual(generated[uid], 1,
+            assert.strictEqual(generated[uid], 1, `Uid ${uid} was generated ${generated[uid]} times: It is not even remotely unique.`);
-                `Uid ${uid} was generated ${generated[uid]} `
-                               + 'times: It is not even remotely unique.');
-            return {};
         });
         done();
     });
 });
 
 describe('Utils: serializeUids', () => {
-    it('serializes to the expected string data', done => {
+    it('serializes to the expected string data', (done) => {
-        const uidList = ['FirstUID', 'SecondUID', 'ThirdUID'];
+        const uidList = [ 'FirstUID', 'SecondUID', 'ThirdUID'];
         const serializedUIDs = serializeUids(uidList);
-        assert.strictEqual(serializedUIDs, 'FirstUID:SecondUID:ThirdUID',
+        assert.strictEqual(serializedUIDs, 'FirstUID:SecondUID:ThirdUID', 'Serialized UID List should match expected value.');
-            'Serialized UID List should match expected value.');
         done();
     });
 
-    it('unserializes the expected number of UIDs', done => {
+    it('unserializes the expected number of UIDs', (done) => {
-        const refUidList = ['FirstUID', 'SecondUID', 'ThirdUID'];
+        const refUidList = [ 'FirstUID', 'SecondUID', 'ThirdUID'];
         const unserializedUIDs = unserializeUids('FirstUID:SecondUID:ThirdUID');
-        assert.deepStrictEqual(unserializedUIDs, refUidList,
+        assert.deepStrictEqual(unserializedUIDs, refUidList, 'Unserialized UID List should match expected value.');
-            'Unserialized UID List should match expected value.');
         done();
     });
 });
 
-describe('Utils: objectCopy', () => {
+describe('Utils: copyFields', () => {
-    it('copies all the properties from source to target object', done => {
+    it('copies all the properties from source to target object', (done) => {
-        const target = { foo: 'bar' };
+        const target = new Map();
+        target.set('foo', 'bar');
         const source = { id: 1, name: 'demo', value: { a: 1, b: 2, c: 3 } };
-        const result = {
+        const result = new Map();
-            foo: 'bar',
+        result.set('foo', 'bar');
-            id: 1,
+        result.set('id', 1);
-            name: 'demo',
+        result.set('name', 'demo');
-            value: { a: 1, b: 2, c: 3 },
+        result.set('value', { a: 1, b: 2, c: 3 });
-        };
+        copyFields(target, source);
-        objectCopy(target, source);
+        assert.deepStrictEqual(target, result, 'target should have the same properties as source');
-        assert.deepStrictEqual(target, result,
-            'target should have the same properties as source');
         done();
     });
-
-    it('copies all the properties from multiple sources to target object',
-        done => {
-            const target = { foo: 'bar' };
-            const source1 = {
-                id: 1,
-                name: 'demo1',
-                value: { a: 1, b: 2, c: 3 },
-            };
-            // eslint-disable-next-line camelcase
-            const source2 = {
-                req_id: 2,
-                method: 'test',
-                err: { code: 'error', msg: 'test' },
-            };
-            const result = {
-                foo: 'bar',
-                id: 1,
-                name: 'demo1',
-                value: { a: 1, b: 2, c: 3 },
-                // eslint-disable-next-line camelcase
-                req_id: 2,
-                method: 'test',
-                err: { code: 'error', msg: 'test' },
-            };
-            objectCopy(target, source1, source2);
-            assert.deepStrictEqual(target, result,
-                'target should have the same properties as source');
-            done();
-        });
 });

tests/unit/fixtures/stderrUtils/catchStderr.js

@@ -1,17 +0,0 @@
-#!/usr/bin/env node
-// Convert string args into primitive value
-const fromStr = (str, primitive) => (str === `${primitive}` ? primitive : str);
-const date = fromStr(process.argv[2], undefined);
-const exitCode = fromStr(fromStr(process.argv[3], null), undefined);
-
-const { stderrUtils } = require('../../../../index');
-
-stderrUtils.catchAndTimestampStderr(
-    date ? () => date : undefined,
-    exitCode,
-);
-
-process.emitWarning('TestWarningMessage');
-// This will print warning after printing error before exit
-throw new Error('TestingError');
-

tests/unit/fixtures/stderrUtils/catchUncaughtException.js

@@ -1,23 +0,0 @@
-#!/usr/bin/env node
-// Convert string args into primitive value
-const fromStr = (str, primitive) => (str === `${primitive}` ? primitive : str);
-const date = fromStr(process.argv[2], undefined);
-const exitCode = fromStr(fromStr(process.argv[3], null), undefined);
-const promise = fromStr(process.argv[4], true);
-
-const { stderrUtils } = require('../../../../index');
-
-stderrUtils.catchAndTimestampUncaughtException(
-    date ? () => date : undefined,
-    exitCode,
-);
-
-// Executed if process does not exit, process is in undefined behavior (bad)
-// eslint-disable-next-line no-console
-setTimeout(() => console.log('EXECUTED AFTER UNCAUGHT EXCEPTION'), 1);
-
-if (promise === true) {
-    Promise.reject();
-} else {
-    throw new Error('TestingError');
-}

tests/unit/fixtures/stderrUtils/catchWarning.js

@@ -1,38 +0,0 @@
-#!/usr/bin/env node
-// Convert string args into primitive value
-const fromStr = (str, primitive) => (str === `${primitive}` ? primitive : str);
-const date = fromStr(process.argv[2], undefined);
-const name = fromStr(process.argv[3], undefined);
-const code = fromStr(process.argv[4], undefined);
-const detail = fromStr(process.argv[5], undefined);
-
-const { stderrUtils } = require('../../../../index');
-
-stderrUtils.catchAndTimestampWarning(
-    date ? () => date : undefined,
-);
-
-const warning = new Error('TestWarningMessage');
-
-if (name) warning.name = name;
-if (code) warning.code = code;
-if (detail) warning.detail = detail;
-
-process.emitWarning(warning);
-
-/*
-Examples:
-
-(node:203831) Error: TestWarningMessage
-    at Object.<anonymous> (catchWarning.js:15:17)
-    ...
-    at node:internal/main/run_main_module:22:47
-Above Warning Date: 2024-06-26T16:32:55.505Z
-
-(node:205151) [TEST01] CUSTOM: TestWarningMessage
-    at Object.<anonymous> (catchWarning.js:15:17)
-    ...
-    at node:internal/main/run_main_module:22:47
-Some additional detail
-Above Warning Date: Tue, 31 Dec 2024 10:20:30 GMT
-*/

tests/unit/stderrUtils.js

@@ -1,309 +0,0 @@
-const assert = require('assert');
-const { execFile } = require('child_process');
-
-const stderrUtils = require('../../lib/stderrUtils');
-
-/** Simple regex for ISO YYYY-MM-DDThh:mm:ss.sssZ */
-// eslint-disable-next-line max-len
-const defaultDateRegex = /\d{4}-[01]\d-[0-3]\dT[0-2]\d:[0-5]\d:[0-5]\d\.\d+(?:[+-][0-2]\d:[0-5]\d|Z)/;
-
-// eslint-disable-next-line valid-jsdoc
-/** another format: Tue, 31 Dec 2024 10:20:30 GMT */
-const customDate = () => new Date('2024-12-31T10:20:30.444Z').toUTCString();
-
-describe('stderrUtils', () => {
-    const errStackRegex = /Error: TestingError\n(?:.*\sat\s.*\n)+/;
-
-    describe('defaultTimestamp', () => {
-        it('should match ISO format', () => {
-            assert.match(stderrUtils.defaultTimestamp(), defaultDateRegex);
-        });
-    });
-
-    describe('printErrorWithTimestamp', () => {
-        let stderrText;
-        const originalStderrWrite = process.stderr.write;
-        const mockedStderrWrite = text => { stderrText = text; return true; };
-        const err = new Error('TestingError');
-        const origin = 'uncaughtException';
-
-        beforeEach(() => {
-            stderrText = undefined;
-            process.stderr.write = mockedStderrWrite;
-        });
-
-        afterEach(() => {
-            process.stderr.write = originalStderrWrite;
-            stderrText = undefined;
-        });
-
-        it(
-            'should write to stderr with current date, origin and stacktrace',
-            () => {
-                const written = stderrUtils
-                    .printErrorWithTimestamp(err, origin);
-
-                assert.strictEqual(written, true);
-                const [firstLine, errStack] = stderrText.split(':\n');
-                const [errDate, errOrigin] = firstLine.split(': ');
-
-                assert.match(errDate, defaultDateRegex);
-                assert.strictEqual(errOrigin, origin);
-                assert.strictEqual(errStack, `${err.stack}\n`);
-            },
-        );
-
-        it(
-            'should write to stderr with custom date, origin and stacktrace',
-            () => {
-                const written = stderrUtils
-                    .printErrorWithTimestamp(err, origin, customDate());
-
-                assert.strictEqual(written, true);
-                const [firstLine, errStack] = stderrText.split(':\n');
-                const [errDate, errOrigin] = firstLine.split(': ');
-
-                assert.strictEqual(errDate, customDate());
-                assert.strictEqual(errOrigin, origin);
-                assert.strictEqual(errStack, `${err.stack}\n`);
-            },
-        );
-    });
-
-    const execOptions = {
-        cwd: __dirname,
-        // Subprocess should always stop alone
-        // But just in case, kill subprocess after 500ms.
-        // Leave enough time for `nyc` that runs slower.
-        timeout: 500,
-    };
-
-    // Execute in another process to notice the process exit
-    // Therefore, looks more like a functional test
-    const timeoutHint = (ms, retries) =>
-        `Test fixture process timed out after ${ms}ms with ${retries} retries.\n` +
-        'Due to nyc coverage first run slowing down process.\nIncrease execOptions.timeout to fix';
-
-    describe('catchAndTimestampUncaughtException', () => {
-        [
-            { desc: 'with default date' },
-            { desc: 'with custom date', date: customDate() },
-            { desc: 'with custom exitCode 42', exitCode: 42 },
-            { desc: 'without exit on uncaught exception', exitCode: null },
-            { desc: 'for unhandled promise', promise: true },
-        ].forEach(({
-            desc, date, exitCode, promise,
-        }) => describe(desc, () => {
-            /** for before all hook that doesn't support this.retries */
-            let retries = 4;
-            let err;
-            let stdout;
-            let stderr;
-            let errStack;
-            let errDate;
-            let errOrigin;
-
-            before('run process catchUncaughtException', function beforeAllHook(done) {
-                execFile(
-                    './fixtures/stderrUtils/catchUncaughtException.js',
-                    [`${date}`, `${exitCode}`, `${promise}`],
-                    execOptions,
-                    (subErr, subStdout, subStderr) => {
-                        if (subErr?.killed) {
-                            retries--;
-                            if (retries <= 0) {
-                                assert.fail(timeoutHint(execOptions.timeout, retries));
-                            }
-                            execOptions.timeout *= 2;
-                            return beforeAllHook(done);
-                        }
-                        err = subErr;
-                        stdout = subStdout;
-                        stderr = subStderr;
-                        let firstLine;
-                        [firstLine, errStack] = stderr.split(':\n');
-                        [errDate, errOrigin] = firstLine.split(': ');
-                        done();
-                    },
-                );
-            });
-
-            if (exitCode === null) {
-                it('should not be an error (or timeout)',
-                    () => assert.ifError(err));
-                it('should have stdout (printed after uncaught exception)',
-                    () => assert.match(stdout,
-                        /^.*EXECUTED AFTER UNCAUGHT EXCEPTION(?:.|\n)*$/));
-            } else {
-                it('should be an error',
-                    () => assert.ok(err));
-                it(`should have exitCode ${exitCode || 1}`,
-                    () => assert.strictEqual(err.code, exitCode || 1));
-                it('should have empty stdout',
-                    () => assert.strictEqual(stdout, ''));
-            }
-
-            it('should have stderr',
-                () => assert.ok(stderr));
-            it('should have date in stderr first line',
-                () => (date
-                    ? assert.strictEqual(errDate, date)
-                    : assert.match(errDate, defaultDateRegex)));
-
-            it('should have origin in stderr first line',
-                () => (promise === true
-                    ? assert.strictEqual(errOrigin, 'unhandledRejection')
-                    : assert.strictEqual(errOrigin, 'uncaughtException')));
-
-            if (!promise) {
-                it('should have stack trace on stderr',
-                    () => assert.match(errStack, errStackRegex));
-            }
-        }));
-    });
-
-    describe('catchAndTimestampWarning (also tests node onWarning)', () => {
-        [
-            { desc: 'with default date' },
-            { desc: 'with custom date', date: customDate() },
-            { desc: 'with deprecation warning', name: 'DeprecationWarning' },
-            {
-                desc: 'with custom warning',
-                name: 'CUSTOM',
-                code: 'TEST01',
-                detail: 'Some additional detail',
-            },
-        ].forEach(({
-            desc, date, name, code, detail,
-        }) => describe(desc, () => {
-            /** for before all hook that doesn't support this.retries */
-            let retries = 4;
-            let err;
-            let stdout;
-            let stderr;
-
-            before('run process catchWarning', function beforeAllHook(done) {
-                execFile(
-                    './fixtures/stderrUtils/catchWarning.js',
-                    [`${date}`, `${name}`, `${code}`, `${detail}`],
-                    execOptions,
-                    (subErr, subStdout, subStderr) => {
-                        if (subErr?.killed) {
-                            retries--;
-                            if (retries <= 0) {
-                                assert.fail(timeoutHint(execOptions.timeout, retries));
-                            }
-                            execOptions.timeout *= 2;
-                            return beforeAllHook(done);
-                        }
-                        err = subErr;
-                        stdout = subStdout;
-                        stderr = subStderr;
-                        done();
-                    },
-                );
-            });
-
-            it('should not be an error (or timeout)',
-                () => assert.ifError(err));
-            it('should have empty stdout',
-                () => assert.strictEqual(stdout, ''));
-            it('should have stderr',
-                () => assert.ok(stderr));
-            it('should have message on stderr first line, then stack trace',
-                () => assert.match(stderr,
-                    /^.*TestWarningMessage\n(?:\s+at\s.*\n)+/));
-
-            if (code) {
-                it('should have code on stderr first line',
-                    () => assert.match(stderr, new RegExp(`^.*[${code}]`)));
-            }
-
-            if (name) {
-                it('should have name on stderr first line',
-                    () => assert.match(stderr, new RegExp(`^.*${name}:`)));
-            }
-
-            if (detail) {
-                it('should have detail on stderr',
-                    () => assert.match(stderr, new RegExp(`.*${detail}.*`)));
-            }
-
-            it(`should have ${date ? 'custom' : 'default'} date on stderr`,
-                () => assert.match(stderr, new RegExp(
-                    `\nAbove Warning Date: ${
-                        date || defaultDateRegex.source}\n`,
-                )));
-        }));
-    });
-
-    describe('catchAndTimestampStderr', () => {
-        [
-            { desc: 'with default date' },
-            { desc: 'with custom date', date: customDate() },
-            { desc: 'with exit code', exitCode: 42 },
-
-        ].forEach(({
-            desc, date, exitCode,
-        }) => describe(desc, () => {
-            /** for before all hook that doesn't support this.retries */
-            let retries = 4;
-            let err;
-            let stdout;
-            let stderr;
-
-            before('run process catchStderr', function beforeAllHook(done) {
-                execFile(
-                    './fixtures/stderrUtils/catchStderr.js',
-                    [`${date}`, `${exitCode}`],
-                    execOptions,
-                    (subErr, subStdout, subStderr) => {
-                        if (subErr?.killed) {
-                            retries--;
-                            if (retries <= 0) {
-                                assert.fail(timeoutHint(execOptions.timeout, retries));
-                            }
-                            execOptions.timeout *= 2;
-                            return beforeAllHook(done);
-                        }
-                        err = subErr;
-                        stdout = subStdout;
-                        stderr = subStderr;
-                        done();
-                    },
-                );
-            });
-
-            it('should be an error',
-                () => assert.ok(err));
-            it(`should have exitCode ${exitCode || 1}`,
-                () => assert.strictEqual(err.code, exitCode || 1));
-            it('should have empty stdout',
-                () => assert.strictEqual(stdout, ''));
-
-            it('should have stderr',
-                () => assert.ok(stderr));
-
-            // 2024-06-26T15:04:55.364Z: uncaughtException:
-            // Error: TestingError
-            //     at Object.<anonymous> (catchStderr.js:16:7)
-            //     at node:internal/main/run_main_module:22:47
-            it('should have error date, origin and stacktrace in stderr',
-                () => assert.match(stderr,
-                    new RegExp(`${date || defaultDateRegex.source
-                    }: uncaughtException:\n${errStackRegex.source}`)));
-
-            // (node:171245) Warning: TestWarningMessage
-            //     at Object.<anonymous> (catchStderr.js:14:9)
-            //     at node:internal/main/run_main_module:22:47
-            // Above Warning Date: 2024-06-26T15:04:55.365Z
-            it('should have warning with stacktrace in stderr', () => {
-                const trace = 'Warning: TestWarningMessage\n(?:\\s+at\\s.*\n)+';
-                const detail = `(?:.|\n)*?(?<=\n)Above Warning Date: ${
-                    date || defaultDateRegex.source}\n`;
-                assert.match(stderr,
-                    new RegExp(`${trace}${detail}`));
-            });
-        }));
-    });
-});