History Items are now added with series. Test makes sure seriesId is sent in.

pull/2/head
Mark McDowall 14 years ago
parent ecd0fb50a8
commit 8d4f65b7dc

@ -24,6 +24,7 @@ namespace NzbDrone.Core.Test
.With(e => e.Episodes = Builder<Episode>.CreateListOfSize(2) .With(e => e.Episodes = Builder<Episode>.CreateListOfSize(2)
.WhereTheFirst(1).Has(s => s.EpisodeId = 12) .WhereTheFirst(1).Has(s => s.EpisodeId = 12)
.AndTheNext(1).Has(s => s.EpisodeId = 99) .AndTheNext(1).Has(s => s.EpisodeId = 99)
.WhereAll().Has(s => s.SeriesId = 5)
.Build()) .Build())
.With(c => c.Quality = new Quality(QualityTypes.DVD, false)) .With(c => c.Quality = new Quality(QualityTypes.DVD, false))
.Build(); .Build();
@ -42,13 +43,12 @@ namespace NzbDrone.Core.Test
.Returns(sabTitle); .Returns(sabTitle);
mocker.GetMock<HistoryProvider>() mocker.GetMock<HistoryProvider>()
.Setup(s => s.Add(It.Is<History>(h => h.EpisodeId == 12))); .Setup(s => s.Add(It.Is<History>(h => h.EpisodeId == 12 && h.SeriesId == 5)));
mocker.GetMock<HistoryProvider>() mocker.GetMock<HistoryProvider>()
.Setup(s => s.Add(It.Is<History>(h => h.EpisodeId == 99))); .Setup(s => s.Add(It.Is<History>(h => h.EpisodeId == 99 && h.SeriesId == 5)));
mocker.Resolve<DownloadProvider>().DownloadReport(parseResult); mocker.Resolve<DownloadProvider>().DownloadReport(parseResult);
mocker.VerifyAllMocks(); mocker.VerifyAllMocks();
} }
} }

@ -48,6 +48,7 @@ namespace NzbDrone.Core.Providers
history.Quality = parseResult.Quality.QualityType; history.Quality = parseResult.Quality.QualityType;
history.NzbTitle = parseResult.NzbTitle; history.NzbTitle = parseResult.NzbTitle;
history.EpisodeId = episode.EpisodeId; history.EpisodeId = episode.EpisodeId;
history.SeriesId = episode.SeriesId;
_historyProvider.Add(history); _historyProvider.Add(history);
} }

Loading…
Cancel
Save