Index: Ringtoets/DuneErosion/src/Ringtoets.DuneErosion.Forms/Views/DuneLocationRow.cs
===================================================================
diff -u -r75cdc7840bc359c9cf70795d58297708e48a27b2 -redece3c8789fe858a32c3609d5cc83d33d81ca7e
--- Ringtoets/DuneErosion/src/Ringtoets.DuneErosion.Forms/Views/DuneLocationRow.cs (.../DuneLocationRow.cs) (revision 75cdc7840bc359c9cf70795d58297708e48a27b2)
+++ Ringtoets/DuneErosion/src/Ringtoets.DuneErosion.Forms/Views/DuneLocationRow.cs (.../DuneLocationRow.cs) (revision edece3c8789fe858a32c3609d5cc83d33d81ca7e)
@@ -32,38 +32,26 @@
namespace Ringtoets.DuneErosion.Forms.Views
{
///
- /// This class represents a row of .
+ /// This class represents a row of .
///
- public class DuneLocationRow : CalculatableRow
+ public class DuneLocationRow : CalculatableRow
{
- private readonly Func getCalculationFunc;
-
///
/// Creates a new instance of .
///
- /// The to wrap.
- /// for obtaining a
- /// based on .
+ /// The to wrap.
/// Thrown when any input parameter is null.
- public DuneLocationRow(DuneLocation duneLocation, Func getCalculationFunc)
- : base(duneLocation)
- {
- if (getCalculationFunc == null)
- {
- throw new ArgumentNullException(nameof(getCalculationFunc));
- }
+ public DuneLocationRow(DuneLocationCalculation duneLocationCalculation)
+ : base(duneLocationCalculation) {}
- this.getCalculationFunc = getCalculationFunc;
- }
-
///
/// Gets the .
///
public string Name
{
get
{
- return CalculatableObject.Name;
+ return CalculatableObject.DuneLocation.Name;
}
}
@@ -74,7 +62,7 @@
{
get
{
- return CalculatableObject.Id;
+ return CalculatableObject.DuneLocation.Id;
}
}
@@ -85,7 +73,7 @@
{
get
{
- return CalculatableObject.Location;
+ return CalculatableObject.DuneLocation.Location;
}
}
@@ -96,7 +84,7 @@
{
get
{
- return CalculatableObject.CoastalAreaId;
+ return CalculatableObject.DuneLocation.CoastalAreaId;
}
}
@@ -107,7 +95,7 @@
{
get
{
- return CalculatableObject.Offset.ToString(DuneErosionDataResources.DuneLocation_Offset_format, CultureInfo.InvariantCulture);
+ return CalculatableObject.DuneLocation.Offset.ToString(DuneErosionDataResources.DuneLocation_Offset_format, CultureInfo.InvariantCulture);
}
}
@@ -119,7 +107,7 @@
{
get
{
- return getCalculationFunc(CalculatableObject).Output?.WaterLevel ?? RoundedDouble.NaN;
+ return CalculatableObject.Output?.WaterLevel ?? RoundedDouble.NaN;
}
}
@@ -131,7 +119,7 @@
{
get
{
- return getCalculationFunc(CalculatableObject).Output?.WaveHeight ?? RoundedDouble.NaN;
+ return CalculatableObject.Output?.WaveHeight ?? RoundedDouble.NaN;
}
}
@@ -143,7 +131,7 @@
{
get
{
- return getCalculationFunc(CalculatableObject).Output?.WavePeriod ?? RoundedDouble.NaN;
+ return CalculatableObject.Output?.WavePeriod ?? RoundedDouble.NaN;
}
}
@@ -154,7 +142,7 @@
{
get
{
- return CalculatableObject.D50;
+ return CalculatableObject.DuneLocation.D50;
}
}
}
Index: Ringtoets/DuneErosion/test/Ringtoets.DuneErosion.Forms.Test/Views/DuneLocationRowTest.cs
===================================================================
diff -u -rc743d1f229ff553fff9a05964c1adac6e510df4a -redece3c8789fe858a32c3609d5cc83d33d81ca7e
--- Ringtoets/DuneErosion/test/Ringtoets.DuneErosion.Forms.Test/Views/DuneLocationRowTest.cs (.../DuneLocationRowTest.cs) (revision c743d1f229ff553fff9a05964c1adac6e510df4a)
+++ Ringtoets/DuneErosion/test/Ringtoets.DuneErosion.Forms.Test/Views/DuneLocationRowTest.cs (.../DuneLocationRowTest.cs) (revision edece3c8789fe858a32c3609d5cc83d33d81ca7e)
@@ -37,17 +37,6 @@
public class DuneLocationRowTest
{
[Test]
- public void Constructor_GetCalculationFuncNull_ThrowsArgumentNullException()
- {
- // Call
- TestDelegate call = () => new DuneLocationRow(new TestDuneLocation(), null);
-
- // Assert
- var exception = Assert.Throws(call);
- Assert.AreEqual("getCalculationFunc", exception.ParamName);
- }
-
- [Test]
[TestCase(34.1)]
[TestCase(34.0)]
public void Constructor_WithOutput_ExpectedValues(double offSet)
@@ -59,7 +48,7 @@
Offset = offSet,
D50 = 0.000183
});
- var duneLocationCalculation = new DuneLocationCalculation(new TestDuneLocation())
+ var duneLocationCalculation = new DuneLocationCalculation(duneLocation)
{
Output = new DuneLocationOutput(CalculationConvergence.CalculatedConverged, new DuneLocationOutput.ConstructionProperties
{
@@ -70,11 +59,11 @@
};
// Call
- var row = new DuneLocationRow(duneLocation, dl => duneLocationCalculation);
+ var row = new DuneLocationRow(duneLocationCalculation);
// Assert
- Assert.IsInstanceOf>(row);
- Assert.AreSame(duneLocation, row.CalculatableObject);
+ Assert.IsInstanceOf>(row);
+ Assert.AreSame(duneLocationCalculation, row.CalculatableObject);
Assert.AreEqual(duneLocation.Id, row.Id);
Assert.AreEqual(duneLocation.Name, row.Name);
Assert.AreSame(duneLocation.Location, row.Location);
@@ -97,11 +86,10 @@
public void Constructor_WithoutOutput_ExpectedValues()
{
// Setup
- var duneLocation = new DuneLocation(1, "test location", new Point2D(3.3, 4.4), new DuneLocation.ConstructionProperties());
var duneLocationCalculation = new DuneLocationCalculation(new TestDuneLocation());
// Call
- var row = new DuneLocationRow(duneLocation, dl => duneLocationCalculation);
+ var row = new DuneLocationRow(duneLocationCalculation);
// Assert
Assert.IsNaN(row.WaterLevel);