fix(market-pallet): deal settling

[cernicc]

Description

I wanted to test if it works. That is why it took some time to figure out which input params to use on the extrinnsics :D A deal should not be removed from the Proposals if the deal is still active. I've also added some logs.

Important points for reviewers

Before merging this please merge #143

Checklist

• Are there important points that reviewers should know?
• Make sure that you described what this change does.
• Have you tested this solution?

[th7nder]

I'm not sure whether we need that many logging in production code, but I agree with the fix. Should we add a unit test to check for not removing a deal from proposals as well?

[cernicc]

I've added two tests for the on_finalize hook. Most of the logs are on the trace level. I think that trace level should not be enabled by default.