Skip to content

Commit 4c32c7e

Browse files
committed
[ZEPPELIN-6212] Fix IRInterpreterTest.testZShow() for R 4.0+ compatibility
1 parent c322a8d commit 4c32c7e

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

rlang/src/test/java/org/apache/zeppelin/r/IRInterpreterTest.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,8 @@ public void testZShow() throws InterpreterException, IOException {
5858
InterpreterResult result = interpreter.interpret(
5959
"df=data.frame(country=c(\"US\", \"GB\", \"BR\"),\n" +
6060
"val1=c(10,13,14),\n" +
61-
"val2=c(23,12,32))", context);
61+
"val2=c(23,12,32),\n" +
62+
"stringsAsFactors = FALSE)", context);
6263
assertEquals(InterpreterResult.Code.SUCCESS, result.code());
6364

6465
context = getInterpreterContext();
@@ -69,9 +70,9 @@ public void testZShow() throws InterpreterException, IOException {
6970
assertEquals(InterpreterResult.Type.TABLE, resultMessages.get(0).getType(),
7071
resultMessages.toString());
7172
assertEquals("country\tval1\tval2\n" +
72-
"3\t10\t23\n" +
73-
"2\t13\t12\n" +
74-
"1\t14\t32\n",
73+
"US\t10\t23\n" +
74+
"GB\t13\t12\n" +
75+
"BR\t14\t32\n",
7576
resultMessages.get(0).getData());
7677

7778
context = getInterpreterContext();
@@ -81,7 +82,7 @@ public void testZShow() throws InterpreterException, IOException {
8182
assertEquals(2, resultMessages.size());
8283
assertEquals(InterpreterResult.Type.TABLE, resultMessages.get(0).getType(), resultMessages.toString());
8384
assertEquals("country\tval1\tval2\n" +
84-
"3\t10\t23\n",
85+
"US\t10\t23\n",
8586
resultMessages.get(0).getData());
8687
assertEquals(InterpreterResult.Type.HTML, resultMessages.get(1).getType(),
8788
resultMessages.toString());

0 commit comments

Comments
 (0)