Как пройти такой unit test?

Question

[fryette]

[TestMethod]
 public void FunctionExtention_CombinePredicates_Should_Avoid_Closure()
        {
            var value = 0;
            var predicates = new Predicate<int>[]
            {
                x => x > value
            };
            var result = FunctionExtentions.CombinePredicates(predicates);
            value = 1000; // This should not affect the call above!
            Assert.IsTrue(result(2));
            Assert.IsTrue(result(5));
            Assert.IsTrue(result(1000));
            Assert.IsFalse(result(-20));
            Assert.IsFalse(result(0));
            Assert.IsFalse(result(-1000));
        }
        public static Predicate<T> CombinePredicates<T>(Predicate<T>[] predicates)
        {            
            return item => predicates.All(predicate => predicate(item));
        }

Answer 1

[Алексей Немиро]

Без читов точно не пройти! :-)

Странный вопрос. Запустить проект, в котором размещен этот тест и посмотреть на отчет.
Если будут ошибки, внеси соответствующие исправления в код. Обычно это делается так.

----------
Подведу итоги по результатам дискуссии в комментариях.

1. Простые/тупые варианты пройти тест успешно, по крайней мере технически, это внести изменения в данные:

return x > value - 1000; // убираем лишнюю тысячу :)

return x + 1000 > value; // добавляем

2. Самый очевидный и, более ли менее правильный вариант, по крайней мере на практике (хотя лучше вообще избегать таких ситуаций), использовать дополнительную переменную для сохранения значения:

var value = 0;
var value2 = value;
var predicates = new Predicate[]
{
  x => x > value2
};

3. Еще могу предложить построить дерево. Но это сложное решение, кода больше. Хотя, если вы проходили/изучаете именно эту тему, то вполне возможно, что этот вариант может быть правильным.

// using System.Linq.Expressions;

[TestMethod]
public void FunctionExtention_CombinePredicates_Should_Avoid_Closure()
{
    var value = 0;

    var iks = Expression.Parameter(typeof(int), "x");
    var v = Expression.Constant(value, typeof(int)); // фиксируем значение value 
    var less = Expression.LessThan(v, iks);
    var l = Expression.Lambda<Func<int, bool>>(less, new ParameterExpression[] { iks }).Compile();

    var predicates = new Predicate<int>[]
    {
      x => l(x)
    };
    var result = FunctionExtentions.CombinePredicates(predicates);
    value = 1000; // This should not affect the call above!
    Assert.IsTrue(result(2));
    Assert.IsTrue(result(5));
    Assert.IsTrue(result(1000));
    Assert.IsFalse(result(-20));
    Assert.IsFalse(result(0));
    Assert.IsFalse(result(-1000));
}

public static Predicate<T> CombinePredicates<T>(Predicate<T>[] predicates)
{            
    return item => predicates.All(predicate => predicate(item));
}

Comments

[fryette]

разумеется запустил и проверил,и разумеется он его не прошел.
Как я понял тут хранится ссылка на обьект,быть может надо сделать глубокое копирование,но подходящего кода не нашел еще.

[Алексей Немиро]

fryette: FunctionExtentions - это просто имя класса теста? Если убрать строку, у которой написано: "This should not affect the call above!", то тест будет успешно пройден.

Либо можно поменять передаваемые значения, чтобы они прошли условие x > value, т.е. значение x должно быть больше 1000:
Assert.IsTrue(result(2000));
Assert.IsTrue(result(5000));
Assert.IsTrue(result(1001));

Это что, очередное школьное задание по информатике? :-))

[fryette]

Алексей Немиро: у вас видимо стереотипы какието по поводу школьников.
Задача,что бы данный тест проходил,не изменяя его,необходимо переписать заданный метод,до этого еще 10 тестов на этот метод и все они успешно пройдены кроме этого.

[fryette]

Алексей Немиро: Если убрать строку, у которой написано: "This should not affect the call above!", то тест будет успешно пройден.
У вас плохо с английским вдобавок вижу)))
Если вообще удалить тест то не нужно будет вообще проходить ничего)

[Алексей Немиро]

fryette: задача слишком странная, поэтому и приходят мысли о школьниках :) Программисты обычно проводят отладку, ищут причину проблемы и решают. Ну и практической пользы от работы показанного фрагмента кода тоже никакой и это еще один признак школьного (или универного, разницы никакой) задания.

Аналогично с комментарием. Учитывая бесполезность кода, от удаления причины, которая мешает пройти тест, плохо не будет, разве что двойку поставят, но это не страшно. Вот если спутники начнут падать, тогда да :-)

Чтобы тест пройти успешно, изменения ведь в любом случае нужно внести. Я предложил несколько вариантов, куда именно вносить, решайте сами.

Условие x > value, тоже, как вариант можно поменять. Ну или сделать x + 1000 > value, тогда не придется менять значения. Сколько спичек можно передвинуть, согласно условию задачи? ;-)

Думаю, можно поискать еще более красивое математическое решение этой задачи.

[fryette]

Алексей Немиро: по сути необходимо создать на 100% клонированную переменную,но вот как это сделать,не представляю возможным,в ней все равно остается ссылка на этот value

[Алексей Немиро]

fryette: чтобы сохранить value, скорее всего придется делать копию:

var value = 0;
var value2 = value;
var predicates = new Predicate[]
{
x => x > value2
};

Или использовать более красивый вариант, предложенный Владимир

[Алексей Немиро]

fryette: обновил свой ответ, добавил еще вариант с деревьями.