resolve comments

wbo4958 · wbo4958 · commit 7868d8d54e9c · 2020-04-07T19:30:21.000+08:00
diff --git a/jvm-packages/xgboost4j/src/test/java/ml/dmlc/xgboost4j/java/DMatrixTest.java b/jvm-packages/xgboost4j/src/test/java/ml/dmlc/xgboost4j/java/DMatrixTest.java
@@ -75,11 +75,13 @@ public void testCreateFromDataIteratorWithDiffFeatureSize() throws XGBoostError
       blist.add(p);
       labelall.add(p.label());
     }
+    boolean success = true;
     try {
       DMatrix dmat = new DMatrix(blist.iterator(), null);
     } catch (XGBoostError e) {
-
+      success = false;
     }
+    TestCase.assertTrue(success == false);
   }
 
   @Test

Original file line number	Diff line number	Diff line change
`@@ -75,11 +75,13 @@ public void testCreateFromDataIteratorWithDiffFeatureSize() throws XGBoostError`
`75`	`75`	`blist.add(p);`
`76`	`76`	`labelall.add(p.label());`
`77`	`77`	`}`
	`78`	`+ boolean success = true;`
`78`	`79`	`try {`
`79`	`80`	`DMatrix dmat = new DMatrix(blist.iterator(), null);`
`80`	`81`	`} catch (XGBoostError e) {`
`81`		`-`
	`82`	`+ success = false;`
`82`	`83`	`}`
	`84`	`+ TestCase.assertTrue(success == false);`
`83`	`85`	`}`
`84`	`86`
`85`	`87`	`@Test`