Ошибка проверки носорога: предыдущий метод 'IEnumerator.MoveNext ();'требует возвращаемого значения или исключения, чтобы бросить - PullRequest
Новая
       21

Ошибка проверки носорога: предыдущий метод 'IEnumerator.MoveNext ();'требует возвращаемого значения или исключения, чтобы бросить

10 голосов
/ 08 июня 2011

У меня есть следующий тестовый код: 

parentViewModel = MockRepository.GenerateMock<IParentViewModel>();
parentViewModel.Expect(x => x.GetPropertyValue<IEnumerable<Milestone>>("JobMilestones")).Return(new Milestone[0]);

viewModel = new JobPenaltiesViewModel(j, new Penalty[0], _opContext, parentViewModel);

Assert.That(viewModel.Milestones.Count(), Is.EqualTo(0));
parentViewModel.VerifyAllExpectations();

List<string> propsChanged = new List<string>();
viewModel.PropertyChanged += (s, e) => propsChanged.Add(e.PropertyName);

parentViewModel.Raise(x => x.PropertyChanged += null, parentViewModel, new PropertyChangedEventArgs("JobMilestones"));

AssertPropertiesChangedAsExepected(propsChanged, 1, "Milestones");

Milestone m1 = GenerateMilestone(j);
List<Milestone> milestones1 = new List<Milestone> { m1 };
parentViewModel.Expect(x => x.GetPropertyValue<IEnumerable<Milestone>>("JobMilestones")).Return(milestones1).Repeat.Any();

IEnumerable<Milestone> milestones = viewModel.Milestones;
Assert.That(milestones.Count(), Is.EqualTo(1));
parentViewModel.VerifyAllExpectations();

Все тесты и утверждения успешно выполняются вплоть до: 

Assert.That(milestones.Count(), Is.EqualTo(1));

Вот где я получаю исключение: 

Previous method 'IEnumerator.MoveNext();' requires a return value or an exception to throw.

Я перепробовал все, что мог придумать, и мое тестирование, кажется, показывает, что Mock parentViewModel возвращает нулевое значение или пустое перечисление (т. Е. Когда я использую отладчик для проверки возвращенного значения, «Представление результатов» говорит перечисление не дало результатов).

Что мне здесь не хватает?

Ответы [ 2 ]

2 голосов
/ 09 июля 2011

milestones.Count() выполняется так (так как это объект IEnumerable):

  1. Установить счетчик на 0.
  2. Получить первый элемент.
  3. Добавьте 1 к счетчику.
  4. Перейти к следующему элементу.
  5. Шаг 3, пока следующий элемент не станет нулевым
  6. Вернуть счетчик

Поэтому я предлагаю вам немного переписать.

Вариант 1:

  1. Создайте не коллекцию IEnumerable, а какой-нибудь более сильный объект, например List или Array:

    var milestones = viewModel.Milestones.ToArray();<br> //var milestones = viewModel.Milestones.ToList();

    После этого вы можете использовать соответственно Count и Length свойство для проверки Assert:

    Assert.That(milestones.Count, Is.EqualTo(1));<br> //Assert.That(milestones.Length, Is.EqualTo(1));

  2. Создать локальную переменную для хранения параметра счетчика:

    var count = viewModel.Milestones.Count(); // .Count() method executes here. Assert.That(count, Is.EqualTo(1));

0 голосов
/ 24 сентября 2011

С тех пор я удалил нарушающий код; однако я так и не понял, почему он так себя ведет.

...