fix: return correct prune_target_block when syncing

[klkvr]

Right now we are likely to fail here when starting e.g Holesky full node and trying to insert genesis block:

reth/crates/storage/provider/src/providers/database/provider.rs

Line 1832 in f17dc01

let contract_log_pruner = self.prune_modes.receipts_log_filter.group_by_block(tip, None)?;

Reason is that this match would return Err for Self::Before(0), tip == 0

reth/crates/prune/types/src/mode.rs

Lines 49 to 54 in f17dc01

	Self::Before(n) if *n == tip + 1 && purpose.is_static_file() => Some((tip, *self)),
	Self::Before(n) if *n > tip => None, // Nothing to prune yet
	Self::Before(n) if tip - n >= segment.min_blocks(purpose) => {
	Some(((*n).saturating_sub(1), *self))
	}
	_ => return Err(PruneSegmentError::Configuration(segment)),

Instead, we should return None if tip - n < segment.min_blocks(purpose) because this might become false once node syncs to a higher tip

[mattsse]

ah, I see, makes sense

failing test needs a closer look from @joshieDo ?

[klkvr]

I think the test was incorrect but anyway would appreciate sanity review here @joshieDo