Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: enable correct_pagination_assets_after_creating_new_one test #5192

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
3 changes: 1 addition & 2 deletions crates/iroha/tests/pagination.rs
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ fn fetch_size_should_work() -> Result<()> {
// use the lower-level API to inspect the batch size
use iroha::data_model::query::{
builder::QueryExecutor as _,
parameters::{FetchSize, QueryParams, Sorting},
parameters::{FetchSize, QueryParams},
QueryWithFilter, QueryWithParams,
};

Expand All @@ -69,7 +69,6 @@ fn fetch_size_should_work() -> Result<()> {
.into(),
QueryParams::new(
Pagination::new(Some(nonzero!(7_u64)), 1),
Sorting::default(),
FetchSize::new(Some(nonzero!(3_u64))),
),
);
Expand Down
347 changes: 0 additions & 347 deletions crates/iroha/tests/sorting.rs

This file was deleted.

4 changes: 1 addition & 3 deletions crates/iroha_core/src/query/store.rs
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ mod tests {
use iroha_data_model::{
permission::Permission,
prelude::SelectorTuple,
query::parameters::{FetchSize, Pagination, QueryParams, Sorting},
query::parameters::{FetchSize, Pagination, QueryParams},
};
use iroha_primitives::json::Json;
use iroha_test_samples::ALICE_ID;
Expand All @@ -297,11 +297,9 @@ mod tests {
let fetch_size = FetchSize {
fetch_size: Some(nonzero!(1_u64)),
};
let sorting = Sorting::default();

let query_params = QueryParams {
pagination,
sorting,
fetch_size,
};

Expand Down
Loading
Loading