Stfaseid http refactor
Description
Draft for new API. Needs extensive manual testing. Only merge if changes in https://code.fbi.h-da.de/danet/api/-/tree/10-refactor-current-proto-definitions-stfahelm and upcoming changes to CLI are approved.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
-
Bug fix (non-breaking change which fixes an issue) -
New feature (non-breaking change which adds functionality) -
Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
-
My code follows the code style of this project. -
My change requires a change to the documentation. -
I have updated the documentation accordingly. -
I have read the CONTRIBUTING document. -
I have added tests to cover my changes. -
All new and existing tests passed.
Edited by Ghost User
Merge request reports
Activity
requested review from @istaester, @istmabauc, and @m.stiemerling
Linter doesn't seem to be happy yet: https://code.fbi.h-da.de/danet/gosdn/-/jobs/368948
mentioned in issue #167 (closed)
mentioned in issue #168
added 1 commit
- fe27e00a - Fix: HTTP API test did not wait for server to start properly
added 1 commit
- 47052145 - fix: integration tests should no longer fail
added 32 commits
-
47052145...44b946b1 - 11 commits from branch
develop
- 0aabcaf4 - updated types, states, etc. according to naming conventions, WIP
- 39fe4769 - updated service and server names according to name changes of refactored api
- 3afcf4f9 - updated pnd api to fit refactoring, maybe change to list version, WIP
- b47ff23c - updated api calls for server/pnd
- f2e745bb - updated api calls for server/core
- 82d5eb42 - updated api/grpc according to other api changes
- b28ea3e3 - fix for error with changeresponse and wron initial southboundtype
- f5d2c229 - generated new mock file to fit api changes
- ba5a06ee - fix for error with southbound types and error handling, api, pnd and core test are WIP
- 7e8c1c4f - fix test case wrong if clause
- 800e4c5a - tests and and minor changes for request logic correctly, WIP
- be26ce24 - added missing method SetSbiList
- 7923de92 - fixed get Pnd test to fit new api
- 166f9769 - Remove TODO comment as task is done
- 15223e20 - String replacement for path in GetPath()
- 9be63670 - Linter pleasing
- dcc6af04 - changed var name to fit naming convention
- a31d9051 - Fix: HTTP API test did not wait for server to start properly
- d67ec889 - linter pleasing
- 32839ba3 - fix: integration tests should no longer fail
- f54e3b0b - Merge branch 'stfaseid_http_refactor' of https://code.fbi.h-da.de/danet/gosdn...
Toggle commit list-
47052145...44b946b1 - 11 commits from branch
Please register or sign in to reply