Use Psr\Http\Message\StreamFactoryInterface not Nyholm\Psr7\Stream

[BrianHenryIE]

Fix #70.

Added some tests too.

[codecov]

Codecov Report

Patch coverage: 50.00% and project coverage change: +26.53% 🎉

Comparison is base (72de5b4) 0.00% compared to head (33fac93) 26.53%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@              Coverage Diff              @@
##             master      #72       +/-   ##
=============================================
+ Coverage      0.00%   26.53%   +26.53%     
=============================================
  Files             4        4               
  Lines            49       49               
=============================================
+ Hits              0       13       +13     
+ Misses           49       36       -13     

Files Changed	Coverage Δ
src/HttpJsonRpcRequestFactory.php	56.52% <50.00%> (+56.52%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[simPod]

Can you explain why do you prefer such change? I'm used to having NS structured as SimPod\JsonRpc\Tests\

[BrianHenryIE]

The guy who wrote PHPUnit does not add \Tests\ to the namespace.

https://github.com/sebastianbergmann/uuid/blob/main/tests/UuidTest.php

Sorry I can’t find (on mobile) a more definitive statement from him.

I don’t see any advantage to adding the \Tests\ part of the namespace. It complicates things by needing imports that are otherwise inherent. I think the fact it’s autoload-dev is sufficient distinction.