IBX-10894: Moved throw error when invitation not found in database to gateway

[mateuszdebinski]

🎫 Issue	IBX-10894

Description:

For QA:

Documentation:

[bnowak]

It's good you added coverage for InvitationService, however it's not the direct gateway layer you touched 😅

I'd recommend adding separate test coverage for gateway too 😉

[mateuszdebinski]

@bnowak I thought about it, and I had tests for Gateway ready, but I'm wondering whether it wouldn't be better to add tests for InvitationService, which are missing, and at the same time also test the gateway - in fact, the tests for the gateway will be a copy of this one

[bnowak]

@bnowak I thought about it, and I had tests for Gateway ready, but I'm wondering whether it wouldn't be better to add tests for InvitationService, which are missing, and at the same time also test the gateway - in fact, the tests for the gateway will be a copy of this one

It's always better to test each layer separately.

in fact, the tests for the gateway will be a copy of this one

That's kind of true, but on the gateway layer we don't have mapping to Invitation value object, which we have in the service (we're testing that part too here). Sometimes, raw db values can also don't reflect VO fields in 1:1 which can be expected and tested too.

anyway, any tests are better than none :)

[mateuszdebinski]

@bnowak I added DoctrineGatewayTest

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud