LogicAppsUX
LogicAppsUX copied to clipboard
Feat(Data Mapper): Refactoring deserialization for v2
- 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:
Code Coverage Report
| 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%