Yes. I realized this after porting the test. It's a shame there are no test data directly for the SignatureHash function in bitcoin core tests. I may try to add said tests to bitcoin core, but I'm afraid my C++ (lack of) knowledge may be a problem. It's an interesting challenge though, and I've always wanted to contribute to the satoshi client.