[fix][test] fix testBatchMetadataStoreMetrics.

[zhaizhibo]

Motivation

In PR #25187, the pulsar_batch_metadata_store_executor_queue_size metric was removed from BatchMetadataStoreStats. Therefore, related tests should also be updated to remove references to this metric.

Modifications

Removed the validation of the pulsar_batch_metadata_store_executor_queue_size metric in the testBatchMetadataStoreMetrics test.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository:

[lhotari]

LGTM, thanks for fixing!

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.66%. Comparing base (db91b93) to head (406786c).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #25241       +/-   ##
=============================================
+ Coverage     30.11%   72.66%   +42.55%     
- Complexity      109    34031    +33922     
=============================================
  Files          1899     1959       +60     
  Lines        151150   155406     +4256     
  Branches      17230    17726      +496     
=============================================
+ Hits          45513   112933    +67420     
+ Misses        98639    33473    -65166     
- Partials       6998     9000     +2002     

Flag	Coverage Δ
inttests	25.75% <ø> (-0.03%)	⬇️
systests	22.37% <ø> (-0.05%)	⬇️
unittests	73.64% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1502 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[BewareMyPower]

I'm wondering why CI passed?

[BewareMyPower]

I see it's in the flaky test group, so it's ignored