fix(pyo3-macros): allow pyclass named Probe

[maurosilber]

The * import was shadowing the name of the pyclass.

Fixes #4792.

I tested on a codebase that had one struct/pyclass named Probe, and many other structs/pyclasses.
But I'm not sure if some of the imports that I removed are necessary in other situations that I have not tested.

[alex]

Can you add a testcase for this so we don't regress?

[maurosilber]

I created a copy of this file, but adding a field to the Probe struct:

pyo3/tests/ui/pyclass_probe.rs

Lines 4 to 19 in 3ffad5a

	#[pyclass]
	pub struct Probe {}
	#[pymethods]
	impl Probe {
	#[new]
	fn new() -> Self {
	Self {}
	}
	}
	#[pymodule]
	fn probe(_py: Python<'_>, m: &Bound<'_, PyModule>) -> PyResult<()> {
	m.add_class::<Probe>()?;
	Ok(())
	}

and then a new line with the corresponding filename here:

pyo3/tests/test_compile_error.rs

Line 95 in 3ffad5a

t.pass("tests/ui/pyclass_probe.rs");

But the test does not fail when I tried locally (before applying my changes, I mean). 🤔

[Icxolu]

We need to revert the commented cfg and add a newsfragment, otherwise I think this looks good.

[Icxolu]

Great, thank you!