chore: fix wasm_bindgen_test macros

[hlbarber]

Motivation

Tests involving wasm_bindgen_test currently fail:

https://github.com/tokio-rs/tracing/actions/runs/5756318807/job/15605512576

Solution

• Use extern crate wasm_bindgen_test to side step no_implicit_prelude.
• use fully qualified names in wasm_bindgen_test for items exported from std prelude rustwasm/wasm-bindgen#3549
• Consume the release wasm_bindgen_test containing said change.

[hawkw]

hmm, looks like CI is still failing?

[hlbarber]

hmm, looks like CI is still failing?

Yeah, we'll have to wait for rustwasm/wasm-bindgen#3549 to be released.

The failures are of the form

error[E0425]: cannot find value `None` in this scope
   --> tracing/tests/macros.rs:946:36
    |
946 | #[cfg_attr(target_arch = "wasm32", wasm_bindgen_test::wasm_bindgen_test)]
    |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ not found in this scope

[hlbarber]

If you want a quick fix I can just pull Option::Some and None into scope (for only target_arch = wasm32 perhaps).

[davidbarsky]

If you want a quick fix I can just pull Option::Some and None into scope (for only target_arch = wasm32 perhaps).

Not necessary yet. I'll ask around first.

[hawkw]

I guess in the meantime, I'm willing to merge PRs while waiting for upstream changes...

[davidbarsky]

given that I got no response, I suppose we should just merge this/potentially with Option being imported.

[hawkw]

given that I got no response, I suppose we should just merge this/potentially with Option being imported.

cool, okay --- @hlbarber, can i get you to add the import, then?

[hlbarber]

@hawkw done

[hawkw]

looks great, thanks for working on this @hlbarber!