Skip to content

Draft : Pagination - #194

Open
mm-kgi wants to merge 23 commits into
developfrom
1511-pagination
Open

Draft : Pagination#194
mm-kgi wants to merge 23 commits into
developfrom
1511-pagination

Conversation

@mm-kgi

@mm-kgi mm-kgi commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

mm-kgi and others added 20 commits July 23, 2026 12:53
…hell descriptors with template mapping support
…th user-context aware caching and automated error mapping
…ion support

- Updated SubmodelRepositoryService to implement pagination for submodel retrieval, including cursor handling.
- Introduced SubmodelPageCursor to manage state across paginated requests.
- Modified IPluginDataHandler and IPluginDataProvider interfaces to include pagination parameters.
- Adjusted PluginDataHandler and PluginDataProvider implementations to support new pagination logic.
- Enhanced unit tests for SubmodelRepositoryService to cover new pagination scenarios.
- Added new tests for SubmodelPageCursor to ensure correct encoding and decoding behavior.
- Updated existing tests in PluginDataHandler and PluginDataProvider to accommodate changes in method signatures.
…ion support

Updated SubmodelRepositoryService to implement pagination for submodel retrieval, including cursor handling.

Introduced SubmodelPageCursor to manage state across paginated requests.

Modified IPluginDataHandler and IPluginDataProvider interfaces to include pagination parameters.

Adjusted PluginDataHandler and PluginDataProvider implementations to support new pagination logic.

Enhanced unit tests for SubmodelRepositoryService to cover new pagination scenarios.

Added new tests for SubmodelPageCursor to ensure correct encoding and decoding behavior.

Updated existing tests in PluginDataHandler and PluginDataProvider to accommodate changes in method signatures.
Refactored `SubmodelDescriptorService` to introduce a two-field composite cursor pagination algorithm for efficient data retrieval. Updated `GetAllSubmodelDescriptorsAsync` to use the new pagination logic and include cursor metadata.

Added helper methods `CollectSubmodelDescriptorPageAsync`, `ProcessShellBatch`, and `GetSubmodelIdsForShell` to support the new pagination mechanism. Introduced `PaginationState` and `SubmodelDescriptorPageResult` to manage pagination state.

Updated test cases in `SubmodelDescriptorServiceTests` to validate the new pagination logic, including a new test method `GetAllSubmodelDescriptorsAsync_ResumesCorrectly_WithTwoFieldCompositeCursor`. Refactored `CreateShell` to support customizable test data.

Removed the obsolete `ExtractDistinctSubmodelIds` method and added necessary `using` directives for additional namespaces.
…ce with pagination and associated test data fixtures
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

mm-kgi added 3 commits August 4, 2026 12:27
…d repository services to utilize new pagination logic
…x minor issues in SubmodelRepositoryService and PluginDataProvider
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 New issue

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Test & Coverage Report

Test Results Summary

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

View Detailed Test Results


Code Coverage

Unit Tests Coverage

Package Line Rate Branch Rate Complexity Health
AAS.TwinEngine.DataEngine 92% 78% 2224
Summary 92% (3543 / 3867) 78% (1481 / 1895) 2224

Minimum allowed line rate is 80%

Module Tests Coverage

Package Line Rate Branch Rate Complexity Health
AAS.TwinEngine.DataEngine 61% 43% 2224
Summary 61% (2359 / 3867) 43% (806 / 1895) 2224

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