Skip to content

Commit dbacebf

Browse files
only reverses ordering for how='right'
1 parent f0939a8 commit dbacebf

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

inst/tests/mergelist.Rraw

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -332,15 +332,21 @@ local({
332332
lhs = data.table(id1=1:2, v1=1:2),
333333
rhs = data.table(id1=2:1, v2=2:1)
334334
)
335-
expected = data.table(id1=1:2, v1=1:2, v2=1:2)
335+
expected = list(
336+
inner = data.table(id1=1:2, v1=1:2, v2=1:2),
337+
left = data.table(id1=1:2, v1=1:2, v2=1:2),
338+
right = data.table(id1=2:1, v1=2:1, v2=2:1),
339+
full = data.table(id1=1:2, v1=1:2, v2=1:2)
340+
)
336341
frac = 0.0
337342
for (how in c("inner", "left", "right", "full")) {
338343
frac = frac + 0.1
339344
for (mult in c("all", "first", "last", "error")) {
340345
frac = frac + 0.01
341-
test(29 + (frac <- frac + 0.001), ans <- mergepair(l$lhs, l$rhs, on="id1", how=how, mult=mult, copy=TRUE), expected)
346+
cat(sprintf("\nhow=%s, mult=%s, frac=%.04f\n", how, mult, frac))
347+
test(29 + (frac <- frac + 0.001), ans <- mergepair(l$lhs, l$rhs, on="id1", how=how, mult=mult, copy=TRUE), expected[[how]])
342348
test(29 + (frac <- frac + 0.001), copied(ans, l))
343-
test(29 + (frac <- frac + 0.001), ans <- mergepair(l$lhs, l$rhs, on="id1", how=how, mult=mult, copy=FALSE), expected)
349+
test(29 + (frac <- frac + 0.001), ans <- mergepair(l$lhs, l$rhs, on="id1", how=how, mult=mult, copy=FALSE), expected[[how]])
344350
test(29 + (frac <- frac + 0.001), notcopied(ans, l, how=how))
345351
}
346352
}

0 commit comments

Comments
 (0)