Enable test for missing ~A section data.

JustinGOSSES / wellio.js

JavaScript for converting well-log standard .las file format to json format

https://justingosses.github.io/wellio.js/

MIT License

34 stars 5 forks source link

Enable test for missing ~A section data. #27

Closed dcslagel closed 4 years ago

dcslagel commented 4 years ago

@JustinGOSSES,

The purpose of this test is to validate that wellio.js doesn't crash or panic when the ~A section has no data. It fixes issue #22 .

The test code was fixed to check the status of the internal data representation for the CURVES (~A) section. No changes were needed to the main wellio.js code.

Let me know if this needs some additional changes before merging.

Thanks,

JustinGOSSES commented 4 years ago

Test looks good! I have some thoughts on maybe giving non-blocking error messages in console for missing sections like this but i'll use another issue for it and merge this.