Чому це твердження видає виняток формату при порівнянні структур?

Я намагаюся стверджувати рівність двох System.Drawing.Sizeструктур, і я отримую виняток формату замість очікуваної помилки затвердження.

[TestMethod]
public void AssertStructs()
{
    var struct1 = new Size(0, 0);
    var struct2 = new Size(1, 1);

    //This throws a format exception, "System.FormatException: Input string was not in a correct format."
    Assert.AreEqual(struct1, struct2, "Failed. Expected {0}, actually it is {1}", struct1, struct2); 

    //This assert fails properly, "Failed. Expected {Width=0, Height=0}, actually it is {Width=1, Height=1}".
    Assert.AreEqual(struct1, struct2, "Failed. Expected " + struct1 + ", actually it is " + struct2); 
}

Це передбачувана поведінка? Я тут роблю щось не так?

Я отримав його. І так, це помилка.

Проблема в тому, що тут відбувається два рівні string.Format.

Перший рівень форматування що - щось на кшталт:

string template  = string.Format("Expected: {0}; Actual: {1}; Message: {2}",
                                 expected, actual, message);

Потім ми використовуємо string.Formatз параметрами, які ви вказали:

string finalMessage = string.Format(template, parameters);

(Очевидно, що там є культури, і якась санітарія ... але недостатньо.)

Це виглядає нормально - якщо очікувані та фактичні значення самі не закінчаться фігурними дужками після перетворення в рядок - що вони і роблять Size. Наприклад, ваш перший розмір перетворюється на:

{Width=0, Height=0}

Отже, другий рівень форматування приблизно такий:

string.Format("Expected: {Width=0, Height=0}; Actual: {Width=1, Height=1 }; " +
              "Message = Failed expected {0} actually is {1}", struct1, struct2);

... і ось що не вдається. Ой.

Дійсно, ми можемо це довести дуже легко, обдуривши форматування, використовуючи наші параметри для очікуваної та фактичної частин:

var x = "{0}";
var y = "{1}";
Assert.AreEqual<object>(x, y, "What a surprise!", "foo", "bar");

Результат:

Assert.AreEqual failed. Expected:<foo>. Actual:<bar>. What a surprise!

Явно зламаний, оскільки ми не очікували, як fooі не було фактичної вартості bar!

В основному це схоже на атаку ін'єкції SQL, але в досить менш страшному контексті string.Format.

Як обхідний шлях ви можете використовувати те, string.Formatяк пропонує StriplingWarrior. Це дозволяє уникнути другого рівня форматування, що виконується в результаті форматування з фактичними / очікуваними значеннями.

Я думаю, ви знайшли помилку.

Це працює (видає виняток із твердження):

var a = 1;
var b = 2;
Assert.AreEqual(a, b, "Not equal {0} {1}", a, b);

І це працює (виводить повідомлення):

var a = new{c=1};
var b = new{c=2};
Console.WriteLine(string.Format("Not equal {0} {1}", a, b));

Але це не працює (кидає a FormatException):

var a = new{c=1};
var b = new{c=2};
Assert.AreEqual(a, b, "Not equal {0} {1}", a, b);

Я не можу придумати жодної причини, що це може бути очікуваною поведінкою. Я б подав звіт про помилку. Тим часом, ось обхідний шлях:

var a = new{c=1};
var b = new{c=2};
Assert.AreEqual(a, b, string.Format("Not equal {0} {1}", a, b));

Я погоджуюсь з @StriplingWarrior, що це справді видається помилкою методу Assert.AreEqual () принаймні 2 перевантажень. Як уже зазначав StiplingWarrior, наступне не вдається;

var a = new { c = 1 };
var b = new { c = 2 };
Assert.AreEqual(a, b, "Not equal {0} {1}", a, b);

Я трохи експериментував над цим, щоб трохи чіткіше використовувати код. Наведене також не працює;

// specify variable data type rather than "var"...no effect, still fails
Size a = new Size(0, 0);
Size b = new Size(1, 1);
Assert.AreEqual(a, b, "Not equal {0} {1}", a, b);

І

// specify variable data type and name the type on the generic overload of AreEqual()...no effect, still fails
Size a = new Size(0, 0);
Size b = new Size(1, 1);
Assert.AreEqual<Size>(a, b, "Not equal {0} {1}", a, b);

Це змусило мене задуматися. System.Drawing.Size - це структура. А як щодо об’єктів? У списку парів чи вказати , що список після stringповідомлення є params object[]. Технічно так конструкції - це об’єкти ... але особливі види об’єктів, тобто типи цінностей. Я думаю, що тут криється помилка. Якщо ми використовуємо наш власний об'єкт з подібним використанням та структурами на Sizeнаступному фактично робить роботу;

private class MyClass
{
    public MyClass(int width, int height)
        : base()
    { Width = width; Height = height; }

    public int Width { get; set; }
    public int Height { get; set; }
}

[TestMethod]
public void TestMethod1()
{
    var test1 = new MyClass(0, 0);
    var test2 = new MyClass(1, 1);
    Assert.AreEqual(test1, test2, "Show me A [{0}] and B [{1}]", test1, test2);
}

Я думаю, що перше твердження є неправильним.

Використовуйте замість цього:

Assert.AreEqual(struct1, 
                struct2, 
                string.Format("Failed expected {0} actually is {1}", struct1, struct2));