Change log level of methods name logged in Scaffolding

[Vinnstah]

Fixes #1702

Change the log level to trace for method names logged by UniFFI internals when FFI calls into Rust.

Augmented existing logging tests in Python, asserting that the changed level works as intended.

Let me know if you want me to put the tests in a new fixture instead of recycling an existing one.

[mhammond]

Hi, is there any interest in removing some of the test code here? If I don't hear back soon, I'll put up the 2-line PR that actually fixes this and close this, but I'd like to offer the opportunity to have it completed here.

[Vinnstah]

Hi Mark, Apologies for the delay. I'll fix the tests and push a new commit asap.

[praveenperera]

thank you for this! @Vinnstah !

[bendk]

I didn't realize this issue has been sitting in the queue for so long. Does https://github.com/mozilla/uniffi-rs/pull/2211 handle your needs or is there still work to be done here?

[praveenperera]

@bendk no my problem is that each time an FFI func is called there's a debug log, that log should be trace level instead, because if not it makes debug logs very noisy.

I've worked around it by putting all the noisy funcs in mod ffi {} and silencing all logs from *ffi* mods, but that's hacky and messy. If this PR is merged in I could get rid of all of those.

[bendk]

I should have pointed you to https://github.com/mozilla/uniffi-rs/pull/2275. As far as I know, there are no more log calls in the current code. I don't think we even have a dependency on the crate. Those calls were replaced by an internal logging mechanism, that's disabled by default.

[praveenperera]

@bendk I'm using master now and its working, no more noisy logs thank you!

Was able to get rid of all my mod ffi: https://github.com/bitcoinppl/cove/commit/ce6cd72120a4d3490a5d69d6deb34527c2f2f996

[mhammond]

We've recently removed all log calls from the generated scaffolding.