Create unit tests for Calculator plugin

[P-Storm]

Summary of the Pull Request

Currently the calculator plugin for PowerToys Run had no unit test (see #5948). With this in place we can test the calculation engine separate. Currently I have also included the problems of #3488 and #3697

PR Checklist

• Applies to Create unit tests for Calculator plugin #5948
• CLA signed. If not, go over here and sign the CLA
• Tests added/passed
• Requires documentation to be updated
• I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place:

Info on Pull Request

I have made a start with the unit test. I had to change to structure of the parsing to a separate class. The culture is needed to mimmick the current output and for the unittest to be system agnostic.

Note:
Unit test ParseValuesKnownIssues has known bug unit tested. These are the limitations of teh

• big decimals are parsed wrong Mage plugin fails with long decimals #3488
• (multiplying without operator [Run, tracker] calculator (mage nuget) having calculation issues #3697
  I did search for some test that were failing before the latest fix and put them into ParseValues.

Validation Steps Performed

Startup the unit test project Microsoft.Plugin.Calculator.UnitTest in src\modules\launcher\Plugins\Microsoft.Plugin.Calculator.UnitTest\Microsoft.Plugin.Calculator.UnitTest.csproj

Please note that this is the first time contributing for PowerToys. If there is something missing, please let me know 👍

[P-Storm]

Labels needed to cover this :

• Area-Tests
• Product-Launcher
• Run-Plugin

It can be reviewed by the core team

[ryanbodrug-microsoft]

@P-Storm On first glance this looks fantastic! Thanks for this PR 👍 . I'll take a deeper look this morning

[ryanbodrug-microsoft]

This is a terrific PR, much appreciated. Other than the one nit comment. The only thing I'd ask is that you add the Microsoft.Plugin.Caclulator.UnitTest project to run in CI by adding it to the .net core tests specified here. This will allow us to validate that these tests run for everyone and prevent future breaking changes.

PowerToys/.pipelines/ci/templates/build-powertoys-steps.yml

Line 83 in 936b028

testAssemblyVer2: |

[P-Storm]

I guess all the feedback is processed. If there is some questions about the new class BracketHelper then I'm glad to clarify or to refine it.

[ryanbodrug-microsoft]

LGTM! Thanks for the Robustness on the BracketHelper class 👍 Also I can see that the tests are passing in CI now 💯