Remove unnecessary uses of #[cfg(not(test))]

[avtrujillo]

Motivation

The main and main-rt macro functions both have the #[cfg(not(test))] with a comment indicating that its inclusion is a workaround for rust-lang/rust#62127, which has since been resolved. Because rust-analyzer enables test by default, this means that these functions are greyed out when using rust-analyzer in VSCode without configuration changes.

Solution

Remove all uses of #[cfg(not(test))] in the tokio-macros crate that are accompanied by comments mentioning rust-lang/rust#62127 as the motivation for usage.

It's possible that the usage in

tokio/tokio/src/lib.rs

Line 677 in 8093712

// Always fail if rt is not enabled.

is also unnecessary, since it appears in the context of using main as an identifier. I'm less sure about that one since it doesn't have a comment mentioning rust-lang/rust#62127, so I've left it alone for now.

[GlenDC]

All good as far as I can see.
Pretty certain you can also remove the same thing in

tokio/tokio/src/lib.rs

Line 677 in 8093712

// Always fail if rt is not enabled.

, as it is the same situation as the cfg_rt! case AFAIK.

A tokio maintainer can probably confirm, this and all, but to me it does look good and working as expected.

[Darksonn]

Thank you.