moana

#### Summary It doesn't make sense that the component_mul_generator takes the generator as an input. To accommodate two different generators `G` and `G_prime` we should have two distinct functions. Also...

**Describe what you want implemented** Add tests that test malicious usage of `component_mul_generator` **Describe "Why" this is needed** To fix eventual vulnerabilities. **Describe alternatives you've considered** N/A **Additional context** part...

#### Summary Change `finalize` in both `Hash` and `HashGadget` to return the hash directly. We can do this since we start and finish the `Safe` instance in the same method...

#### Summary Add a test where the opening is tampered with. The leaf and root should be fine but the opening should be tempered with.

Resolves #2069

#### Summary The internal `Payload` in both moonlight and `phoenix` transaction can be considered an implementation detail that doesn't need to leak to the user.

#### Summary Limit code duplication in the transfer-contract tests by isolating the creation of a transaction in a test-helper function.