refactor: remove obsolete clippy allows in non_finalized_state

[bishopcheckmate]

Motivation

I was reading the codebase and found these two #[allow()]s obsolete

PR Author Checklist

Check before marking the PR as ready for review:

• Will the PR name make sense to users?
• Does the PR have a priority label?
• Have you added or updated tests?
• Is the documentation up to date?

For significant changes:

• Is there a summary in the CHANGELOG?
• Can these changes be split into multiple PRs?

If a checkbox isn't relevant to the PR, mark it as done.

Solution

Just remove the obsolete allows

Testing

cargo +nightly clippy -p zebra-state

Reviewer Checklist

Check before approving the PR:

• Does the PR scope match the ticket?
• Are there enough tests to make sure it works? Do the tests cover the PR motivation?
• Are all the PR blockers dealt with?
  PR blockers can be dealt with in new tickets or PRs.

And check the PR Author checklist is complete.

[teor2345]

Thank you for this cleanup!

Please let us know if you'd like more suggestions for what to work on.

[bishopcheckmate]

@teor2345 I'd appreciate some issue to get started. Do you have anything in mind?

[teor2345]

@teor2345 I'd appreciate some issue to get started. Do you have anything in mind?

You could try something from this list of tickets:
https://github.com/ZcashFoundation/zebra/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+label%3A%22good+first+issue%22

Or you could ask the developers in the #zebra channel in the Zcash R&D discord:
https://discord.com/channels/809218587167293450/809251029673312267

Here's a link that will let you join the Discord for the next week:
https://discord.gg/GvhwHwXR