Returning model with `enum` field with `find_by_statement`

[qRoC]

database: postgres

Enum:

#[derive(Debug, Clone, PartialEq, Eq, EnumIter, DeriveActiveEnum)]
#[sea_orm(rs_type = "String", db_type = "Enum", enum_name = "test_status")]
pub enum TestStatus {
    #[sea_orm(string_value = "new")]
    New,
    #[sea_orm(string_value = "processing")]
    Processing,
}

Model:

#[derive(Clone, Debug, PartialEq, DeriveEntityModel, Eq)]
#[sea_orm(table_name = "test")]
pub struct Model {
    #[sea_orm(primary_key)]
    pub id: i64,
    pub status: Status,
    pub created_at: DateTime,
}

Query:

    let select_query = Query::select()
        .column(test::Column::Id)
        .from(test::Entity)
        .and_where(test::Column::Status.eq(TestStatus::New))
        .limit(1)
        .lock_with_behavior(LockType::Update, LockBehavior::SkipLocked)
        .to_owned();

    let update_query = Query::update()
        .table(test::Entity)
        .value(test::Column::Status, TestStatus::Processing.as_enum())
        .and_where(test::Column::Id.in_subquery(select_query))
        .returning_all()
        .to_owned();

    let stmt = db.get_database_backend().build(&update_query);

    test::Model::find_by_statement(stmt).one(db).await

Query is executed, but final result is:

Query Error: error occurred while decoding column "status": mismatched types; Rust type `core::option::Option<alloc::string::String>` (as SQL type `TEXT`) is not compatible with SQL type `test_status`

[Huliiiiii]

Could you check if #2551 solves your issue? (Released in 1.1.9)

[sinder38]

Stumbled into the same issue.
No, #2551 doesn't solve the issue as it is NOT implemented for SelectStatement.

Although, my setup is quite different from OP and involves selecting from a function output.

	let mut query = Query::select();
	query.column(Asterisk).from_function(
		Func::cust(TreeFn::Function).arg(array_value),
		Alias::new("tree"),
	);
	let mut rows = Vec::with_capacity(query_results.len());
	for qr in query_results {
		rows.push(
			TreeFn::from_query_result(&qr, "")?; // <-- Error occurs here 
		);
	}

More info:
I am migrating my codebase from sqlx + sea_query to sea_orm and originally this was working as intended. Sqlx can handle type conversion.ee

I am already working on repro and fix. @Huliiiiii please assign

[Huliiiiii]

I added a variant for enums to Value in sea query, which avoids the need to manually cast enums to text. Will this solve your issue?

#2955

[sinder38]

Thanks, I will see. But if I remember correctly cast isn't needed in select statements

[sinder38]

I tested it and it doesn't work.

@Huliiiiii
Looking at your work int sea_query it should handle this case, once finished.
Should I create a separate PR to try too fix in a current codebase or contribute to your Value::Enum branches?
if second option please allow me to push to them

[Huliiiiii]

For compatibility, the entity must be updated with rs_type = "Enum" so that ActiveEnum will be converted to Value::Enum. Did you test it under this condition?

[sinder38]

Yes, ofcource. I tried several other ways also

[Huliiiiii]

That is odd. You can open a PR to my branch.

[sinder38]

okay, I will add a test that will trigger the same error

[sinder38]

So?
Huliiiiii#1