Skip to content

Commit

Permalink
tests: reenable joinp join validation now that its working again …
Browse files Browse the repository at this point in the history
…in Polars 0.38.2
  • Loading branch information
jqnatividad committed Mar 11, 2024
1 parent b299956 commit 5faf90e
Showing 1 changed file with 13 additions and 13 deletions.
26 changes: 13 additions & 13 deletions tests/test_joinp.rs
Expand Up @@ -369,19 +369,19 @@ joinp_test_comments!(
}
);

// joinp_test!(
// joinp_outer_left_validate_manytoone,
// |wrk: Workdir, mut cmd: process::Command| {
// cmd.arg("--left").args(["--validate", "onetomany"]).arg("--low-memory");
// let got: String = wrk.output_stderr(&mut cmd);
// assert_eq!(
// got,
// "Polars error: ComputeError(ErrString(\"the join keys did not fulfil m:1 \
// validation\"))\n"
// );
// wrk.assert_err(&mut cmd);
// }
// );
joinp_test!(
joinp_outer_left_validate_manytoone,
|wrk: Workdir, mut cmd: process::Command| {
cmd.arg("--left").args(["--validate", "manytoone"]);
let got: String = wrk.output_stderr(&mut cmd);
assert_eq!(
got,
"Polars error: ComputeError(ErrString(\"the join keys did not fulfil m:1 \
validation\"))\n"
);
wrk.assert_err(&mut cmd);
}
);

joinp_test!(joinp_full, |wrk: Workdir, mut cmd: process::Command| {
cmd.arg("--full");
Expand Down

0 comments on commit 5faf90e

Please sign in to comment.