telefonicaid / iotagent-node-lib

Module to enable IoT Agent developers to build custom agents for their devices that can easily connect to NGSI Context Brokers
https://iotagent-node-lib.rtfd.io/
GNU Affero General Public License v3.0
58 stars 84 forks source link

typeInformation contain global config values #1580

Closed KeshavSoni2511 closed 4 months ago

KeshavSoni2511 commented 4 months ago

Fix for issue #1515

AlvaroVega commented 4 months ago

Could you please include the simplification of blocks referenced in https://github.com/telefonicaid/iotagent-node-lib/issues/1515#issuecomment-1923480572 ?

KeshavSoni2511 commented 4 months ago

Could you please include the simplification of blocks referenced in #1515 (comment) ?

Hi @AlvaroVega, I have done & tested the simplication in https://github.com/telefonicaid/iotagent-node-lib/blob/d1586fb65a4848ba615ad0cf0da7e39afb2fa350/lib/services/ngsi/entities-NGSI-v2.js#L304-L309

to

Screenshot 2024-02-16 174748

but test case httpBindings-test.js was failing.

The issue can be fixed by adding attribute "timestamp": "true" in provisionDeviceProduction.json. In case of iotagent-json it works without any test case failure.

Please confirm if this change is required or you have any other suggestion.

KeshavSoni2511 commented 4 months ago

Could you please include the simplification of blocks referenced in #1515 (comment) ?

Hi @AlvaroVega, I have done & tested the simplication in

https://github.com/telefonicaid/iotagent-node-lib/blob/d1586fb65a4848ba615ad0cf0da7e39afb2fa350/lib/services/ngsi/entities-NGSI-v2.js#L304-L309

to

Screenshot 2024-02-16 174748

but test case httpBindings-test.js was failing.

The issue can be fixed by adding attribute "timestamp": "true" in provisionDeviceProduction.json. In case of iotagent-json it works without any test case failure.

Please confirm if this change is required or you have any other suggestion.

Hi @AlvaroVega, please review my understanding. Thanks

fgalan commented 4 months ago

Hi @AlvaroVega, please review my understanding. Thanks

It seems there is a conflict on CHANGES_NEXT_RELEASE. This has to be solved (it's easy :) before merging.

AlvaroVega commented 4 months ago

Could you please include the simplification of blocks referenced in #1515 (comment) ?

Hi @AlvaroVega, I have done & tested the simplication in https://github.com/telefonicaid/iotagent-node-lib/blob/d1586fb65a4848ba615ad0cf0da7e39afb2fa350/lib/services/ngsi/entities-NGSI-v2.js#L304-L309

to Screenshot 2024-02-16 174748 but test case httpBindings-test.js was failing. The issue can be fixed by adding attribute "timestamp": "true" in provisionDeviceProduction.json. In case of iotagent-json it works without any test case failure. Please confirm if this change is required or you have any other suggestion.

Hi @AlvaroVega, please review my understanding. Thanks

Good news about iotagent-json tests works without any modifications after simplifying checks in iotagent-node-lib. I've see not problem in perform another PR to modify iotagent-ul tests in the way you describe (change timestamp to false in https://github.com/telefonicaid/iotagent-ul/blob/master/test/deviceProvisioning/provisionDeviceProduction.json and complete current PR with the simplifications in related checks

KeshavSoni2511 commented 4 months ago

Good news about iotagent-json tests works without any modifications after simplifying checks in iotagent-node-lib. I've see not problem in perform another PR to modify iotagent-ul tests in the way you describe (change timestamp to false in https://github.com/telefonicaid/iotagent-ul/blob/master/test/deviceProvisioning/provisionDeviceProduction.json and complete current PR with the simplifications in related checks

Hi @AlvaroVega, I have performed iotagent-ul test with "timestamp": "true" in https://github.com/telefonicaid/iotagent-ul/blob/master/test/deviceProvisioning/provisionDeviceProduction.json as global config value contain "timestamp": "true" and test case passed with the proposed change. Please confirm my understanding if I can proceed further and raise another PR on iotagent-ul. Please confirm if you have any other suggestion. Thanks

AlvaroVega commented 4 months ago

Good news about iotagent-json tests works without any modifications after simplifying checks in iotagent-node-lib. I've see not problem in perform another PR to modify iotagent-ul tests in the way you describe (change timestamp to false in https://github.com/telefonicaid/iotagent-ul/blob/master/test/deviceProvisioning/provisionDeviceProduction.json and complete current PR with the simplifications in related checks

Hi @AlvaroVega, I have performed iotagent-ul test with "timestamp": "true" in https://github.com/telefonicaid/iotagent-ul/blob/master/test/deviceProvisioning/provisionDeviceProduction.json as global config value contain "timestamp": "true" and test case passed with the proposed change. Please confirm my understanding if I can proceed further and raise another PR on iotagent-ul. Please confirm if you have any other suggestion. Thanks

Yes, you can proced it with another PR for iotagent-ul

fgalan commented 4 months ago

Thanks for the contribution!