search

NRL-Plasma-Physics-Division / block-on-spring

TurboPy app to compute the motion of a block on a spring
0 stars 5 forks source link

Update test_spring.py #7

Closed kphlips closed 4 years ago

kphlips commented 4 years ago

Addresses #2

kphlips commented 4 years ago

I moved the test output files to compare to into a new folder called output, but git won't let me add them. On my personal machine with these files, the test passes.

arichar6 commented 4 years ago

Also, the folder name output is blocked because it is in .gitignore. If you use a different name it should let you add them.

kphlips commented 4 years ago

@arichar6 Sorry, only now seeing the requested changes... I moved some files around and created new reference csv files, as the ones from the main turbopy repo were not for the same test configurations. Also, I'll go ahead and get rid of the .DS_store files that got pushed with the commit.

arichar6 commented 4 years ago

This looks good, and is pretty much ready to go.

Can you convert these tests into fixtures? I'll open a separate issue for that...