Skip to content
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

WEBPA SET parameter request with empty string value #67

Merged
merged 2 commits into from
Jul 12, 2024

Conversation

Sreedevi29
Copy link

DESCRIPTION: 1. To Allow WEBPA set request with empty string value.
2. The wdmp-c code is handled to allow the empty value for all data types. WEBPA SET query will be success only for empty string param value and failed for other data types.
3. Added the Unit test for the set empty param value.

TEST DONE : Verified WEBPA SET parameter request is succeeded for empty string param values and failed for other data types.

@CLAassistant
Copy link

CLAassistant commented Jul 9, 2024

CLA assistant check
All committers have signed the CLA.

DESCRIPTION: 1. To Allow WEBPA set request with empty string value.

             2. Currently the wdmp code is handled to allow the empty value for all data types. But WEBPA SET query will be success only for empty string param value and failed for other data types.

             3. Added the Unit test for the set empty param value.

TEST DONE :  1. Verfied WEBPA SET parameter request is succeeded for empty string param
values and failed for other data types.

Signed-off-by: sreedevi v <SREEDEVI.V@t-systems.com>
@schmidtw
Copy link
Member

The PR looks good to me - I'll try to fix the sonarcloud issue.

@schmidtw schmidtw merged commit e93bd45 into xmidt-org:master Jul 12, 2024
2 checks passed
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.

4 participants