Can not use .eq(Decimal_field + xxx) about diesel

[kakalos12]

ok @weiznich , Let me try it.

[weiznich]

Thanks for opening this bug report.

Seems like it's just missing the BigDecimal type in this list of allowed types:

diesel/diesel_derives/src/table.rs

Line 577 in 12429cb

const NUMERIC_TYPES: &[&str] = &[

Mind submitting a PR that adds it there + adds a test somewhere in diesel_test

[kakalos12]

Hi @weiznich , Sorry, I'm new to rust, Can you guide me how to setup the working env for the diesel. When I clone the repository to my local machine and try to run cargo t, it has some errors

error[E0432]: unresolved import `diesel`
  --> diesel/src/expression/functions/date_and_time.rs:30:1
   |
30 | / define_sql_function! {
31 | |     /// Represents the SQL `DATE` function. The argument should be a Timestamp
32 | |     /// expression, and the return value will be an expression of type Date.
33 | |     ///
...  |
50 | |     fn date(expr: Timestamp) -> Date;
51 | | }
   | |_^ could not find `sqlite` in the crate root
   |
   = note: this error originates in the macro `define_sql_function` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0432]: unresolved import `diesel`
  --> diesel/src/sql_types/mod.rs:40:48
   |
40 | #[derive(Debug, Clone, Copy, Default, QueryId, SqlType)]
   |                                                ^^^^^^^ could not find `pg` in the crate root
   |
   = note: this error originates in the derive macro `SqlType` (in Nightly builds, run with -Z macro-backtrace for more info)

Thanks so much.

[weiznich]

It depends a bit on where exactly you've called cargo test, but in general you always need to set some feature flags to enable one or the other backend. The contribution documentation has some more details and there is the bin/test script which executes most of the tests in the repo.

For the diesel crate itself you usually want something like DATABASE_URL=… cargo test --features "{backend_to_test} extras" where {backend_to_test} is one of postgres, mysql or sqlite.

[weiznich]

Fixed by #4102