apollo_l1_provider: make sure wrong event types do not get scraped

[guy-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[guy-starkware]

• papyrus_base_layer: delete l1_endpoint_monitor crate #10736
• papyrus_base_layer: delete monitored base layer #10731
• apollo_l1_provider: re-write FinalityTooHigh error as LatestBlockNumberTooLow #10729
• apollo_l1_provider: use cyclic base layer wrapper instead of monitored base layer #10703
• apollo_l1_provider: make sure wrong event types do not get scraped #10700 👈 (View in Graphite)
• papyrus_base_layer: gate integration tests #10671
• apollo_l1_provider: add test for each type of L1 event we filter on #10595
• papyrus_base_layer: test util for creating an ethereum base layer using a mock provider #10588
• papyrus_base_layer: extract check and cycle URLs into a helper function with logging #10618
• papyrus_base_layer: add CyclicBaseLayerWrapper #10412
• papyrus_base_layer: make BaseLayer trait functions mutable #10404
• papyrus_base_layer: add the option to cycle the URL directly into the base layer #10357
• papyrus_base_layer: add ordered list of provider URLs to the config #10354
• apollo_l1_provider: remove finality_too_high helper function, add MissingBlock error #10348
• apollo_l1_gas_price: add test for low latest L1 block number #10346
• apollo_l1_provider: add test for low latest L1 block number #10345
• apollo_l1_provider: replace mutex with atomic u64 in a scraper test #10344
• papyrus_base_layer: remove finality from base layer #10303
• papyrus_base_layer: remove latest_l1_block from base layer #10300
• apollo_central_sync: remove base layer call to latest_proved_block #10246
• papyrus_base_layer: add test for a base layer without a starknet contract #10242
• main-v0.14.1

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

[ShahakShama]

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

---

crates/apollo_l1_provider/tests/flow_test_event_filters.rs line 159 at r1 (raw file):

    let result = base_layer.events(0..=1, event_identifiers_to_track()).await;

    assert!(result.is_err(), "expected error");

Could you also check that the error type is correct

[guy-starkware]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)

---

crates/apollo_l1_provider/tests/flow_test_event_filters.rs line 159 at r1 (raw file):

Previously, ShahakShama wrote…

Could you also check that the error type is correct

Done.

[graphite-app]

Merge activity

• Dec 16, 6:03 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.
• Dec 16, 6:03 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

[guy-starkware]

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