aviaviavi / curl-runnings

A declarative test framework for quickly and easily writing integration tests against JSON APIs.
MIT License
158 stars 13 forks source link

Option to write test results to a file as JSON #55

Closed aviaviavi closed 4 years ago

aviaviavi commented 4 years ago

closes #53

aviaviavi commented 4 years ago

@Rashadse let me know what you think

ghost commented 4 years ago

nice! for some reason the time elapsed for each case says wrong values like 18 or 15 and so on, it should be 0.1 and 0.2 etc like you did in the time feature. is this a bug?

aviaviavi commented 4 years ago

Oh good catch. Internally, elapsed time is tracked in milliseconds, I forgot to have the conversion in there. Will change

aviaviavi commented 4 years ago

cc @Rashadse

ghost commented 4 years ago

Hello again, the latest fix shows the correct number but in the wrong form, we are getting 1.0e-2 instead of the simplified form

aviaviavi commented 4 years ago

Interestingly, I actually can't seem to reproduce this. Is this happening reliably? What system are you on?

ghost commented 4 years ago

I am on ubuntu

aviaviavi commented 4 years ago

Still can't reproduce, but try again and let me know if that helped

ghost commented 4 years ago

what we are getting is the scientific notation of the time "elapsedTimeSeconds": 4.6e-2 this is actually fine for us i think we got what we wanted you can go ahead and merge the commit

aviaviavi commented 4 years ago

Weird, I tried this out on ubuntu and I still can't seem to get it to spit out scientific notation. If this works for you I'll merge for now. We can circle back to it if needed.