Adding series with a ID of 0 is now blocked.
This commit is contained in:
parent
70dcacb41b
commit
e8e0e8e5dc
|
@ -46,6 +46,15 @@ namespace NzbDrone.Core.Test.ProviderTests
|
||||||
series.First().SeasonFolder.Should().Be(useSeasonFolder);
|
series.First().SeasonFolder.Should().Be(useSeasonFolder);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
[TestCase(0)]
|
||||||
|
[TestCase(-1)]
|
||||||
|
public void add_series_should_fail_if_series_is_less_than_zero(int seriesId)
|
||||||
|
{
|
||||||
|
WithRealDb();
|
||||||
|
Assert.Throws<ArgumentOutOfRangeException>(() => Mocker.Resolve<SeriesProvider>().AddSeries("C:\\Test", seriesId, 1));
|
||||||
|
}
|
||||||
|
|
||||||
[Test]
|
[Test]
|
||||||
public void find_series_empty_repo()
|
public void find_series_empty_repo()
|
||||||
{
|
{
|
||||||
|
|
|
@ -50,7 +50,7 @@ namespace NzbDrone.Core.Jobs
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
seriesToUpdate = new List<Series>() { _seriesProvider.GetSeries(targetId) };
|
seriesToUpdate = new List<Series> { _seriesProvider.GetSeries(targetId) };
|
||||||
}
|
}
|
||||||
|
|
||||||
//Update any Daily Series in the DB with the IsDaily flag
|
//Update any Daily Series in the DB with the IsDaily flag
|
||||||
|
|
|
@ -104,6 +104,11 @@ namespace NzbDrone.Core.Providers
|
||||||
{
|
{
|
||||||
Logger.Info("Adding Series [{0}] Path: [{1}]", tvDbSeriesId, path);
|
Logger.Info("Adding Series [{0}] Path: [{1}]", tvDbSeriesId, path);
|
||||||
|
|
||||||
|
if (tvDbSeriesId <=0)
|
||||||
|
{
|
||||||
|
throw new ArgumentOutOfRangeException("tvDbSeriesId", tvDbSeriesId.ToString());
|
||||||
|
}
|
||||||
|
|
||||||
var repoSeries = new Series();
|
var repoSeries = new Series();
|
||||||
repoSeries.SeriesId = tvDbSeriesId;
|
repoSeries.SeriesId = tvDbSeriesId;
|
||||||
repoSeries.Path = path;
|
repoSeries.Path = path;
|
||||||
|
|
Loading…
Reference in New Issue