mirror of
https://github.com/nushell/nushell
synced 2024-12-30 15:03:25 +00:00
75180d07de
<!-- if this PR closes one or more issues, you can automatically link the PR with them by using one of the [*linking keywords*](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword), e.g. - this PR should close #xxxx - fixes #xxxx you can also mention related issues, PRs or discussions! --> # Description <!-- Thank you for improving Nushell. Please, check our [contributing guide](../CONTRIBUTING.md) and talk to the core team before making major changes. Description of your pull request goes here. **Provide examples and/or screenshots** if your changes affect the user experience. --> This PR is related to **Tests: clean up unnecessary use of cwd, pipeline(), etc. [#8670](https://github.com/nushell/nushell/issues/8670)** - Removed the `r#"..."#` raw string literal syntax, which is unnecessary when there are no special characters that need quoting from the tests that use the `nu!` macro. - `cwd:` and `pipeline()` has not changed # User-Facing Changes <!-- List of all changes that impact the user experience here. This helps us keep track of breaking changes. --> # Tests + Formatting <!-- Don't forget to add tests that cover your changes. Make sure you've run and fixed any issues with these commands: - `cargo fmt --all -- --check` to check standard code formatting (`cargo fmt --all` applies these changes) - `cargo clippy --workspace -- -D warnings -D clippy::unwrap_used -A clippy::needless_collect -A clippy::result_large_err` to check that you're using the standard code style - `cargo test --workspace` to check that all tests pass - `cargo run -- -c "use std testing; testing run-tests --path crates/nu-std"` to run the tests for the standard library > **Note** > from `nushell` you can also use the `toolkit` as follows > ```bash > use toolkit.nu # or use an `env_change` hook to activate it automatically > toolkit check pr > ``` --> # After Submitting <!-- If your PR had any user-facing changes, update [the documentation](https://github.com/nushell/nushell.github.io) after the PR is merged, if necessary. This will help us keep the docs up to date. -->
250 lines
6.5 KiB
Rust
250 lines
6.5 KiB
Rust
use nu_test_support::fs::Stub::FileWithContentToBeTrimmed;
|
|
use nu_test_support::playground::Playground;
|
|
use nu_test_support::{nu, pipeline};
|
|
|
|
#[test]
|
|
fn removes_duplicate_rows() {
|
|
Playground::setup("uniq_test_1", |dirs, sandbox| {
|
|
sandbox.with_files(vec![FileWithContentToBeTrimmed(
|
|
"los_tres_caballeros.csv",
|
|
r#"
|
|
first_name,last_name,rusty_at,type
|
|
Andrés,Robalino,10/11/2013,A
|
|
JT,Turner,10/12/2013,B
|
|
Yehuda,Katz,10/11/2013,A
|
|
JT,Turner,10/12/2013,B
|
|
Yehuda,Katz,10/11/2013,A
|
|
"#,
|
|
)]);
|
|
|
|
let actual = nu!(
|
|
cwd: dirs.test(), pipeline(
|
|
"
|
|
open los_tres_caballeros.csv
|
|
| uniq
|
|
| length
|
|
"
|
|
));
|
|
|
|
assert_eq!(actual.out, "3");
|
|
})
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_values() {
|
|
Playground::setup("uniq_test_2", |dirs, sandbox| {
|
|
sandbox.with_files(vec![FileWithContentToBeTrimmed(
|
|
"los_tres_caballeros.csv",
|
|
r#"
|
|
first_name,last_name,rusty_at,type
|
|
Andrés,Robalino,10/11/2013,A
|
|
JT,Turner,10/12/2013,B
|
|
Yehuda,Katz,10/11/2013,A
|
|
JT,Turner,10/12/2013,B
|
|
Yehuda,Katz,10/11/2013,A
|
|
"#,
|
|
)]);
|
|
|
|
let actual = nu!(
|
|
cwd: dirs.test(), pipeline(
|
|
"
|
|
open los_tres_caballeros.csv
|
|
| select type
|
|
| uniq
|
|
| length
|
|
"
|
|
));
|
|
|
|
assert_eq!(actual.out, "2");
|
|
})
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_empty() {
|
|
let actual = nu!("[] | uniq | to nuon");
|
|
|
|
assert_eq!(actual.out, "[]");
|
|
}
|
|
|
|
#[test]
|
|
fn nested_json_structures() {
|
|
Playground::setup("uniq_test_3", |dirs, sandbox| {
|
|
sandbox.with_files(vec![FileWithContentToBeTrimmed(
|
|
"nested_json_structures.json",
|
|
r#"
|
|
[
|
|
{
|
|
"name": "this is duplicated",
|
|
"nesting": [ { "a": "a", "b": "b" },
|
|
{ "c": "c", "d": "d" }
|
|
],
|
|
"can_be_ordered_differently": {
|
|
"array": [1, 2, 3, 4, 5],
|
|
"something": { "else": "works" }
|
|
}
|
|
},
|
|
{
|
|
"can_be_ordered_differently": {
|
|
"something": { "else": "works" },
|
|
"array": [1, 2, 3, 4, 5]
|
|
},
|
|
"nesting": [ { "b": "b", "a": "a" },
|
|
{ "d": "d", "c": "c" }
|
|
],
|
|
"name": "this is duplicated"
|
|
},
|
|
{
|
|
"name": "this is unique",
|
|
"nesting": [ { "a": "b", "b": "a" },
|
|
{ "c": "d", "d": "c" }
|
|
],
|
|
"can_be_ordered_differently": {
|
|
"array": [],
|
|
"something": { "else": "does not work" }
|
|
}
|
|
},
|
|
{
|
|
"name": "this is unique",
|
|
"nesting": [ { "a": "a", "b": "b", "c": "c" },
|
|
{ "d": "d", "e": "e", "f": "f" }
|
|
],
|
|
"can_be_ordered_differently": {
|
|
"array": [],
|
|
"something": { "else": "works" }
|
|
}
|
|
}
|
|
]
|
|
"#,
|
|
)]);
|
|
|
|
let actual = nu!(
|
|
cwd: dirs.test(), pipeline(
|
|
"
|
|
open nested_json_structures.json
|
|
| uniq
|
|
| length
|
|
"
|
|
));
|
|
assert_eq!(actual.out, "3");
|
|
})
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_when_keys_out_of_order() {
|
|
let actual = nu!(pipeline(
|
|
r#"
|
|
[{"a": "a", "b": [1,2,3]}, {"b": [1,2,3], "a": "a"}]
|
|
| uniq
|
|
| length
|
|
"#
|
|
));
|
|
|
|
assert_eq!(actual.out, "1");
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_counting() {
|
|
let actual = nu!(pipeline(
|
|
r#"
|
|
["A", "B", "A"]
|
|
| wrap item
|
|
| uniq --count
|
|
| flatten
|
|
| where item == A
|
|
| get count
|
|
| get 0
|
|
"#
|
|
));
|
|
assert_eq!(actual.out, "2");
|
|
|
|
let actual = nu!(pipeline(
|
|
r#"
|
|
["A", "B", "A"]
|
|
| wrap item
|
|
| uniq --count
|
|
| flatten
|
|
| where item == B
|
|
| get count
|
|
| get 0
|
|
"#
|
|
));
|
|
assert_eq!(actual.out, "1");
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_unique() {
|
|
let actual = nu!("[1 2 3 4 1 5] | uniq --unique");
|
|
let expected = nu!("[2 3 4 5]");
|
|
assert_eq!(actual.out, expected.out);
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_simple_vals_ints() {
|
|
let actual = nu!("[1 2 3 4 1 5] | uniq");
|
|
let expected = nu!("[1 2 3 4 5]");
|
|
assert_eq!(actual.out, expected.out);
|
|
}
|
|
|
|
#[test]
|
|
fn uniq_simple_vals_strs() {
|
|
let actual = nu!("[A B C A] | uniq");
|
|
let expected = nu!("[A B C]");
|
|
assert_eq!(actual.out, expected.out);
|
|
}
|
|
|
|
#[test]
|
|
fn table() {
|
|
let actual = nu!(pipeline(
|
|
"
|
|
[[fruit day]; [apple monday] [apple friday] [Apple friday] [apple monday] [pear monday] [orange tuesday]]
|
|
| uniq
|
|
"
|
|
));
|
|
|
|
let expected = nu!("[[fruit day]; [apple monday] [apple friday] [Apple friday] [pear monday] [orange tuesday]]");
|
|
assert_eq!(actual.out, expected.out);
|
|
}
|
|
|
|
#[test]
|
|
fn table_with_ignore_case() {
|
|
let actual = nu!(pipeline(
|
|
r#"
|
|
[[origin, people];
|
|
[World, (
|
|
[[name, meal];
|
|
['Geremias', {plate: 'bitoque', carbs: 100}]
|
|
]
|
|
)],
|
|
[World, (
|
|
[[name, meal];
|
|
['Martin', {plate: 'bitoque', carbs: 100}]
|
|
]
|
|
)],
|
|
[World, (
|
|
[[name, meal];
|
|
['Geremias', {plate: 'Bitoque', carbs: 100}]
|
|
]
|
|
)],
|
|
] | uniq -i
|
|
"#
|
|
));
|
|
|
|
let expected = nu!(pipeline(
|
|
r#"
|
|
echo [[origin, people];
|
|
[World, (
|
|
[[name, meal];
|
|
['Geremias', {plate: 'bitoque', carbs: 100}]
|
|
]
|
|
)],
|
|
[World, (
|
|
[[name, meal];
|
|
['Martin', {plate: 'bitoque', carbs: 100}]
|
|
]
|
|
)],
|
|
]
|
|
"#
|
|
));
|
|
|
|
assert_eq!(actual.out, expected.out);
|
|
}
|