Skip to content

Commit

Permalink
[#142] parse_having 테스트코드 작성
Browse files Browse the repository at this point in the history
  • Loading branch information
myyrakle committed Jul 29, 2024
1 parent 2dbe881 commit c229278
Show file tree
Hide file tree
Showing 2 changed files with 65 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/ast/dml/parts/having.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ use crate::ast::types::SQLExpression;

use serde::{Deserialize, Serialize};

#[derive(Deserialize, Serialize, Clone, Debug, PartialEq)]
#[derive(Deserialize, Serialize, Clone, Debug, PartialEq, Default)]
pub struct HavingClause {
pub expression: Box<SQLExpression>,
}
64 changes: 64 additions & 0 deletions src/parser/test/select.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2249,3 +2249,67 @@ fn test_parse_where() {
}
}
}

#[test]
fn test_parse_having() {
struct TestCase {
name: String,
input: Vec<Token>,
expected: HavingClause,
want_error: bool,
}

let test_cases = vec![
TestCase {
name: "HAVING p.id = 1".into(),
input: vec![
Token::Having,
Token::Identifier("p".into()),
Token::Period,
Token::Identifier("id".into()),
Token::Operator(OperatorToken::Eq),
Token::Integer(1),
],
expected: HavingClause {
expression: BinaryOperatorExpression {
operator: BinaryOperator::Eq,
lhs: SelectColumn::new(Some("p".into()), "id".into()).into(),
rhs: SQLExpression::Integer(1),
}
.into(),
},
want_error: false,
},
TestCase {
name: "실패: SELECT".into(),
input: vec![Token::Select],
expected: Default::default(),
want_error: true,
},
TestCase {
name: "실패: 빈 토큰".into(),
input: vec![],
expected: Default::default(),
want_error: true,
},
];

for t in test_cases {
let mut parser = Parser::new(t.input);

let got = parser.parse_having(Default::default());

assert_eq!(
got.is_err(),
t.want_error,
"{}: want_error: {}, error: {:?}",
t.name,
t.want_error,
got.err()
);

if let Ok(statements) = got {
assert_eq!(statements, t.expected, "TC: {}", t.name);
}
}
}

0 comments on commit c229278

Please sign in to comment.