Agreed. Never did understand why people do persistent mapping when they map during logon with a script.

Either map it once and be done with it and don't do it in your script, or don't persistent map it.

Yes, you should ALWAYS unmap or at least test if it's mapped or not. Maybe a user mapped it in which case your mapping would probably fail.