Unit tests

add missing parameter for percentiles #3246

Triggered via pull request August 28, 2023 10:59

PSeitz

opened #2157

test_missing_column

Status Success

Total duration 7m 49s

Artifacts –

test.yml

on: pull_request

check

1m 0s

Matrix: test

Annotations

44 warnings

an array of `Range` that is only one element: src/snippet/mod.rs#L684

warning: an array of `Range` that is only one element --> src/snippet/mod.rs:684:66 | 684 | assert_eq!(&collapse_overlapped_ranges(&[0..3, 1..2,]), &[0..3]); | ^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_range_in_vec_init help: if you wanted a `Vec` that contains the entire range, try | 684 | assert_eq!(&collapse_overlapped_ranges(&[0..3, 1..2,]), &(0..3).collect::<std::vec::Vec<usize>>()); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ help: if you wanted an array of len 3, try | 684 | assert_eq!(&collapse_overlapped_ranges(&[0..3, 1..2,]), &[0; 3]); | ~~~~

an array of `Range` that is only one element: src/snippet/mod.rs#L683

warning: an array of `Range` that is only one element --> src/snippet/mod.rs:683:66 | 683 | assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..3,]), &[0..3]); | ^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_range_in_vec_init help: if you wanted a `Vec` that contains the entire range, try | 683 | assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..3,]), &(0..3).collect::<std::vec::Vec<usize>>()); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ help: if you wanted an array of len 3, try | 683 | assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..3,]), &[0; 3]); | ~~~~

an array of `Range` that is only one element: src/snippet/mod.rs#L682

warning: an array of `Range` that is only one element --> src/snippet/mod.rs:682:66 | 682 | assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..2,]), &[0..2]); | ^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_range_in_vec_init = note: `#[warn(clippy::single_range_in_vec_init)]` on by default help: if you wanted a `Vec` that contains the entire range, try | 682 | assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..2,]), &(0..2).collect::<std::vec::Vec<usize>>()); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ help: if you wanted an array of len 2, try | 682 | assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..2,]), &[0; 2]); | ~~~~

useless use of `vec!`: src/query/phrase_query/mod.rs#L78

warning: useless use of `vec!` --> src/query/phrase_query/mod.rs:78:32 | 78 | let terms: Vec<Term> = vec!["a", "b", "c"] | ^^^^^^^^^^^^^^^^^^^ help: you can use an array directly: `["a", "b", "c"]` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec

useless use of `vec!`: src/positions/mod.rs#L122

warning: useless use of `vec!` --> src/positions/mod.rs:122:43 | 122 | let mut output_delta_pos_buffer = vec![0u32; 5]; | ^^^^^^^^^^^^^ help: you can use an array directly: `[0u32; 5]` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec

useless use of `vec!`: src/fastfield/mod.rs#L1016

warning: useless use of `vec!` --> src/fastfield/mod.rs:1016:23 | 1016 | let numbers = vec![1000, 1001, 1003]; | ^^^^^^^^^^^^^^^^^^^^^^ help: you can use an array directly: `[1000, 1001, 1003]` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec

useless use of `vec!`: src/fastfield/mod.rs#L942

warning: useless use of `vec!` --> src/fastfield/mod.rs:942:23 | 942 | let numbers = vec![100, 200, 300]; | ^^^^^^^^^^^^^^^^^^^ help: you can use an array directly: `[100, 200, 300]` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec

useless use of `vec!`: src/aggregation/metric/percentiles.rs#L502

warning: useless use of `vec!` --> src/aggregation/metric/percentiles.rs:502:37 | 502 | let num_values_in_segment = vec![100, 30_000, 8000]; | ^^^^^^^^^^^^^^^^^^^^^^^ help: you can use an array directly: `[100, 30_000, 8000]` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec = note: `#[warn(clippy::useless_vec)]` on by default

unnecessary hashes around raw string literal: src/schema/schema.rs#L518

warning: unnecessary hashes around raw string literal --> src/schema/schema.rs:518:50 | 518 | assert_eq!(&super::locate_splitting_dots(r#"a\..b.c"#), &[3, 5]); | ^^^^^^^^^^^^ help: try: `r"a\..b.c"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/schema/schema.rs#L517

warning: unnecessary hashes around raw string literal --> src/schema/schema.rs:517:50 | 517 | assert_eq!(&super::locate_splitting_dots(r#"a\.b.c"#), &[4]); | ^^^^^^^^^^^ help: try: `r"a\.b.c"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/query/query_parser/query_parser.rs#L1737

warning: unnecessary hashes around raw string literal --> src/query/query_parser/query_parser.rs:1737:46 | 1737 | let query = query_parser.parse_query(r#"a\.b:hello"#).unwrap(); | ^^^^^^^^^^^^^^^ help: try: `r"a\.b:hello"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/query/query_parser/query_parser.rs#L1734

warning: unnecessary hashes around raw string literal --> src/query/query_parser/query_parser.rs:1734:39 | 1734 | schema_builder.add_text_field(r#"a\.b"#, STRING); | ^^^^^^^^^ help: try: `r"a\.b"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/query/query_parser/query_parser.rs#L1308

warning: unnecessary hashes around raw string literal --> src/query/query_parser/query_parser.rs:1308:42 | 1308 | extract_query_term_json_path(r#"json.k8s\.node\.name:hello"#), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `r"json.k8s\.node\.name:hello"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/fastfield/mod.rs#L1101

warning: unnecessary hashes around raw string literal --> src/fastfield/mod.rs:1101:32 | 1101 | .column_opt::<i64>(r#"json.attr\.age"#) | ^^^^^^^^^^^^^^^^^^^ help: try: `r"json.attr\.age"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

private item shadows public glob re-export: src/aggregation/bucket/mod.rs#L33

warning: private item shadows public glob re-export --> src/aggregation/bucket/mod.rs:33:16 | 33 | pub(crate) use range::SegmentRangeCollector; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | note: the name `SegmentRangeCollector` in the type namespace is supposed to be publicly re-exported here --> src/aggregation/bucket/mod.rs:34:9 | 34 | pub use range::*; | ^^^^^^^^ note: but the private item here shadows it --> src/aggregation/bucket/mod.rs:33:16 | 33 | pub(crate) use range::SegmentRangeCollector; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

private item shadows public glob re-export: src/aggregation/bucket/mod.rs#L31

warning: private item shadows public glob re-export --> src/aggregation/bucket/mod.rs:31:16 | 31 | pub(crate) use histogram::SegmentHistogramCollector; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | note: the name `SegmentHistogramCollector` in the type namespace is supposed to be publicly re-exported here --> src/aggregation/bucket/mod.rs:32:9 | 32 | pub use histogram::*; | ^^^^^^^^^^^^ note: but the private item here shadows it --> src/aggregation/bucket/mod.rs:31:16 | 31 | pub(crate) use histogram::SegmentHistogramCollector; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ = note: `#[warn(hidden_glob_reexports)]` on by default

unnecessary hashes around raw string literal: src/indexer/mod.rs#L130

warning: unnecessary hashes around raw string literal --> src/indexer/mod.rs:130:30 | 130 | .parse_query(r#"json.k8s\.container\.name:prometheus"#) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `r"json.k8s\.container\.name:prometheus"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/indexer/mod.rs#L92

warning: unnecessary hashes around raw string literal --> src/indexer/mod.rs:92:30 | 92 | .parse_query(r#"json.k8s\.container\.name:prometheus"#) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `r"json.k8s\.container\.name:prometheus"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/core/json_utils.rs#L636

warning: unnecessary hashes around raw string literal --> src/core/json_utils.rs:636:41 | 636 | let json_path = split_json_path(r#"toto\titi"#); | ^^^^^^^^^^^^^^ help: try: `r"toto\titi"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/core/json_utils.rs#L631

warning: unnecessary hashes around raw string literal --> src/core/json_utils.rs:631:34 | 631 | assert_eq!(&json_path, &[r#"toto\titi"#]); | ^^^^^^^^^^^^^^ help: try: `r"toto\titi"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/core/json_utils.rs#L630

warning: unnecessary hashes around raw string literal --> src/core/json_utils.rs:630:41 | 630 | let json_path = split_json_path(r#"toto\\titi"#); | ^^^^^^^^^^^^^^^ help: try: `r"toto\\titi"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/core/json_utils.rs#L624

warning: unnecessary hashes around raw string literal --> src/core/json_utils.rs:624:43 | 624 | let json_path_2 = split_json_path(r#"k8s\.container\.name"#); | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `r"k8s\.container\.name"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes

unnecessary hashes around raw string literal: src/core/json_utils.rs#L622

warning: unnecessary hashes around raw string literal --> src/core/json_utils.rs:622:41 | 622 | let json_path = split_json_path(r#"toto\.titi"#); | ^^^^^^^^^^^^^^^ help: try: `r"toto\.titi"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes = note: `#[warn(clippy::needless_raw_string_hashes)]` on by default

private item shadows public glob re-export: src/aggregation/bucket/mod.rs#L33