-
Notifications
You must be signed in to change notification settings - Fork 671
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed "${TEST_INDEX} is initialized to 'null' for video recording path pattern" (close #3455) #3481
Conversation
❌ Tests for the commit 8171385 have failed. See details: |
❌ Tests for the commit 1f55b48 have failed. See details: |
✅ Tests for the commit 1f55b48 have passed. See details: |
src/notifications/warning-message.js
Outdated
|
||
problematicPathPatternPlaceholderForVideoRecording: '{placeholderList} path pattern placeholder{suffix} {verb} not suitable for the video recording\'s "pathPattern" option.\n' + | ||
'\n' + | ||
'{pronoun} value{suffix} will be replaced with an empty string.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dirk's version:
problematicPathPatternPlaceholderForVideoRecording: 'The {placeholderList} path pattern placeholder{suffix} cannot be applied to the recorded video.\n' +
'\n' +
'The placeholder{suffix} {verb} replaced with an empty string.'
where {suffix}
is s
and {verb}
is either was
or were
.
❌ Tests for the commit 53fd567 have failed. See details: |
It was the Past tense - |
❌ Tests for the commit fb412ab have failed. See details: |
@testcafe-build-bot retest |
❌ Tests for the commit fb412ab have failed. See details: |
✅ Tests for the commit fb412ab have passed. See details: |
@VasilyStrelyaev FPR |
…h pattern" (close DevExpress#3455) (DevExpress#3481) * initial commit * functional test * unit tests * updated the test name * fix test * fix test * Update warning-message.js * Update warning template * fixed the message and restructured the string utils
@AndreyBelym @AlexKamaev @VasilyStrelyaev