Skip to content

Commit f70fe2a

Browse files
committed
refactor(cubesql): Use Default struct update for V1LoadRequestQuery in test_user_change
1 parent 11e09c1 commit f70fe2a

File tree

1 file changed

+7
-33
lines changed

1 file changed

+7
-33
lines changed

rust/cubesql/cubesql/src/compile/test/test_user_change.rs

Lines changed: 7 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,8 @@ async fn test_change_user_via_filter() {
3232
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string(),]),
3333
segments: Some(vec![]),
3434
dimensions: Some(vec![]),
35-
time_dimensions: None,
3635
order: Some(vec![]),
37-
limit: None,
38-
offset: None,
39-
filters: None,
40-
ungrouped: None,
36+
..Default::default()
4137
}
4238
)
4339
}
@@ -63,12 +59,8 @@ async fn test_change_user_via_in_filter() {
6359
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string(),]),
6460
segments: Some(vec![]),
6561
dimensions: Some(vec![]),
66-
time_dimensions: None,
6762
order: Some(vec![]),
68-
limit: None,
69-
offset: None,
70-
filters: None,
71-
ungrouped: None,
63+
..Default::default()
7264
}
7365
)
7466
}
@@ -87,12 +79,8 @@ async fn test_change_user_via_in_filter_thoughtspot() {
8779
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string()]),
8880
segments: Some(vec![]),
8981
dimensions: Some(vec![]),
90-
time_dimensions: None,
9182
order: Some(vec![]),
92-
limit: None,
93-
offset: None,
94-
filters: None,
95-
ungrouped: None,
83+
..Default::default()
9684
};
9785

9886
let cube_scan = query_plan.as_logical_plan().find_cube_scan();
@@ -130,12 +118,8 @@ async fn test_change_user_via_filter_powerbi() {
130118
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string(),]),
131119
segments: Some(vec![]),
132120
dimensions: Some(vec![]),
133-
time_dimensions: None,
134121
order: Some(vec![]),
135-
limit: None,
136-
offset: None,
137-
filters: None,
138-
ungrouped: None,
122+
..Default::default()
139123
}
140124
)
141125
}
@@ -158,18 +142,15 @@ async fn test_change_user_via_filter_and() {
158142
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string(),]),
159143
segments: Some(vec![]),
160144
dimensions: Some(vec![]),
161-
time_dimensions: None,
162145
order: Some(vec![]),
163-
limit: None,
164-
offset: None,
165146
filters: Some(vec![V1LoadRequestQueryFilterItem {
166147
member: Some("KibanaSampleDataEcommerce.customer_gender".to_string()),
167148
operator: Some("equals".to_string()),
168149
values: Some(vec!["male".to_string()]),
169150
or: None,
170151
and: None,
171152
}]),
172-
ungrouped: None,
153+
..Default::default()
173154
}
174155
)
175156
}
@@ -210,12 +191,9 @@ async fn test_user_with_join() {
210191
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string()]),
211192
dimensions: Some(vec![]),
212193
segments: Some(vec![]),
213-
time_dimensions: None,
214194
order: Some(vec![]),
215-
limit: None,
216-
offset: None,
217-
filters: None,
218195
ungrouped: Some(true),
196+
..Default::default()
219197
}
220198
);
221199

@@ -253,12 +231,8 @@ async fn test_change_user_via_filter_with_alias() {
253231
measures: Some(vec!["KibanaSampleDataEcommerce.count".to_string(),]),
254232
segments: Some(vec![]),
255233
dimensions: Some(vec![]),
256-
time_dimensions: None,
257234
order: Some(vec![]),
258-
limit: None,
259-
offset: None,
260-
filters: None,
261-
ungrouped: None,
235+
..Default::default()
262236
}
263237
)
264238
}

0 commit comments

Comments
 (0)