-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
222 integration of ngsi ld endpoints #247
base: NGSI-LD
Are you sure you want to change the base?
Conversation
Addes reference to readme
edit authors and alumni
adjusted copyright year
# Conflicts: # filip/models/base.py
Add authorization field to fiware-header
216 release v0 3 0
# Conflicts: # README.md
…ible-with-Pydantic-v2 fix: updated regex to pattern
…hub.com/RWTH-EBC/FiLiP into 221-ngsi-ld-migrate-V2-datamodels-in-ld
Signed-off-by: iripiri <iris.m.koester@gmail.com>
…1' into 222-Integration-of-NGSI-LD-endpoints
Signed-off-by: iripiri <iris.m.koester@gmail.com>
…mentation Signed-off-by: iripiri <iris.m.koester@gmail.com>
…hub.com/RWTH-EBC/FiLiP into 221-ngsi-ld-migrate-V2-datamodels-in-ld
Signed-off-by: iripiri <iris.m.koester@gmail.com>
…hub.com/RWTH-EBC/FiLiP into 221-ngsi-ld-migrate-V2-datamodels-in-ld
Signed-off-by: iripiri <iris.m.koester@gmail.com>
Signed-off-by: iripiri <iris.m.koester@gmail.com>
…1' into 222-Integration-of-NGSI-LD-endpoints
Endpoints (implemented & tested):Entities
Batch operations
Subscriptions
|
csf: Optional[str] = None, | ||
limit: Optional[PositiveInt] = 100, | ||
response_format: Optional[Union[AttrsFormat, str]] = AttrsFormat.NORMALIZED.value, | ||
) -> Union[Dict[str, Any]]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iripiri The type hint should be adjusted because normally, a list of ContextLDEntity will be returned.
""" | ||
Cleanup entities from test server | ||
""" | ||
entity_test_types = ["MyType", "room"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why should the cleanup limited to these types of entities?
CB_URL = "http://137.226.248.200:1027" | ||
self.cb_client = ContextBrokerLDClient(url=CB_URL, | ||
fiware_header=self.fiware_header) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a cleanup step here will increase the robustness of the tests
entity_test_types = ["MyType", "room"] | ||
|
||
fiware_header = FiwareLDHeader() | ||
with ContextBrokerLDClient(fiware_header=fiware_header) as client: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to implement a clean up function for LD context broker? It can be placed under filip.utils.cleanup
self.cb_client.post_entity(entity=self.entity_2) | ||
entity_list = self.cb_client.get_entity_list() | ||
self.assertEqual(len(entity_list), 2) | ||
self.assertEqual(entity_list[0].id, self.entity.id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can actually not ensure that the order of the entities is not changing.
self.assertEqual(response.status_code, 404) | ||
self.assertEqual(response.json()["title"], "Entity Not Found") | ||
|
||
"""Test2""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This scenario is different as the the description above.
close #222