Skip to content

Commit

Permalink
test: Ignore c3 data_types null crashes
Browse files Browse the repository at this point in the history
This gets triggered by the CPUUsage DonutChart in
pkg/kubernetes/scripts/virtual-machines/components/VmMetricsTab.jsx.

This seems to be some race condition when switching pages, which
supposedly also means "(un)render in React". Cockpit code has no control
over this, and it also does not break the user experience, so ignore the
exception.

See c3js/c3#2187 and simlilar issues
bcbcarl/react-c3js#22 or
c3js/c3#1205.
  • Loading branch information
martinpitt authored and larskarlitski committed Jul 27, 2018
1 parent 9d3c17b commit 436bfe4
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion test/common/cdp-driver.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,15 @@ function setupLogging(client) {
if (details.exception && details.exception.className === "PhWaitCondTimeout")
return;

unhandledExceptions.push(details)
process.stderr.write(details.description || JSON.stringify(details) + "\n");

// ignore c3 crashes (https://github.com/c3js/c3/issues/2187)
if (details.exception && details.exception.description &&
details.exception.description.indexOf("TypeError: Cannot read property 'data_types' of null") >= 0 &&
details.exception.description.indexOf("/kubernetes.js") >= 0)
return;

unhandledExceptions.push(details)
});

client.Log.enable();
Expand Down

0 comments on commit 436bfe4

Please sign in to comment.