CI: Run `async-connection-wrapper` feature tests too

weiznich / diesel_async

Diesel async connection implementation

Apache License 2.0

656 stars 81 forks source link

CI: Run `async-connection-wrapper` feature tests too #193

Closed Turbo87 closed 3 weeks ago

Turbo87 commented 4 weeks ago

tests/sync_wrapper.rs is gated by #[cfg(feature = "async-connection-wrapper")]. Since the async-connection-wrapper feature wasn't explicitly enabled, we were not actually running these tests on CI.

Turbo87 commented 3 weeks ago

looks like the sqlite variant of one of the tests was unhappy. I've added another commit that should fix it.

Turbo87 commented 3 weeks ago

looks like this is finally green now too 😅