Remove duplicated assert in test

[lukas-vlcek]

Description

This line can be safely removed because the two lines following this assert test exactly the same condition.

Related Issues

n/a

Check List

• [ ] New functionality includes testing.
  • [ ] All tests pass
• [ ] New functionality has been documented.
  • [ ] New functionality has javadoc added
• Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
• Commits are signed per the DCO using --signoff
• [ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
• [ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[lukas-vlcek]

This change does not require a CHANGELOG.md addition (it is a small modification of a test). How can I get rid of the failing check then? Is striking through the CHANGELOG.md line in the "Check List" not enough?

[github-actions]

Compatibility status:

Checks if related components are compatible with change faca1fa

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/sql.git]

[github-actions]

❌ Gradle check result for faca1fa: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[lukas-vlcek]

Created new flaky test issues:

• [BUG] org.opensearch.indices.stats.IndexStatsIT.classMethod is flaky #12619
• [BUG] org.opensearch.ratelimitting.admissioncontrol.AdmissionControlMultiNodeIT.testAdmissionControlRejectionOnDisabled is flaky #12620
• [BUG] org.opensearch.ratelimitting.admissioncontrol.AdmissionControlMultiNodeIT.testAdmissionControlEnforcedOnNonACEnabledActions is flaky #12622

[kotwanikunal]

This change does not require a CHANGELOG.md addition (it is a small modification of a test). How can I get rid of the failing check then? Is striking through the CHANGELOG.md line in the "Check List" not enough?

Added in a label to skip changelog.

[github-actions]

❕ Gradle check result for faca1fa: UNSTABLE

• TEST FAILURES:

      2 org.opensearch.cluster.MinimumClusterManagerNodesIT.testThreeNodesNoClusterManagerBlock

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.37%. Comparing base (b15cb0c) to head (faca1fa).
Report is 13 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #12616      +/-   ##
============================================
- Coverage     71.42%   71.37%   -0.05%     
- Complexity    59978    59997      +19     
============================================
  Files          4985     4985              
  Lines        282275   282337      +62     
  Branches      40946    40952       +6     
============================================
- Hits         201603   201530      -73     
- Misses        63999    64148     +149     
+ Partials      16673    16659      -14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.