Using the Assert.AreEqual with the ignoreCase parameter is better because it doesn't require the creation of a new string (and, as pointed out by @dtb, you could work following the rules of a specific culture info)
Assert.AreEqual(user1.UserName, user2.UserName, true, CultureInfo.CurrentCulture);