search

jacereda / fsatrace

Filesystem access tracer
ISC License
81 stars 12 forks source link

Remove the test FIXME's #32

Closed ndmitchell closed 5 years ago

ndmitchell commented 5 years ago

Turns on the entire test suite. Requires #31 merging to make it pass on Windows.

ndmitchell commented 5 years ago

I rebased the patch. I was expecting that to pass the CI, but both failed, for reasons I can't immediately see. I'd suggest rerunning both to see what is going on and if its consistent or not.

jacereda commented 5 years ago

My fault, I removed a required memset(). And after fixing it I reproduced the behaviour you mentioned (nesting the injection). I'll try to figure out what's happening.

jacereda commented 5 years ago

I've merged manually, it should be in master now. I'll try to figure out how to catch the nested injection error.