[TST-18] added refs to jira TST board #788
Conversation
Codecov Report
@@ Coverage Diff @@
## master #788 +/- ##
=======================================
Coverage 65.53% 65.53%
=======================================
Files 161 161
Lines 3128 3128
Branches 49 49
=======================================
Hits 2050 2050
Misses 1078 1078Continue to review full report at Codecov.
|
| @@ -84,3 +84,4 @@ | |||
| </tbody></table> | |||
| </body> | |||
| </html> | |||
| <!--covers test TST-18--> | |||
There was a problem hiding this comment.
Would make more sense to me to have this closer to the start of the file. Is there a reason to drop it at the end?
Also, do you think it would make sense to add this as a meta-tag?
<meta name="ats:test-case" content="Vehicle whatever test">
<meta name="ats:test-jira-ticket" content="TST-18">
<meta name="ats:test-jira-url" content="https://advancedtelematic...">
Or is your HTML somehow auto-generated?
There was a problem hiding this comment.
I think it's auto generated. The way I have it now basically adds a comment in the script that appears in the Selenium IDE UI. I'll see how it handles meta tags though, and if it doesn't break, go with that.
There was a problem hiding this comment.
I also put it at the end because there would be another couple of tickets that this test case covers, and having them stacked at the end looked 'right' to me. So no valid reason really :D
There was a problem hiding this comment.
Per slack, let's extend the name="" attribute to include the test ID. That will allow us to support multiple tickets in the same file.
EpicNuts
force-pushed
the
feat/TST-18/return-Device-List
branch
from
b6f961f to
cdd7ea2
Compare
EpicNuts
force-pushed
the
feat/TST-18/return-Device-List
branch
from
cdd7ea2 to
e85e20c
Compare
EpicNuts
changed the title
|
bombs away! :D |
TST-18
TST-18
added reference to Jira