Add ParameterGroupName to ParameterEditor; fixes #720

COMET.Web.Common.Test/COMET.Web.Common.Test.csproj

@@ -28,7 +28,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="bunit.web" Version="1.26.64" />
+    <PackageReference Include="bunit.web" Version="1.36.0" />
     <PackageReference Include="DevExpress.Blazor" Version="23.2.9" />
     <PackageReference Include="Moq" Version="4.20.72" />
   </ItemGroup>

COMET.Web.Common.Tests/COMET.Web.Common.Tests.csproj

@@ -46,7 +46,7 @@
     </ItemGroup>
 
     <ItemGroup>
-        <PackageReference Include="bunit" Version="1.26.64" />
+        <PackageReference Include="bunit" Version="1.36.0" />
         <PackageReference Include="DotNetSeleniumExtras.WaitHelpers" Version="3.11.0" />
         <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
         <PackageReference Include="Moq" Version="4.20.72" />

COMET.Web.Common/COMET.Web.Common.csproj

@@ -31,8 +31,8 @@
     <ItemGroup>
         <PackageReference Include="AsyncEnumerator" Version="4.0.2" />
         <PackageReference Include="Blazored.FluentValidation" Version="2.2.0" />
-        <PackageReference Include="CDP4ServicesDal-CE" Version="27.4.0" />
-        <PackageReference Include="CDP4Web-CE" Version="27.4.0" />
+        <PackageReference Include="CDP4ServicesDal-CE" Version="27.4.1" />
+        <PackageReference Include="CDP4Web-CE" Version="27.4.1" />
         <PackageReference Include="DevExpress.Blazor" Version="23.2.9" />
         <PackageReference Include="FluentResults" Version="3.16.0" />
         <PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="8.0.10" />

COMETwebapp.Tests/COMETwebapp.Tests.csproj

@@ -15,7 +15,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="bunit" Version="1.26.64" />
+    <PackageReference Include="bunit" Version="1.36.0" />
     <PackageReference Include="DotNetSeleniumExtras.WaitHelpers" Version="3.11.0" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
     <PackageReference Include="Moq" Version="4.20.72" />

COMETwebapp.Tests/Pages/ModelDashboard/ModelDashboardTestFixture.cs

@@ -172,27 +172,28 @@ public async Task VerifyIterationSelection()
         public void VerifyIterationPreselection()
         {
             this.openedIterations.AddRange(new List<Iteration> { this.firstIteration });
-
-            this.context.RenderComponent<ModelDashboard>(parameters =>
-            {
-                parameters.Add(p => p.IterationId, this.firstIteration.Iid.ToShortGuid());
-            });
 
-            var navigation = this.context.Services.GetService<NavigationManager>();
+            var navigationManager = this.context.Services.GetRequiredService<NavigationManager>();
+            var testUri = $"/ModelDashboard?IterationId={this.firstIteration.Iid.ToShortGuid()}";
+
+            // Act: Navigate to the URI and render the component
+            navigationManager.NavigateTo(testUri);
+            this.context.RenderComponent<ModelDashboard>();
 
             Assert.Multiple(() =>
             {
                 Assert.That(this.viewModel.SelectedThing, Is.EqualTo(this.firstIteration));
-                Assert.That(navigation.Uri.Contains("server"), Is.True);
-                Assert.That(navigation.Uri.Contains(this.firstIteration.Iid.ToShortGuid()), Is.True);
+                Assert.That(navigationManager.Uri.Contains("server"), Is.True);
+                Assert.That(navigationManager.Uri.Contains(this.firstIteration.Iid.ToShortGuid()), Is.True);
             });
 
             this.viewModel.SelectedThing = null;
 
-            this.context.RenderComponent<ModelDashboard>(parameters =>
-            {
-                parameters.Add(p => p.IterationId, this.secondIteration.Iid.ToShortGuid());
-            });
+            testUri = $"/ModelDashboard?IterationId={this.secondIteration.Iid.ToShortGuid()}";
+
+            // Act: Navigate to the URI and render the component
+            navigationManager.NavigateTo(testUri);
+            this.context.RenderComponent<ModelDashboard>();
 
             Assert.That(this.viewModel.SelectedThing, Is.EqualTo(this.firstIteration));
         }

COMETwebapp.Tests/Pages/ParameterEditor/ParameterEditorTestFixture.cs

@@ -162,20 +162,27 @@ public void VerifyIterationPreselection()
         {
             this.openedIterations.AddRange(new List<Iteration> { this.firstIteration });
 
-            this.context.RenderComponent<ParameterEditor>(parameters => { parameters.Add(p => p.IterationId, this.firstIteration.Iid.ToShortGuid()); });
+            var navigationManager = this.context.Services.GetRequiredService<NavigationManager>();
+            var testUri = $"/ParameterEditor?IterationId={this.firstIteration.Iid.ToShortGuid()}";
 
-            var navigation = this.context.Services.GetService<NavigationManager>();
+            // Act: Navigate to the URI and render the component
+            navigationManager.NavigateTo(testUri);
+            this.context.RenderComponent<ParameterEditor>();
 
             Assert.Multiple(() =>
             {
                 Assert.That(this.viewModel.SelectedThing, Is.EqualTo(this.firstIteration));
-                Assert.That(navigation.Uri.Contains("server"), Is.True);
-                Assert.That(navigation.Uri.Contains(this.firstIteration.Iid.ToShortGuid()), Is.True);
+                Assert.That(navigationManager.Uri.Contains("server"), Is.True);
+                Assert.That(navigationManager.Uri.Contains(this.firstIteration.Iid.ToShortGuid()), Is.True);
             });
 
             this.viewModel.SelectedThing = null;
 
-            this.context.RenderComponent<ParameterEditor>(parameters => { parameters.Add(p => p.IterationId, this.secondIteration.Iid.ToShortGuid()); });
+            testUri = $"/ParameterEditor?IterationId={this.secondIteration.Iid.ToShortGuid()}";
+
+            // Act: Navigate to the URI and render the component
+            navigationManager.NavigateTo(testUri);
+            this.context.RenderComponent<ParameterEditor>();
 
             Assert.That(this.viewModel.SelectedThing, Is.EqualTo(this.firstIteration));
         }

COMETwebapp.Tests/Pages/Viewer/ViewerTestFixture.cs

@@ -151,20 +151,27 @@ public void VerifyIterationPreselection()
         {
             this.openedIterations.AddRange(new List<Iteration> { this.firstIteration });
 
-            this.context.RenderComponent<Viewer>(parameters => { parameters.Add(p => p.IterationId, this.firstIteration.Iid.ToShortGuid()); });
+            var navigationManager = this.context.Services.GetRequiredService<NavigationManager>();
+            var testUri = $"/Viewer?IterationId={this.firstIteration.Iid.ToShortGuid()}";
 
-            var navigation = this.context.Services.GetService<NavigationManager>();
+            // Act: Navigate to the URI and render the component
+            navigationManager.NavigateTo(testUri);
+            this.context.RenderComponent<Viewer>();
 
             Assert.Multiple(() =>
             {
                 Assert.That(this.viewModel.SelectedThing, Is.EqualTo(this.firstIteration));
-                Assert.That(navigation.Uri.Contains("server"), Is.True);
-                Assert.That(navigation.Uri.Contains(this.firstIteration.Iid.ToShortGuid()), Is.True);
+                Assert.That(navigationManager.Uri.Contains("server"), Is.True);
+                Assert.That(navigationManager.Uri.Contains(this.firstIteration.Iid.ToShortGuid()), Is.True);
             });
 
             this.viewModel.SelectedThing = null;
 
-            this.context.RenderComponent<Viewer>(parameters => { parameters.Add(p => p.IterationId, this.secondIteration.Iid.ToShortGuid()); });
+            testUri = $"/Viewer?IterationId={this.secondIteration.Iid.ToShortGuid()}";
+
+            // Act: Navigate to the URI and render the component
+            navigationManager.NavigateTo(testUri);
+            this.context.RenderComponent<Viewer>();
 
             Assert.That(this.viewModel.SelectedThing, Is.EqualTo(this.firstIteration));
         }

COMETwebapp.Tests/ViewModels/Components/ParameterEditor/ParameterTableViewModelTestFixture.cs

@@ -50,6 +50,8 @@ public class ParameterTableViewModelTestFixture
         private Iteration iteration;
         private Option option;
         private CDPMessageBus messageBus;
+        private ParameterGroup parameterGroup;
+        private ParameterGroup parameterGroup2;
 
         [SetUp]
         public void Setup()
@@ -86,12 +88,26 @@ public void Setup()
                 ShortName = "kg"
             };
 
+            this.parameterGroup = new ParameterGroup()
+            {
+                Iid = Guid.NewGuid(),
+                Name = "ParameterGroup 1"
+            };
+
+            this.parameterGroup2 = new ParameterGroup()
+            {
+                Iid = Guid.NewGuid(),
+                Name = "ParameterGroup 2",
+                ContainingGroup = this.parameterGroup
+            };
+
             var parameter1 = new Parameter()
             {
                 Iid = Guid.NewGuid(),
                 Owner = this.domain,
                 ParameterType = parameterType,
                 Scale = scale,
+                Group = this.parameterGroup,
                 ValueSet =
                 {
                     new ParameterValueSet()
@@ -111,6 +127,7 @@ public void Setup()
                 Owner = this.domain,
                 ParameterType = parameterType,
                 Scale = scale,
+                Group = this.parameterGroup,
                 ValueSet =
                 {
                     new ParameterValueSet()
@@ -130,6 +147,7 @@ public void Setup()
                 ParameterType = parameterType,
                 Scale = scale,
                 Owner = this.domain,
+                Group = this.parameterGroup2,
                 ValueSet =
                 {
                     new ParameterValueSet()
@@ -166,7 +184,8 @@ public void Setup()
             {
                 Iid = Guid.NewGuid(),
                 Name = "Container",
-                Parameter = { parameter3 }
+                Parameter = { parameter3 },
+                ParameterGroup = { this.parameterGroup, this.parameterGroup2 }
             };
 
             var elementDefinition = new ElementDefinition()
@@ -228,6 +247,7 @@ public void VerifyParameterRowProperties()
                 Assert.That(parameterRow.ParameterName, Is.EqualTo("mass"));
                 Assert.That(parameterRow.Option, Is.Empty);
                 Assert.That(parameterRow.State, Is.Empty);
+                Assert.That(parameterRow.ParameterGroupPath, Is.EqualTo($"{this.parameterGroup.Name} => {this.parameterGroup2.Name}"));
             });
         }
 

COMETwebapp/COMETwebapp.csproj

@@ -38,4 +38,8 @@
       </Content>
     </ItemGroup>
 
+    <ItemGroup>
+      <Folder Include="logs\" />
+    </ItemGroup>
+
 </Project>

COMETwebapp/Components/ParameterEditor/ParameterTable.razor

@@ -27,7 +27,7 @@
 
 @if (this.ViewModel.IsOnEditMode)
 {
     var parameters = new Dictionary<string, object>()
     {
         {"ViewModel", this.ViewModel.HaveComponentParameterTypeEditorViewModel},
         {"IsOnEditMode", true},
@@ -37,7 +37,7 @@
     var headerText = string.Empty;
     var isCloseButtonAvailable = true;
 
     switch (this.ViewModel.HaveComponentParameterTypeEditorViewModel)
     {
         case CompoundParameterTypeEditorViewModel:
         {
@@ -109,6 +109,7 @@
             </DxGridDataColumn>
             <DxGridDataColumn FieldName="@nameof(ParameterBaseRowViewModel.ModelCode)" Caption="@("Model Code")" AllowGroup="false" AllowSort="true" />
             <DxGridDataColumn FieldName="@nameof(ParameterBaseRowViewModel.OwnerName)" Caption="@("Owner")" AllowGroup="false" AllowSort="true" />
+            <DxGridDataColumn FieldName="@nameof(ParameterBaseRowViewModel.ParameterGroupPath)" Caption="@("Parameter Group Path")" AllowGroup="false" AllowSort="true" />
         </Columns>
     </DxGrid>
 </div>

COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs

@@ -73,12 +73,12 @@
 
             this.ParameterSwitchKindSelectorViewModel = new ParameterSwitchKindSelectorViewModel(this.Switch, this.IsReadOnly)
             {
                 OnUpdate = new EventCallbackFactory().Create(this, this.UpdateParameterValueSwitch)
             };
 
             this.ParameterTypeEditorSelectorViewModel = new ParameterTypeEditorSelectorViewModel(this.ParameterType, this.ValueSet, this.IsReadOnly, messageBus)
             {
                 ParameterValueChanged = new EventCallbackFactory().Create<(IValueSet,int)>(this, this.OnParameterValueChanged)
             };
 
             this.Disposables.Add(this.WhenAnyValue(x => x.ParameterSwitchKindSelectorViewModel.SwitchValue)
@@ -89,7 +89,7 @@
         /// Initializes this row view model properties
         /// </summary>
         /// <param name="isReadOnly">Value asserting if this row view model should be readonly or not</param>
         private void InitializesProperties(bool isReadOnly)
         {
             this.IsReadOnly = isReadOnly;
             this.ParameterName = this.Parameter.ParameterType is not null ? this.Parameter.ParameterType.Name : string.Empty;
@@ -112,6 +112,36 @@
             this.Option = this.ValueSet.ActualOption is not null ? this.ValueSet.ActualOption?.Name : string.Empty;
             this.State = this.ValueSet.ActualState is not null ? this.ValueSet.ActualState.Name : string.Empty;
             this.Switch = this.ValueSet.ValueSwitch;
+            this.CalculateParameterGroupData();
+        }
+
+        /// <summary>
+        /// Calculate and set the values of <see cref="ParameterGroup"/> parameters
+        /// </summary>
+        private void CalculateParameterGroupData()
+        {
+            var parameterGroupPath = string.Empty;
+
+            if (this.Parameter.Group != null)
+            {
+                var group = this.Parameter.Group;
+                parameterGroupPath = this.Parameter.Group.Name;
+
+                while (true)
+                {
+                    if (group.ContainingGroup != null)
+                    {
+                        parameterGroupPath = $"{group.ContainingGroup.Name} => {parameterGroupPath}";
+                        group = group.ContainingGroup;
+                    }
+                    else
+                    {
+                        break;
+                    }
+                }
+            }
+
+            this.ParameterGroupPath = parameterGroupPath;
         }
 
         /// <summary>
@@ -168,6 +198,11 @@
         /// </summary>
         public string OwnerName { get; private set; }
 
+        /// <summary>
+        /// Gets the <see cref="ParameterGroup" /> name
+        /// </summary>
+        public string ParameterGroupPath { get; private set; }
+
         /// <summary>
         /// Gets the switch for the published value
         /// </summary>
@@ -214,7 +249,7 @@
         /// </summary>
         /// <param name="value">The updated <see cref="IValueSet"/> with the index</param>
         /// <returns>an asynchronous operation</returns>
         private async Task OnParameterValueChanged((IValueSet valueSet, int _) value)
         {
             if (!this.IsReadOnly && value.valueSet is ParameterValueSetBase parameterValueSetBase)
             {
@@ -239,7 +274,7 @@
         /// Update the <see cref="ParameterSwitchKind" />
         /// </summary>
         /// <returns></returns>
         private async Task UpdateParameterValueSwitch()
         {
             if (!this.IsReadOnly && this.ParameterSwitchKindSelectorViewModel.SwitchValue != this.ValueSet.ValueSwitch && this.ValueSet is ParameterValueSetBase parameterValueSetBase)
             {