Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add unit tests #2

Closed
mdpiper opened this Issue Aug 20, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@mdpiper
Copy link
Contributor

mdpiper commented Aug 20, 2018

Unit tests, written by the developer, are extremely helpful for ensuring that software behaves as a developer intends. (I've caught so many defects through unit tests.) Unit tests should be added to this package.

@mdpiper

This comment has been minimized.

Copy link
Contributor Author

mdpiper commented Aug 20, 2018

A suggestion: mgunit is a mature and widely used unit testing framework for IDL.

@danehkar

This comment has been minimized.

Copy link
Contributor

danehkar commented Feb 20, 2019

@mdpiper I added mgunit in the folder test_mgunit for automated tests.

To test, you can download AtomNeb-idl with its dependencies as follows

git clone --recursive https://github.com/atomneb/AtomNeb-idl.git

You need to unpack rc_o_iii_SSB17.fits.tar.gz in atomic-data-rc:

tar -xvf rc_o_iii_SSB17.fits.tar.gz

To run the test, you need to run the following command:

idl test_all.pro

There should not be any failure in the test-results.html file.

@danehkar danehkar closed this Feb 27, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.