claunia/SabreTools
1
0
Fork 0
mirror of https://github.com/claunia/SabreTools.git synced 2025-12-16 19:14:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't allow null arrays in constructor

Browse Source
This commit is contained in:
Matt Nadareski
2024-10-24 03:23:56 -04:00
parent 14fc7777cb
commit 6c8c635588
2 changed files with 3 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
SabreTools.Core/Filter/FilterRunner.cs
View File

@@ -13,21 +13,15 @@ namespace SabreTools.Core.Filter
/// <summary> /// <summary>
/// Set of filters to be run against an object /// Set of filters to be run against an object
/// </summary> /// </summary>
public FilterObject[] Filters { get; } public readonly FilterObject[] Filters;
public FilterRunner(FilterObject[]? filters) public FilterRunner(FilterObject[] filters)
{ {
if (filters == null)
throw new ArgumentNullException(nameof(filters));
Filters = filters; Filters = filters;
} }
public FilterRunner(string[]? filterStrings) public FilterRunner(string[] filterStrings)
{ {
if (filterStrings == null)
throw new ArgumentNullException(nameof(filterStrings));
var filters = new List<FilterObject>(); var filters = new List<FilterObject>();
foreach (string filterString in filterStrings) foreach (string filterString in filterStrings)
{ {

10
SabreTools.Test/Filter/PopulationTests.cs
View File

@@ -6,16 +6,6 @@ namespace SabreTools.Test.Filter
{ {
public class PopulationTests public class PopulationTests
{ {
[Fact]
public void PopulateFilterRunnerNullListTest()
{
// Setup the list
string[]? filters = null;
// Setup the filter
Assert.Throws<ArgumentNullException>(() => new FilterRunner(filters));
}
[Fact] [Fact]
public void PopulateFilterRunnerEmptyListTest() public void PopulateFilterRunnerEmptyListTest()
{ {