Skip to content

Commit a90ae91

Browse files
committed
fix
1 parent 2417873 commit a90ae91

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_evaluator.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -239,9 +239,7 @@ impl CubeEvaluator for MockCubeEvaluator {
239239
let granularity = &path[3];
240240

241241
// Validate granularity is one of the supported ones
242-
let valid_granularities = vec![
243-
"second", "minute", "hour", "day", "week", "month", "quarter", "year",
244-
];
242+
let valid_granularities = ["second", "minute", "hour", "day", "week", "month", "quarter", "year"];
245243

246244
if !valid_granularities.contains(&granularity.as_str()) {
247245
return Err(CubeError::user(format!(

rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ impl MockMemberSql {
5151
let mut path = String::new();
5252
let mut found_closing = false;
5353

54-
while let Some(ch) = chars.next() {
54+
for ch in chars.by_ref() {
5555
if ch == '}' {
5656
found_closing = true;
5757
break;

0 commit comments

Comments
 (0)