Skip to content

Commit

Permalink
fix(test): Fixed Quaint tests (#5231)
Browse files Browse the repository at this point in the history
- Fixed broken Quaint doc-tests
- Allowed running Quaint doc-tests on CI
  • Loading branch information
viktor-ferenczi authored Mar 6, 2025
1 parent 6128620 commit ab62ab1
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test-quaint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
fail-fast: false
matrix:
features:
- "--lib --features=all-native"
- "--features=all-native"
env:
TEST_MYSQL: "mysql://root:prisma@localhost:3306/prisma"
TEST_MYSQL8: "mysql://root:prisma@localhost:3307/prisma"
Expand Down
2 changes: 1 addition & 1 deletion quaint/src/ast/delete.rs
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ impl<'a> Delete<'a> {
/// let query = Delete::from_table("users").returning(vec!["id"]);
/// let (sql, _) = Sqlite::build(query)?;
///
/// assert_eq!("DELETE FROM `users` RETURNING \"id\"", sql);
/// assert_eq!("DELETE FROM `users` RETURNING `id` AS `id`", sql);
/// # Ok(())
/// # }
pub fn returning<K, I>(mut self, columns: I) -> Self
Expand Down
2 changes: 1 addition & 1 deletion quaint/src/ast/insert.rs
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ impl<'a> Insert<'a> {
/// let insert = Insert::from(query).comment("trace_id='5bd66ef5095369c7b0d1f8f4bd33716a', parent_id='c532cb4098ac3dd2'");
/// let (sql, _) = Sqlite::build(insert)?;
///
/// assert_eq!("INSERT INTO `users` DEFAULT VALUES; /* trace_id='5bd66ef5095369c7b0d1f8f4bd33716a', parent_id='c532cb4098ac3dd2' */", sql);
/// assert_eq!("INSERT INTO `users` DEFAULT VALUES /* trace_id='5bd66ef5095369c7b0d1f8f4bd33716a', parent_id='c532cb4098ac3dd2' */", sql);
/// # Ok(())
/// # }
/// ```
Expand Down
12 changes: 8 additions & 4 deletions quaint/src/connector/result_set/result_row.rs
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,12 @@ impl IntoIterator for ResultRow {
///
/// ```
/// # use quaint::connector::*;
/// let names = vec!["id".to_string(), "name".to_string()];
/// let names = vec!["id".to_string(), "name".to_string()]; /// # let column_types = vec![ColumnType::Text, ColumnType::Text];
/// let column_types = vec![ColumnType::Text, ColumnType::Text];
/// let rows = vec![vec!["1234".into(), "Musti".into()]];
///
/// let result_set = ResultSet::new(names, rows);
/// let result_set = ResultSet::new(names, column_types, rows);
///
/// let row = result_set.first().unwrap();
///
/// assert_eq!(row[0], row["id"]);
Expand Down Expand Up @@ -100,8 +102,9 @@ impl<'a> ResultRowRef<'a> {
/// ```
/// # use quaint::connector::*;
/// # let names = vec!["id".to_string(), "name".to_string()];
/// # let column_types = vec![ColumnType::Text, ColumnType::Text];
/// # let rows = vec![vec!["1234".into(), "Musti".into()]];
/// # let result_set = ResultSet::new(names, rows);
/// # let result_set = ResultSet::new(names, column_types, rows);
/// # let row = result_set.first().unwrap();
/// assert_eq!(Some(&row[0]), row.at(0));
/// ```
Expand All @@ -118,8 +121,9 @@ impl<'a> ResultRowRef<'a> {
/// ```
/// # use quaint::connector::*;
/// # let names = vec!["id".to_string(), "name".to_string()];
/// # let column_types = vec![ColumnType::Text, ColumnType::Text];
/// # let rows = vec![vec!["1234".into(), "Musti".into()]];
/// # let result_set = ResultSet::new(names, rows);
/// # let result_set = ResultSet::new(names, column_types, rows);
/// # let row = result_set.first().unwrap();
/// assert_eq!(Some(&row["id"]), row.get("id"));
/// ```
Expand Down

0 comments on commit ab62ab1

Please sign in to comment.