Fixes #171.

[halfninja]

Before we reassign a number variable to a string, check it is still actually a number

[clayreimann]

@halfninja Can you share possible values for what testcase.attr.time would actually be in this case? Also, please add a regression test to ensure we don't break this again in the future.

[halfninja]

@halfninja Can you share possible values for what testcase.attr.time would actually be in this case? Also, please add a regression test to ensure we don't break this again in the future.

It's either a number like 6.54321 or a fixed-decimal string like "6.543". I'll have another go at a test.

[halfninja]

I've added a test case that was failing without the fix. As I don't know exactly how Cypress gets into this situation it directly calls getXml with some test data crafted to hit the error.

[BenFranzi]

Can you please publish this change to npm

[clayreimann]

@BenFranzi now on npm