Skip to content

Added unit tests for pin details.#232

Open
nithyaganeshng wants to merge 4 commits into
masterfrom
niganesh/add-tests-for-pindetails
Open

Added unit tests for pin details.#232
nithyaganeshng wants to merge 4 commits into
masterfrom
niganesh/add-tests-for-pindetails

Conversation

@nithyaganeshng

@nithyaganeshng nithyaganeshng commented Mar 17, 2023

Copy link
Copy Markdown
Contributor

Add unit tests for serialization of pin details.
Removed unused legacy test dependencies.
Fixed a failing test.

Solution:
Used library A to complete scenario X with quality and style.

Validation:
Please include here how it was verified that the PR works as intended.

Type of change:

  • Feature work
  • Bug fix
  • Documentation
  • Engineering change
  • Test
  • Logging/Telemetry

Risk:

  • High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
  • Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
  • Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Work Item links:
Please include here links for this work item, or deferred work, or related work. E.g. if the refactoring is too big to fit in this PR, or the localized strings need to be updated later, please link the TODO work items here.

Documentation Links:
Please include here links to any related background documentation for this PR.

@nithyaganeshng nithyaganeshng requested review from a team, dangodb and symorton as code owners March 17, 2023 18:37
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.

1 participant