feat: add query.OUTPUTS to collect outputs status

[xtimmy86x]

Related Issues

Proposed Changes:

Add query.OUTPUTS to collect the status of the outputs in control panel

Testing:

Extra Notes (optional):

Checklist

• Related issues and proposed changes are filled
• Tests are defining the correct and expected behavior
• Code is well-documented via docstrings

[palazzem]

The change is good, but we need to add tests for it. Actually, we don't have tests that are verifying that the method actually returns outputs as it should. You need to add tests and fixtures like these:

• https://github.com/palazzem/econnect-python/blob/main/tests/test_client.py#L1326-L1423
• https://github.com/palazzem/econnect-python/blob/main/tests/fixtures/responses.py#L193-L242
• https://github.com/palazzem/econnect-python/blob/main/tests/conftest.py#L31

In general, for any change, you should:

1. Update current tests so that they don't fail. In doing this, you must be sure you are not changing the behavior as if you are adding something, the rest of the software should work as before.
2. Always add new tests. You must not reuse current tests to check new behavior as they were designed to test another behavior. In your case you added what happens if the outputs key is missing (that is good), but it's not enough to be sure it works as expected.
3. Add a fixture where you actually put a static response to avoid real tests (responses.py)
4. Update the conftest.py where you actually add the response you generated in (3) to the call you expect in client fixture.

After these changes, I think it should be all set! In the meantime, thank you very much for the help!

[coveralls]

Pull Request Test Coverage Report for Build 6810659178

• 10 of 10 (100.0%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 6627095528:	0.0%
Covered Lines:	358
Relevant Lines:	358

---

💛 - Coveralls

[palazzem]

Great addition! Looks great with the added tests!