Fix: Handle empty _nameToDisplay_ on first run to enable P2P testing

[artburkart]

On first launch, nameToDisplay resolves to "" instead of the expected “???”. This breaks the app’s flow, making P2P functionality untestable.

This patch adds a check for .isEmpty and ensures editNamePopoverShown opens when the value is empty. With this change, the app becomes usable out of the box for testing.

My guess is the library doesn't "load a default value" but rather just "".
Not claiming this is the definitive fix, but it gets things working for now.

Pictured below is what the app looks like on first boot.

And here are the logs when you attempt to turn on the P2P functionality:

P2PNET: Updated bonjour network listener to advertise name
No peer name is set on the provider

[heckj]

Fix looks fine to me, I certainly could have left it in an awkward state while iterating and never wrapping that up properly.
The failing "test" is a CI barf - in #50 I'm disabling the CI from attempting to run simulator tests, as there's just nothing meaningful actually being tested, and it's just throwing up spurious issues here.