René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Pino's deprecated prettyPrint and its problem with Jest","articleBody":"I have the solution to this issue, I just wanted to share it.\r\n\r\nI noticed when working on the previous chapter of this tutorial that pino complains saying that prettyPrint is deprecated for the logger().\r\n\r\n```\r\n(node:54078) [PINODEP008] PinoWarning: prettyPrint is deprecated, look at https://github.com/pinojs/pino-pretty for alternatives.\r\n```\r\n\r\nSo, I found that the preferred method is using a transport option, like so:\r\n\r\nBefore\r\n```\r\nconst log = logger({\r\n  prettyPrint: true,\r\n  base: {\r\n    pid: false,\r\n  },\r\n  timestamp: () =\u003e `,\"time\":\"${dayjs().format()}\"`,\r\n});\r\n```\r\nAfter\r\n```\r\nconst log = logger({\r\n  transport: {\r\n    target: \"pino-pretty\",\r\n    options: {\r\n      colorize: true,\r\n    },\r\n  },\r\n  base: {\r\n    pid: false,\r\n  },\r\n  timestamp: () =\u003e `,\"time\":\"${dayjs().format()}\"`,\r\n});\r\n```\r\n\r\nBut starting on this tutorial for testing I found a head-scratching error faced me when using `--detectOpenHandles`:\r\n```\r\nJest has detected the following 1 open handle potentially keeping Jest from exiting:\r\n\r\n  ●  WORKER\r\n\r\n      4 | const testMode = process.env.NODE_ENV === \"test\";\r\n      5 |\r\n    \u003e 6 | const log = logger({\r\n        |                   ^\r\n```\r\n\r\nAfter a short amount of Googling I found that the transport option was to blame. To check, I restored `prettyPrint: true`, removed `transport: {...}` and the error disappeared.\r\n\r\nI don't know if this is perhaps the best method to fix it, but noticing that the env variable `NODE_ENV` was set when running jest (which is then utilized by `config` to switch to a different config file, `test.js`) I decided to take advantage of it and wrote this adjustment:\r\n\r\n```\r\n// NODE_ENV is set to \"test\" when jest is running.\r\nconst inTest = process.env.NODE_ENV === \"test\";\r\n\r\nconst log = logger({\r\n  enabled: !inTest,\r\n  transport: !inTest ? {\r\n    target: \"pino-pretty\",\r\n    options: {\r\n      colorize: true,\r\n    },\r\n  } : undefined,\r\n  base: {\r\n    pid: false,\r\n  },\r\n  timestamp: () =\u003e `,\"time\":\"${dayjs().format()}\"`,\r\n});\r\n```\r\nThat solved it and I hope this is helpful.","author":{"url":"https://github.com/TiE23","@type":"Person","name":"TiE23"},"datePublished":"2022-04-14T18:57:42.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/3/Testing-Express-REST-API/issues/3"}