Copy the steps and execution notes for failed tc#307
Closed
delcroip wants to merge 4 commits intoTestLinkOpenSourceTRMS:testlink_1_9_20_fixedfrom
Closed
Copy the steps and execution notes for failed tc#307delcroip wants to merge 4 commits intoTestLinkOpenSourceTRMS:testlink_1_9_20_fixedfrom
delcroip wants to merge 4 commits intoTestLinkOpenSourceTRMS:testlink_1_9_20_fixedfrom
Conversation
if the test is failed, if the step is not passed or there is an execution note then this is added to the "note/description" Step <step number>(<test status>): <step execution notes>
Contributor
|
Thanks, important thing IMHO this need to be managed by a configuration option. regards |
Contributor
|
Please also use a constant to express the FAILED value not the 'f', this will be improve code understanding without need of comments thanks |
Author
|
I'll check that |
Author
|
@fmancardi with the new gui coming is there a preferred way to send the Param to the front end or having a checkbox (prechecked based on config) will be enough ? |
Contributor
|
We can start with a system wide config i.e. for all test projects, and as
you suggested set the value ( a yes/no select will be better than a
checkbox)
That user can change each time.
Another option will be to create a custom field that you can use at
testplan level, if it's present this will mean default value will be new
behaviour, is doen't old one.
Give a try to these alternatives.
Regards
Il Lun 29 Mar 2021, 19:07 Patrick Delcroix ***@***.***> ha
scritto:
… @fmancardi <https://github.com/fmancardi> with the new gui coming is
there a preferred way to send the Param to the front end or having a
checkbox (prechecked based on config) will be enough ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#307 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABZRMKDCYXOFTJFFXNIID3TGC6VRANCNFSM4ZY7FYPQ>
.
|
Author
|
replaced by #320 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
if the test is failed,
if the step is not passed or there is an execution note then this is added to the "note/description"
Step {step number}({test status}): {step execution notes}