MLE-25554 Excluding fragile tests on PR builds

[rjrudin]

Also removed the second mlWaitTillReady call based on updates to ml-gradle

[github-actions]

Copyright Validation Results
Total: 1 | Passed: 0 | Failed: 0 | Skipped: 1 | at: 2025-11-25 19:15:05 UTC | commit: 91d26d3

⏭️ Skipped (Excluded) Files

• Jenkinsfile

✅ All files have valid copyright headers!

[Copilot]

Pull request overview

This PR excludes fragile tests from PR builds and simplifies the deployment process by removing a redundant mlWaitTillReady call. The changes target improved build stability by temporarily skipping problematic document data movement tests in PR builds while still running them in other build contexts.

• Excludes fragile tests (documents-data-movement-*) from PR builds
• Removes redundant mlWaitTillReady call and sleep
• Converts shell script from single quotes to double quotes for variable interpolation

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The excluded test pattern 'documents-data-movement-*' is hardcoded. Consider extracting this to a pipeline parameter or configuration variable to make it easier to maintain and modify the list of excluded tests without code changes.

Suggested change

	def runTests(excludeFragileTests = false) {
	def mochaGlob = excludeFragileTests ?
	'test-basic/!(documents-data-movement-*).js' :
	def runTests(excludeFragileTests = false, excludedTestPattern = 'documents-data-movement-*') {
	def mochaGlob = excludeFragileTests ?
	"test-basic/!(${excludedTestPattern}).js" :