must have tested traceOut after coerce fix

Author: archmoj

src/traces/mesh3d/defaults.js

@@ -39,6 +39,7 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         return;
     }
 
+    readComponents(['i', 'j', 'k']);
     // three indices should be all provided or not
     if(
         (traceOut.i && (!traceOut.j || !traceOut.k)) ||
@@ -48,7 +49,6 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         traceOut.visible = false;
         return;
     }
-    readComponents(['i', 'j', 'k']);
 
     var handleCalendarDefaults = Registry.getComponentMethod('calendars', 'handleTraceDefaults');
     handleCalendarDefaults(traceIn, traceOut, ['x', 'y', 'z'], layout);

test/jasmine/tests/mesh3d_test.js

@@ -386,7 +386,7 @@ describe('Test mesh3d', function() {
             .then(done);
         });
 
-        it('@gl mesh3d should be visible when the index arrays are not arrays - string case', function(done) {
+        it('@gl mesh3d should be invisible when the index arrays are not arrays - string case', function(done) {
             Plotly.plot(gd, [{
                 x: [0, 1, 0.5, 0.5],
                 y: [0, 0.5, 1, 0.5],
@@ -397,19 +397,13 @@ describe('Test mesh3d', function() {
                 type: 'mesh3d'
             }])
             .then(function() {
-                assertVisibility(true, 'to be invisible');
-            })
-            .then(function() {
-                assertPositions(4, 'to be OK positions');
-            })
-            .then(function() {
-                assertCells(3, 'to be OK cells');
+                assertVisibility(false, 'to be visible');
             })
             .catch(failTest)
             .then(done);
         });
 
-        it('@gl mesh3d should be visible when the index arrays are not arrays - object case', function(done) {
+        it('@gl mesh3d should be invisible when the index arrays are not arrays - object case', function(done) {
             Plotly.plot(gd, [{
                 x: [0, 1, 0.5, 0.5],
                 y: [0, 0.5, 1, 0.5],
@@ -420,19 +414,13 @@ describe('Test mesh3d', function() {
                 type: 'mesh3d'
             }])
             .then(function() {
-                assertVisibility(true, 'to be invisible');
-            })
-            .then(function() {
-                assertPositions(4, 'to be OK positions');
-            })
-            .then(function() {
-                assertCells(3, 'to be OK cells');
+                assertVisibility(false, 'to be visible');
             })
             .catch(failTest)
             .then(done);
         });
 
-        it('@gl mesh3d should be visible when the index arrays are not arrays - boolean case', function(done) {
+        it('@gl mesh3d should be invisible when the index arrays are not arrays - boolean case', function(done) {
             Plotly.plot(gd, [{
                 x: [0, 1, 0.5, 0.5],
                 y: [0, 0.5, 1, 0.5],
@@ -443,19 +431,13 @@ describe('Test mesh3d', function() {
                 type: 'mesh3d'
             }])
             .then(function() {
-                assertVisibility(true, 'to be invisible');
-            })
-            .then(function() {
-                assertPositions(4, 'to be OK positions');
-            })
-            .then(function() {
-                assertCells(3, 'to be OK cells');
+                assertVisibility(false, 'to be visible');
             })
             .catch(failTest)
             .then(done);
         });
 
-        it('@gl mesh3d should be visible when the index arrays are not arrays - number case', function(done) {
+        it('@gl mesh3d should be invisible when the index arrays are not arrays - number case', function(done) {
             Plotly.plot(gd, [{
                 x: [0, 1, 0.5, 0.5],
                 y: [0, 0.5, 1, 0.5],
@@ -466,13 +448,7 @@ describe('Test mesh3d', function() {
                 type: 'mesh3d'
             }])
             .then(function() {
-                assertVisibility(true, 'to be invisible');
-            })
-            .then(function() {
-                assertPositions(4, 'to be OK positions');
-            })
-            .then(function() {
-                assertCells(3, 'to be OK cells');
+                assertVisibility(false, 'to be visible');
             })
             .catch(failTest)
             .then(done);