Skip to content

#257: Enhancement TestPlugin to verify access SubmodelElement inside a SubmodelCollection within a SubmodelList#19

Merged
HolgerSantelmann merged 4 commits intodevelopfrom
feature/257_review_SubmodelCol
Jan 22, 2026
Merged

#257: Enhancement TestPlugin to verify access SubmodelElement inside a SubmodelCollection within a SubmodelList#19
HolgerSantelmann merged 4 commits intodevelopfrom
feature/257_review_SubmodelCol

Conversation

@HolgerSantelmann
Copy link
Copy Markdown

Purpose: Verifiy fix: [Bug] Error when requesting SubmodelElement endpoint for SubmodelCollection within SubmodelList #257

Enhancement of test plugin so it supports the following scenario:
SubmodelElementList

  • SubmodelElementCollection1
    • SubmodelElement11
    • SubmodelElement12
  • SubmodelElementCollection2
    • SubmodelElement21
    • SubmodelElement22

Marking section has been enhanced with a further SubmodelElementCollection.
Further apiCollection call "Get Submodel Element - Markings" has been introduced.

@github-actions
Copy link
Copy Markdown

Test & Coverage Report

Test Results Summary

Metric Count
✅ Passed 461
❌ Failed 0
⏭️ Skipped 0

View Detailed Test Results


Code Coverage

Unit Tests Coverage

Package Line Rate Branch Rate Complexity Health
AAS.TwinEngine.DataEngine 90% 80% 1188
Summary 90% (2053 / 2283) 80% (836 / 1044) 1188

Minimum allowed line rate is 80%

Module Tests Coverage

Package Line Rate Branch Rate Complexity Health
AAS.TwinEngine.DataEngine 56% 40% 1188
Summary 56% (1279 / 2283) 40% (421 / 1044) 1188

@HolgerSantelmann HolgerSantelmann merged commit 2b973bd into develop Jan 22, 2026
3 checks passed
@HolgerSantelmann HolgerSantelmann deleted the feature/257_review_SubmodelCol branch January 22, 2026 11:58
mm-psy referenced this pull request in mm-psy/AAS.TwinEngine.DataEngine Jan 23, 2026
* Add dependency-review (#3)

* Add dependency-review

* Fix branch

* Add Scorecard supply-chain security (#2)

* Create docker-publish.yml (#7)

* Create dotnet (#6)

* Create dotnet.yml

* Add comment

* Test

* Test

* Enhance CI workflow with code coverage reporting and PR comments

* Fix wrong test run

* Refactor CI workflow to improve test result handling and add module test execution

* Enhance CI workflow by combining test and coverage reports, and updating paths for coverage files

* Enhance Submodel Element request handling(#8)

* Add dependabot-version-updates (#4)

* Add dependabot-version-updates

* Enhancement of DataEngine Implementation  (#12)

* Added example folder With Dpp-plugin  (#13)

* Restructure Dockerfile for improved build process and clarity (#14)

* Added Test Plugin and Example for 3 submodels using Test plugin (#11)

* #182: Test plugin review results (#17)

* #182: Rename of folder

* #182: Refinement of apiCollection Readmes and environments

* #182: Also rename of Aas.TwinEngine.Plugin.TestPlugin to AAS.TwinEngine.Plugin.TestPlugin in files

* Remove obsolete test project files and add new project files for Plugin testing

* Remove obsolete test project files and add new project files for Plugin testing

* #182: Rename of projects

* #182: Readme refined

* #182: Refinement of readme

* #182: Icons removed from readme

---------

Co-authored-by: Kevalkumar <kgi@mm-software.com>

* Add codeql (#5)

* Create codeql.yml

* Update branches

* Enhancement of DataEngine Implementation (#18)

* Add hotfix develop release branches to actions (#15)

* Update dependency-review.yml

* Update dotnet.yml

* Update docker-publish.yml

* Update docker-publish.yml to refine branch triggers and enhance Docker metadata extraction

* Fix image name

* Add manually trigger a workflow

* #257: Enhancement TestPlugin to verify access SubmodelElement inside a SubmodelCollection within a SubmodelList (#19)

* README.md with detailed DataEngine overview and feature descriptions (#22)

* Refactor SyncShellDescriptorsAsync to log errors and return instead of throwing exceptions for null checks (#20)

---------

Co-authored-by: Kevalkumar Ghelani <kgi@mm-software.com>
Co-authored-by: Hardi Shah <hsh@mm-software.com>
Co-authored-by: mm-hsn <hsn@mm-software.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants