Fixes #2432 Create new MoleculeProperties parameters as Global

New parameters added to a MoleculeProperties container were created with
Local build mode, inconsistent with all other MoleculeProperties parameters
(Global). This caused formula references to fail because the parameter was
resolved as local while living at the global <MOLECULE>|parameter path.

DefaultParameterBuildMode now returns Global for MoleculeProperties containers.

Author: rwmcintosh

src/MoBi.Core/Domain/Extensions/ContainerExtensions.cs

@@ -25,7 +25,8 @@ public static ParameterBuildMode DefaultParameterBuildMode(this IContainer conta
       {
          var needsGlobalParameter = container.IsAnImplementationOf<TransporterMoleculeContainer>() ||
                                     container.IsAnImplementationOf<ReactionBuilder>() ||
-                                    container.IsAnImplementationOf<InteractionContainer>();
+                                    container.IsAnImplementationOf<InteractionContainer>() ||
+                                    container.IsMoleculeProperties();
 
          return needsGlobalParameter ? ParameterBuildMode.Global : ParameterBuildMode.Local;
       }

tests/MoBi.Tests/Core/Commands/AddParameterToContainerCommandSpecs.cs

@@ -75,6 +75,19 @@ protected override ParameterBuildMode GetBuildModeType()
       }
    }
 
+   public class When_adding_a_new_parameter_to_molecule_properties_container : When_testing_for_appropriate_parameter_build_modes
+   {
+      protected override IContainer GetContainer()
+      {
+         return new Container().WithName(Constants.MOLECULE_PROPERTIES);
+      }
+
+      protected override ParameterBuildMode GetBuildModeType()
+      {
+         return ParameterBuildMode.Global;
+      }
+   }
+
    public class When_adding_a_new_parameter_to_a_generic_container : When_testing_for_appropriate_parameter_build_modes
    {
       protected override IContainer GetContainer()