Proposal "Test Reports Folder structure"#2
Conversation
GetmanetsIrina
requested review from
AByzhynar,
AKalinich-Luxoft,
LitvinenkoIra,
aderiabin,
dboltovskyi and
dev-gh
| ## Introduction | ||
|
|
||
| For now ATF reports folder have inconvenient nested struct. ATF creates struct of folders from relative path to script. | ||
|
|
There was a problem hiding this comment.
-
folder has
-
It would be good to see the current folder structure and what exactly is inconvenient in order to understand the difference with proposed solution
-
maybe it worth to name the proposal like "Change ATF test report folder structure" so it will reflect the sense more clear
|
@GetmanetsIrina |
| Main folder 'TestingReports' | ||
| -> Subfolder '<ScriptName>_<timestamp>' | ||
| ->Subfolder 'ATF' | ||
| ->Subfolder 'SDL' |
There was a problem hiding this comment.
@GetmanetsIrina
In the final view this text loses its formatting so very hard to read and understand what is the proposed structure. Please wrap this part as code or insert picture or whatever to make that clear for the people to understand.
| ## Proposed solution | ||
|
|
||
| The solution is to create new struct: | ||
| <pre><code>Main folder 'TestingReports' |
There was a problem hiding this comment.
@GetmanetsIrina
Sugestion is to remove 'Main folder', 'Subfolder' and single quotes in order to conform example in 'Introduction' section
|
|
||
| ## Impact on existing code | ||
|
|
||
| Impact on ATF reporting functionality. |
There was a problem hiding this comment.
@GetmanetsIrina Please note that CI jobs will require an update
|
|
||
| ## Introduction | ||
|
|
||
| For now ATF reports folder has inconvenient nested struct. ATF creates struct of folders from relative path to script, e.g. runned script fom test_scripts/API/Navidation/Subscriptions/. |
There was a problem hiding this comment.
@GetmanetsIrina What is struct? Please use full words instead of abbreviations.
|
@dboltovskyi , @AByzhynar please fing changes in commit 2d2ef88 |
|
|
||
| ## Introduction | ||
|
|
||
| For now ATF reports folder has inconvenient nested struct. ATF creates structure of folders from relative path to script, e.g. runned script fom test_scripts/API/Navidation/Subscriptions/. |
There was a problem hiding this comment.
@GetmanetsIrina For now ATF reports folder has inconvenient nested struct
|
|
||
| ## Proposed solution | ||
|
|
||
| The solution is to create new struct: |
There was a problem hiding this comment.
@GetmanetsIrina I propose use structure instead of struct here
GetmanetsIrina
force-pushed
the
feature/ATFtestReportsFolderStruct
branch
from
b860be1 to
d7fdb1b
Compare
|
@aderiabin, @AByzhynar, @dev-gh, please find changes in commit d7fdb1b |
Updated Motivation section according to review comment.
* SendLocation for Mobile Nav This proposal is about extending the capability for apps to receive and service SendLocation requests from other apps which is currently limited to the Head Unit's embedded navigation system. * Updating links to assets * Updated links to assets * Create a * Asset #1 * asset #2 * Delete a * Delete SendLocationForMobileNav_RAI.jpg * Add files via upload * Delete SendLocationForMobileNav_RequestResponse.jpg * Add files via upload * Addressed comments
7 participants
This PR contains proposal for SDL evolution regarding changing of ATF test report structure