Skip to content

Commit 86ecce7

Browse files
committed
covering SharedParameterFile.Save with tests
1 parent 2d284a1 commit 86ecce7

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

src/Parameters/Shared/SharedParameterFile.ParameterDefinition.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ public class ParameterCollection : List<ParameterDefinition>
246246
/// <exception cref="T:System.ArgumentNullException">parameterFile</exception>
247247
/// <inheritdoc />
248248
public ParameterCollection(SharedParameterFile parameterFile, IEnumerable<ParameterDefinition> parameters)
249-
:base(parameters)
249+
:base(parameters ?? new List<ParameterDefinition>())
250250
{
251251
this.parameterFile = parameterFile ?? throw new ArgumentNullException(nameof(parameterFile));
252252
}

tests/SharedParameterFileTests.cs

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -343,5 +343,19 @@ public void FileIsSerializedProperly()
343343

344344
Assert.Equal(simpleSharedFromBuiltText, simpleSharedFromDisk);
345345
}
346+
/// <summary>
347+
/// Saves the a catalog to a file.
348+
/// </summary>
349+
[Fact]
350+
public void SavingSharedParametersToFile()
351+
{
352+
var sharedParamFile = new SharedParameterFile();
353+
sharedParamFile.Parameters.Add(
354+
new Guid("61ff3d56-09d7-4049-8c78-4abe745e4e5a"), "EquipmentName",
355+
"Identity Data", ParameterType.Text
356+
);
357+
var sharedParamFileTmp = $"{Path.GetTempFileName()}.txt";
358+
Assert.True(sharedParamFile.Save(sharedParamFileTmp) && File.Exists(sharedParamFileTmp));
359+
}
346360
}
347361
}

0 commit comments

Comments
 (0)