David Rousselie
David Rousselie
@lidatong I have rebased the branch to fix the merge conflict. It should be mergeable now.
Sorry, my description was not clear enough. For tests (in my app, not in the lib), I need to make the queue name dynamic, each test must have a unique...
@geofmureithi Of course, I could limit the test thread count to 1 but this defeats the purpose of having parallel test execution and thus slowing their execution down 🤷
Here are the [tests](https://github.com/universal-inbox/universal-inbox/blob/main/api/tests/api/test_slack_webhook.rs). They call an API [endpoint](https://github.com/universal-inbox/universal-inbox/blob/main/api/src/routes/webhook.rs) that creates jobs asynchronously handled [here](https://github.com/universal-inbox/universal-inbox/blob/main/api/src/jobs/slack.rs). A unique API, worker and DB (ie. database name) are created for each test [here](https://github.com/universal-inbox/universal-inbox/blob/main/api/tests/api/helpers/mod.rs#L197-L213)....
> Could you consider annotating the parts you have added with #[cfg(test)] and #[doc(hidden)]? I'm not 💯 sure but if I add `#[cfg(test)]` to my changes here, would I still...
I have made a test with `#[cfg(test)]` and indeed, it does not work for dependencies. I tried another thing by declaring a new feature `test` in `apalis-redis` and it does...
@savil what I have in mind would be an option (as an env variable) to permanently disable automatic installation of packages by Devbox.
@savil I have created a [PR](https://github.com/jetpack-io/devbox/pull/1963) that would answer my need. It seems reasonable as it adds an option that is already available for the global use case.