Remove unused wallet context args

[ekzyis]

Description

We were passing models and me into the wallet test functions. I think they were used in the past for logging but we no longer need them.

I also removed logger from the NWC functions since they don't use it.

Additional Context

Maybe I am overdoing it with small PRs but it's nice to get small changes out of the way so I can focus on the relevant stuff in #1558.

Checklist

Are your changes backwards compatible? Please answer below:

yes

On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:

5. I looked at all testSendPayment and testCreateInvoice function signatures and saw none of them use these arguments.

Regarding removal of logger: removing an unused argument doesn't break any function calls in JS since you can always call a function with as many arguments as you want, any arguments that aren't defined as parameters are simply not used.

For frontend changes: Tested on mobile, light and dark mode? Please answer below:

n/a

Did you introduce any new environment variables? If so, call them out explicitly here:

no

[huumn]

Maybe I am overdoing it with small PRs but it's nice to get small changes out of the way so I can focus on the relevant stuff in #1558.

You aren't. It's easier to review one "subject" at a time.