Missing check that the symbol import is public.

[esdrubal]

We should fix the todo message in:

sway/sway-core/src/semantic_analysis/namespace/root.rs

Lines 61 to 62 in 44f46a0

	// TODO: check that the symbol import is public?
	self.resolve_symbol(handler, src_path, true_symbol)

          Is this still TODO?

Originally posted by @IGI-111 in #5048 (comment)

[jjcnn]

root.rs has been changed quite a lot since the commit that contains the quoted code snippet, so the fix requires a little bit more than just a simple visibility check.

As I mentioned on slack I've got some tests for this exact issue that I've had to disable because our path resolution is broken. I suggest we fix this one as part of the fix for path resolution, which is the next thing I'll be looking at.