Fix issues with leaking gdb processes and add to unit test

lanl / BEE

Other

13 stars 3 forks source link

Fix issues with leaking gdb processes and add to unit test #746

Closed aquan9 closed 7 months ago

aquan9 commented 7 months ago

When Jake and I were looking at the old PR it looked like there was some weirdness between the state that the PR was in, and the state that the branch was in, so I stashed my changes and moved everything over to a new branch and PR. Lets see if this passes unit tests....

aquan9 commented 7 months ago

I'm adding this comment, just to link PR #738 Since that was the original PR that were going to close in favor of this one.

pagrubel commented 7 months ago

This looks good to me and I played around with several scenarios, no leftover neo4j instances.