Add get and set and notification support to the fields in the configuration by sahithi-nukala · Pull Request #405 · eclipse-score/communication

sahithi-nukala · 2026-05-11T12:16:04Z

Add useGetIfAvailable/useSetIfAvailable to LoLa field deployment

Introduce dedicated LolaFieldInstanceDeployment class replacing the former LolaEventInstanceDeployment type alias, adding field-specific members use_get_if_available_ and use_set_if_available_
Fix mw_com_config_schema.json: move useGetIfAvailable and useSetIfAvailable into the field "properties" object so they are correctly validated when additionalProperties is false.
Adapt existing test and add new tests for LolaFieldInstanceDeployment
Update PUML diagram and readme document

Issue: SWP-250429

- Introduce dedicated LolaFieldInstanceDeployment class replacing the former LolaEventInstanceDeployment type alias, adding field-specific members use_get_if_available_ and use_set_if_available_. - Fix mw_com_config_schema.json: move useGetIfAvailable and useSetIfAvailable into the field "properties" object so they are correctly validated when additionalProperties is false. Issue: SWP-250429

- Update config_parser.cpp to read optional field values useGetIfAvailable/useSetIfAvailable - Fix existing unit test of loloa field instance deployment. Issue: SWP-250429

Update existing tests which refer LolaFieldInstanceDeployment to support additional optional useGetIfAvailable and useSetIfAvailable Issue: SWP-250429

- Add tests to verfiy behaviour of option field values useGetIfAvailable/useSetIfAvailable Issue: SWP-250429

- Update ReadMe and UML diagram to refelct updated changes optional field values useGetIfAvailable/useSetIfAvailable Issue: SWP-250429

- Remove the redundant unit tests and update the exsiting test for default field setter/getter value - Update the default values of useGetIfAvailable/useSetIfAvailable from default false to true - Update the plant UML diagram to retain only necessary information Issue: SWP-250429

… LolaEventInstanceDeployment members - Delegate GetNumberOfSampleSlots(), GetNumberOfTracingSlots(), and SetNumberOfSampleSlots() to the inner lola_event_instance_deployment_ member - Delegate CreateFromJson() event parsing and Serialize() event part to LolaEventInstanceDeployment - Add a non-template GetSkeletonEventProperties() overload in skeleton_service_element_binding_factory_impl.h that delegates to the event deployment overload - Update all call sites: config_parser.cpp, configuration_test_resources.cpp, and all affected test files

bemerybmw · 2026-05-22T05:23:59Z

+class LolaFieldInstanceDeployment
+{
+  public:
+    using SampleSlotCountType = std::uint16_t;


These aliases are part of LolaEventInstanceDeployment so should be removed. If you need them here, then you should access them via LolaEventInstanceDeployment.

I have removed -
using SampleSlotCountType = std::uint16_t; using SubscriberCountType = std::uint8_t; using TracingSlotSizeType = std::uint8_t;
and updated with the "LolaEventInstanceDeployment". example like below -
void SetNumberOfSampleSlots(LolaEventInstanceDeployment::SampleSlotCountType number_of_sample_slots) noexcept;

bemerybmw · 2026-05-22T05:33:40Z

                        "type": "string",
                        "title": "Service Instance Specifier",
-                        "description": "Represents the InstanceSpecifier value. This is itself an AUTOSAR short-name-path (representing a port prototype of a software component instance). For comfort reasons, we allow also the usage of abbreviated short-name-paths (short-name-paths with a number of components starting from root being removed) in case they are still unique within an executable (this json/configuration is a per executable config!). From AUTOSAR concept perspective an executable contains a 'root sw-component', which itself can be a nested composition of AdaptiveApplicationSwComponentTypes). Because of this 'compositing feature' it could be the case, that the direct short-name of a port prototype isn’t unique within the composition making up the executable. In reality however the port names are 95% unique and forcing the usage of lengthy short-name-paths can be annoying! So this translates to the following rules:\n1. A config file with two “instanceSpecifiers”, where one is only a trailing substring of the other is INVALID!\n2. If user code makes a resolve/lookup providing an 'instanceSpecifier' in the form of an abbreviated short-name-path, which turns out NOT being unique with respect to the config, this is a hard programming/configuration fault, which shall lead to an abort."
+                        "description": "Represents the InstanceSpecifier value. This is itself an AUTOSAR short-name-path (representing a port prototype of a software component instance). For comfort reasons, we allow also the usage of abbreviated short-name-paths (short-name-paths with a number of components starting from root being removed) in case they are still unique within an executable (this json/configuration is a per executable config!). From AUTOSAR concept perspective an executable contains a 'root sw-component', which itself can be a nested composition of AdaptiveApplicationSwComponentTypes). Because of this 'compositing feature' it could be the case, that the direct short-name of a port prototype isn\u2019t unique within the composition making up the executable. In reality however the port names are 95% unique and forcing the usage of lengthy short-name-paths can be annoying! So this translates to the following rules:\n1. A config file with two \u201cinstanceSpecifiers\u201d, where one is only a trailing substring of the other is INVALID!\n2. If user code makes a resolve/lookup providing an 'instanceSpecifier' in the form of an abbreviated short-name-path, which turns out NOT being unique with respect to the config, this is a hard programming/configuration fault, which shall lead to an abort."


Why did this change?

bemerybmw · 2026-05-22T05:41:31Z

    ExpectLolaFieldInstanceDeploymentObjectsEqual(reconstructed_unit, unit);
 }

+TEST_F(LolaFieldInstanceDeploymentFixture, UseGetIfAvailableIsTrueAfterRoundTripSerialisation)


You can use a paramaterised test for this. You can search for TEST_P in the code base for examples.

bemerybmw · 2026-05-22T05:53:19Z


-            const auto& service_instance_map = [service_element_type, &lola_service_instance_deployment]() {
-                if (service_element_type == ServiceElementType::EVENT)
+            const auto service_element_name = service_element.service_element_name.data();


To minimize the duplicated code, maybe we can change the lambda to be get_number_of_tracing_slots which accepts auto service_elements_map (which can be events_ or fields_) and do the map lookup within the lambda?

bemerybmw · 2026-05-22T05:57:30Z

+    EXPECT_EQ(secondDeploymentInfo.fields_.at("CurrentTemperatureFrontLeft")
+                  .lola_event_instance_deployment_.max_concurrent_allocations_.value(),
+              1);
+    EXPECT_FALSE(secondDeploymentInfo.fields_.at("CurrentTemperatureFrontLeft").use_get_if_available_);


The values should be changed so that they're true. Makes it more likely to catch if we're not correctly parsing the value (since defualt is false).

Done. And updated the mw_com_config.json file also accordingly

bemerybmw · 2026-05-22T06:06:04Z

+    {static} + CreateFromJson(json_object : const score::json::Object&) : LolaFieldInstanceDeployment
+    + Serialize() const : score::json::Object
+    --
+    <u>Type Aliases:</u>


Aliases should be removed since they're part of the event deployment.

bemerybmw · 2026-05-22T06:06:46Z

-    + is_set_configured: bool
-    + is_get_configured: bool
+    {static} + constexpr serializationVersion : std::uint32_t
+    + max_subscribers_ : std::optional<SubscriberCountType>


We now compose a LolaEventInstanceDeployment.

bemerybmw · 2026-05-22T07:32:55Z

+    // coverity[autosar_cpp14_m11_0_1_violation]
+    LolaEventInstanceDeployment lola_event_instance_deployment_;
+    // coverity[autosar_cpp14_m11_0_1_violation]
+    bool use_get_if_available_;


Should be an optional because it's mandatory on proxy side and not used on skeleton side. Doesn't make sense to have a value on skeleton side.

sahithi-nukala requested review from LittleHuba, bemerybmw, castler, crimson11, hoe-jo and limdor as code owners May 11, 2026 12:16

sahithi-nukala force-pushed the suya_Add_get_and_set_and_notification_support_to_the_fields_in_the_configuration branch 2 times, most recently from 65d3943 to 7c3e519 Compare May 11, 2026 13:11

soldier-sky added 6 commits May 18, 2026 18:14

Update config parser to read useGetIfAvailable/useSetIfAvailable

6a83ef6

- Update config_parser.cpp to read optional field values useGetIfAvailable/useSetIfAvailable - Fix existing unit test of loloa field instance deployment. Issue: SWP-250429

Adapt existing tests to support fields with optional setter/getter

1875251

Update existing tests which refer LolaFieldInstanceDeployment to support additional optional useGetIfAvailable and useSetIfAvailable Issue: SWP-250429

Add unit tests to verify optional setter and getter field values

5204e02

- Add tests to verfiy behaviour of option field values useGetIfAvailable/useSetIfAvailable Issue: SWP-250429

Update configuration documentation

5752204

- Update ReadMe and UML diagram to refelct updated changes optional field values useGetIfAvailable/useSetIfAvailable Issue: SWP-250429

sahithi-nukala marked this pull request as draft May 18, 2026 12:45

sahithi-nukala force-pushed the suya_Add_get_and_set_and_notification_support_to_the_fields_in_the_configuration branch from 7c3e519 to 4ccfdb8 Compare May 18, 2026 12:46

sahithi-nukala marked this pull request as ready for review May 18, 2026 13:14

sahithi-nukala marked this pull request as draft May 20, 2026 13:11

sahithi-nukala force-pushed the suya_Add_get_and_set_and_notification_support_to_the_fields_in_the_configuration branch from 2adacc7 to 4ccfdb8 Compare May 20, 2026 13:22

sahithi-nukala force-pushed the suya_Add_get_and_set_and_notification_support_to_the_fields_in_the_configuration branch 2 times, most recently from 4ccfdb8 to eef7275 Compare May 20, 2026 13:34

sahithi-nukala marked this pull request as ready for review May 20, 2026 13:34

bemerybmw requested changes May 22, 2026

View reviewed changes

bemerybmw reviewed May 22, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add get and set and notification support to the fields in the configuration#405

Add get and set and notification support to the fields in the configuration#405
sahithi-nukala wants to merge 7 commits into
eclipse-score:mainfrom
sahithi-nukala:suya_Add_get_and_set_and_notification_support_to_the_fields_in_the_configuration

sahithi-nukala commented May 11, 2026 •

edited

Loading

Uh oh!

bemerybmw May 22, 2026

Uh oh!

sahithi-nukala May 25, 2026

Uh oh!

bemerybmw May 22, 2026

Uh oh!

bemerybmw May 22, 2026

Uh oh!

bemerybmw May 22, 2026

Uh oh!

bemerybmw May 22, 2026

Uh oh!

sahithi-nukala May 25, 2026 •

edited

Loading

Uh oh!

bemerybmw May 22, 2026

Uh oh!

bemerybmw May 22, 2026

Uh oh!

bemerybmw May 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

sahithi-nukala commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sahithi-nukala May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

sahithi-nukala commented May 11, 2026 •

edited

Loading

sahithi-nukala May 25, 2026 •

edited

Loading