LogicAppsUX icon indicating copy to clipboard operation
LogicAppsUX copied to clipboard

Feat(Data Mapper): Refactoring deserialization for v2

Open DanielleCogs opened this issue 1 year ago • 1 comments

  • Please check if the PR fulfills these requirements
  • [x] The commit message follows our guidelines
  • [x] Tests for the changes have been added (for bug fixes/features)
  • [ ] Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) refactor

  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)? deserialization is easier to understand, and should have less bugs! Not completely done yet, so tests that are not passing are disabled

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Please Include Screenshots or Videos of the intended change:

image

image

DanielleCogs avatar May 07 '24 19:05 DanielleCogs

Code Coverage Report

Code Coverage

Package Base Coverage New Coverage Difference
mapHandling 🔴 9.09% 🟢 83.54% 🟢 74.45%
utils 🔴 45.86% 🟠 51.37% 🟢 5.51%
Overall Coverage 🟢 20.98% 🟢 26.05% 🟢 5.07%

Minimum allowed coverage is 0%, this run produced 26.05%

github-actions[bot] avatar May 21 '24 01:05 github-actions[bot]