Skip to content

Commit

Permalink
fix: respect client.logging option
Browse files Browse the repository at this point in the history
  • Loading branch information
snitin315 committed Sep 16, 2022
1 parent 87072c7 commit d1d9d47
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 8 deletions.
2 changes: 1 addition & 1 deletion client-src/index.js
Expand Up @@ -94,7 +94,7 @@ if (typeof parsedResourceQuery.reconnect !== "undefined") {
options.reconnect = Number(parsedResourceQuery.reconnect);
}

logEnabledFeatures(enabledFeatures);
logEnabledFeatures(enabledFeatures, options.logging);

/**
* @param {string} level
Expand Down
3 changes: 2 additions & 1 deletion client-src/utils/log.js
Expand Up @@ -18,7 +18,8 @@ setLogLevel(defaultLevel);

const log = logger.getLogger(name);

const logEnabledFeatures = (features) => {
const logEnabledFeatures = (features, logLevel = "info") => {
setLogLevel(logLevel);
const enabledFeatures = Object.keys(features);
if (!features || enabledFeatures.length === 0) {
return;
Expand Down
6 changes: 0 additions & 6 deletions test/e2e/__snapshots__/logging.test.js.snap.webpack5
Expand Up @@ -116,7 +116,6 @@ Array [

exports[`logging should work and log only error (sockjs) 1`] = `
Array [
"[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.",
"Hey.",
"[webpack-dev-server] Errors while compiling. Reload prevented.",
"[webpack-dev-server] ERROR
Expand All @@ -126,7 +125,6 @@ Error from compilation",

exports[`logging should work and log only error (ws) 1`] = `
Array [
"[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.",
"Hey.",
"[webpack-dev-server] Errors while compiling. Reload prevented.",
"[webpack-dev-server] ERROR
Expand Down Expand Up @@ -160,7 +158,6 @@ Array [

exports[`logging should work and log warning and errors (sockjs) 1`] = `
Array [
"[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.",
"Hey.",
"[webpack-dev-server] Warnings while compiling.",
"[webpack-dev-server] WARNING
Expand All @@ -173,7 +170,6 @@ Error from compilation",

exports[`logging should work and log warning and errors (ws) 1`] = `
Array [
"[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.",
"Hey.",
"[webpack-dev-server] Warnings while compiling.",
"[webpack-dev-server] WARNING
Expand Down Expand Up @@ -240,14 +236,12 @@ Array [

exports[`logging should work when the "client.logging" is "none" (sockjs) 1`] = `
Array [
"[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.",
"Hey.",
]
`;

exports[`logging should work when the "client.logging" is "none" (ws) 1`] = `
Array [
"[webpack-dev-server] Server started: Hot Module Replacement enabled, Live Reloading enabled, Progress disabled, Overlay enabled.",
"Hey.",
]
`;
Expand Down

0 comments on commit d1d9d47

Please sign in to comment.