[YANGTOOLS-714] Reduce YangSchemaSourceImpl visibility Created: 11/Nov/16 Updated: 10/Apr/22 Resolved: 10/Jan/17 |
|
| Status: | Resolved |
| Project: | yangtools |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Robert Varga | Assignee: | Igor Foltin |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||||||||||||||||||
| External issue ID: | 7159 | ||||||||||||||||||||||||
| Description |
|
We interact directly with the reactor all over the place – yangtools, mdsal, netconf, leading to implementation-level leaks, which prevent us from refactoring. Provide a set of test utility classes which will be usable in all context where YangStatementSourceImpl is being imported. This should be centered around a SharedSchemaRepository, which can load files and resources and provide a full schemacontext – very much like SimpleModuleTest does in yang-model-export. |
| Comments |
| Comment by Robert Varga [ 11/Nov/16 ] |
|
Once the artifact is available, convert all the places to use it, so we can refactor our parser implementation without breaking the world. |
| Comment by Igor Foltin [ 08/Dec/16 ] |
| Comment by Robert Varga [ 08/Dec/16 ] |
|
Looks good, but we also need conversion in mdsal, controller and netconf to unblock #7051. |
| Comment by Martin Ciglan [ 12/Dec/16 ] |
|
mdsal patch: |
| Comment by Robert Varga [ 22/Dec/16 ] |
|
Test-utils have been delivered, we now need to have the projects migrated. Changed summary and turning this issue into a tracker blocked by per-project issues. |