RffWorkout test is not stable and randomy fails during the build.

[wschin]

As title. I failed multiple times for some unknown reasons. It needs to be improved for a more smooth merging process.

[singlis]

Thanks @wschin, I will take a look.

[singlis]

@wschin Is there a specific platform this is failing on?

[singlis]

Pasting failures for reference:
RffStatic Test failure on Windows: https://dnceng.visualstudio.com/public/_build/results?buildId=52992
RffWorkout Test on Mac debug: https://dnceng.visualstudio.com/public/_build/results?buildId=52926
RffWorkout Test on Windows: https://dnceng.visualstudio.com/public/_build/results?buildId=52760

[Ivanidzo4ka]

I download RffStatic test output file and it looks like we have error in 7th digit or so. So easy solution is to just change precision.
For RffWorkout, it's kinda mystery. It fails in Done method which shouldn't happen. Done checks certain variable which we set to true during initialization, and never changed in test.

[Zruty0]

I believe that we actually set the 'passed' field during CheckSameSchema and CheckSameValues, which is consistent with the assumption that RFF is not deterministic.

[singlis]

Please see this PR:
#1829

It changes the precision to 1e-3 from 1e-6.

[singlis]

@Zruty0 - I will check the passed field as well.

[singlis]

From the thread on #1829, the intermittent failures were due to change #1657, which was reverted in #1838. I have also not seen it repro in the build since the revert, so marking this as closed.