Skip to content

FIX add PR #1721 test to test effect on master#1726

Closed
fgalan wants to merge 4 commits intomasterfrom
fix/attribute-loss-when-updating-a-device-test-on-master
Closed

FIX add PR #1721 test to test effect on master#1726
fgalan wants to merge 4 commits intomasterfrom
fix/attribute-loss-when-updating-a-device-test-on-master

Conversation

@fgalan
Copy link
Copy Markdown
Member

@fgalan fgalan commented Aug 1, 2025

Temporal PR, just for testing.

See #1721 (comment)

Comment thread test/functional/testCases.js
Comment thread test/functional/testCases.js Outdated
Comment thread test/functional/testCases.js Outdated
fgalan and others added 2 commits October 29, 2025 13:50
Co-authored-by: Edwin Vautier <edwin.vautier@gmail.com>
@fgalan
Copy link
Copy Markdown
Member Author

fgalan commented Feb 4, 2026

At the present point in time I do confirm the test in this PR is the same in PR #1721

@fgalan
Copy link
Copy Markdown
Member Author

fgalan commented Feb 4, 2026

The error in this PR is as expected:

1) FUNCTIONAL TESTS AUTO
       0021c - Simple group with active attributes with special names in object_id
         A - WHEN sending defined object_ids with special format names in attributes through http IT should send measures to Context Broker preserving value types, name mappings and metadatas:

      Assertion Error - Context Broker received payload differs from expectation
      + expected - actual

         "mainDoorEvent": {
           "type": "Boolean"
           "value": 1
         }
      +  "psSMDUHPower14": {
      +    "type": "Text"
      +    "value": 2
      +  }
         "psSMDUHPower17": {
           "type": "Number"
           "value": 3
         }
      
      at Object.testCase (test/functional/testUtils.js:264:74)
      at process.processTicksAndRejections (node:internal/process/task_queues:103:5)
      at async Context.<anonymous> (test/functional/functional-tests-runner.js:116:25)

@fgalan
Copy link
Copy Markdown
Member Author

fgalan commented Feb 4, 2026

After merging the fix in PR #1721 and sync this PR (commit bf93ac8) the tests are now passing, as expected.

This PR as fulfilled its purposed and can be now closed.

@fgalan fgalan closed this Feb 4, 2026
@fgalan fgalan deleted the fix/attribute-loss-when-updating-a-device-test-on-master branch February 4, 2026 09:03
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