Merge pull request #16 from DataDog/yoann/fix-output-hook

Fix renderer hook

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@datadog/build-plugin",
-    "version": "0.3.1",
+    "version": "0.3.2",
     "license": "MIT",
     "author": "Datadog",
     "repository": "git://github.com/DataDog/build-plugin.git",

src/hooks/__tests__/outputFiles.test.ts

@@ -46,6 +46,7 @@ describe('Output Files', () => {
         // Cleaning
         await Promise.all(getRemoveProms(output));
     });
+
     test('It should allow a relative path', async () => {
         // Relative path
         const output = './test2/';
@@ -57,4 +58,9 @@ describe('Output Files', () => {
         // Cleaning
         await Promise.all(getRemoveProms(path.join(__dirname, output)));
     });
+
+    test('It should export hooks', () => {
+        const outputFiles = require('../outputFiles');
+        expect(typeof outputFiles.hooks).toBe('object');
+    });
 });

src/hooks/__tests__/renderer.test.ts

@@ -1,5 +1,5 @@
 describe('Renderer', () => {
-    test('it should outputGenerals the same with Webpack 5 and 4', () => {
+    test('It should outputGenerals the same with Webpack 5 and 4', () => {
         const { outputGenerals } = require('../renderer');
 
         const ar = [{ name: 'element1' }, { name: 'element2' }];
@@ -41,4 +41,9 @@ describe('Renderer', () => {
 
         expect(outputWebpack4).toBe(outputWebpack5);
     });
+
+    test('It should export hooks', () => {
+        const renderer = require('../renderer');
+        expect(typeof renderer.hooks).toBe('object');
+    });
 });

src/hooks/datadog/__tests__/index.test.ts

@@ -27,4 +27,9 @@ describe('Datadog Hook', () => {
 
         expect(typeof obj).toBe('object');
     });
+
+    test('It should export hooks', () => {
+        const datadog = require('../index');
+        expect(typeof datadog.hooks).toBe('object');
+    });
 });

src/hooks/datadog/index.ts

@@ -75,9 +75,7 @@ const postoutput = async function postoutput(
     return { metrics };
 };
 
-module.exports = {
-    hooks: {
-        preoutput,
-        postoutput,
-    },
+export const hooks = {
+    preoutput,
+    postoutput,
 };

src/hooks/outputFiles.ts

@@ -44,4 +44,4 @@ const output = async function output(this: BuildPlugin, { report, metrics, stats
     }
 };
 
-module.exports = { hooks: { output } };
+export const hooks = { output };

src/hooks/renderer.ts

@@ -134,16 +134,14 @@ const outputModules = (times: ResultModules, deps: LocalModules) => {
     render(modulesPerTime, (module) => formatDuration(module.duration));
 };
 
-export default {
-    hooks: {
-        async output(this: BuildPlugin, { report, stats }: HooksContext) {
-            if (this.options.output === false) {
-                return;
-            }
-            outputTapables(report.timings.tapables);
-            outputLoaders(report.timings.loaders);
-            outputModules(report.timings.modules, report.dependencies);
-            outputGenerals(stats);
-        },
+export const hooks = {
+    async output(this: BuildPlugin, { report, stats }: HooksContext) {
+        if (this.options.output === false) {
+            return;
+        }
+        outputTapables(report.timings.tapables);
+        outputLoaders(report.timings.loaders);
+        outputModules(report.timings.modules, report.dependencies);
+        outputGenerals(stats);
     },
 };