Revert #112758 and add test case

[bvanjoi]

Fixes #112831.

Cannot unwrap update_resolution for resolution.single_imports.remove(&Interned::new_unchecked(import)); because there is a relationship between the Import and &NameBinding in NameResolution. This issue caused by my unfamiliarity with the data structure and I apologize for it.

This PR had been reverted, and test case have been added.

r? @Nilstrieb
cc @petrochenkov

[Noratrieb]

no worries, mistakes happen, that's what nightly is for. at least we got a nice test for it :)
@bors r+ p=1
let's try getting this into today's nightly

[bors]

📌 Commit 09d4a82 has been approved by Nilstrieb

It is now in the queue for this repository.

[bors]

⌛ Testing commit 09d4a82 with merge a34cead...

[bors]

☀️ Test successful - checks-actions
Approved by: Nilstrieb
Pushing a34cead to master...

[rust-timer]

Finished benchmarking commit (a34cead): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	3.3%	[3.3%, 3.3%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-2.4%	[-2.4%, -2.4%]	1
All ❌✅ (primary)	3.3%	[3.3%, 3.3%]	1

Cycles

This benchmark run did not return any relevant results for this metric.

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 657.549s -> 657.004s (-0.08%)