apollo_batcher: reorder async_trait and automock annotation

[itamar-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[itamar-starkware]

• apollo_batcher: reorder async_trait and automock annotation #10438 👈 (View in Graphite)
• main-v0.14.1

This stack of pull requests is managed by Graphite. Learn more about stacking.

[TzahiTaub]

@TzahiTaub reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @itamar-starkware)

---

crates/apollo_batcher/src/batcher_test.rs line 146 at r1 (raw file):

            let (non_working_pre_confirmed_tx_sender, _) = tokio::sync::mpsc::channel(1);
            let mut mock_writer = Box::new(MockPreconfirmedBlockWriterTrait::new());
            mock_writer.expect_run().returning(|| Ok(()));

Why was this changed?

Suggestion:

.return_once(||

[itamar-starkware]

crates/apollo_batcher/src/batcher_test.rs line 146 at r1 (raw file):

Previously, TzahiTaub (Tzahi) wrote…

Why was this changed?

Not a good reason. return once is more accurate.
Fixed.

[TzahiTaub]

@TzahiTaub reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @itamar-starkware)