ExtentReports - The @BeforeMethod annotation steps listed in the previous test logs info at the end #257
Comments
MohabMohie
added a commit
that referenced
this issue
Feb 16, 2021
. Separate method logs for test and configuration methods . Standerdize naming convention to use className.methodName and then add Description as a log entry . Disable Gif embedding in the extentreport to save space Signed-off-by: Mohab Mohie <[email protected]>
MohabMohie
added a commit
that referenced
this issue
Feb 16, 2021
. Removed logs for configuration methods to reduce report size . Created method to force pass tests on extent reports . Removed logic to force fail or force pass assertions logs to better map with allure report numbers . Applied Codacy comments to enhance readability . Created private method to enhance performance for extent report logging Signed-off-by: Mohab Mohie <[email protected]>
|
Closed, pending PR merge. |
MohabMohie
added a commit
that referenced
this issue
Feb 16, 2021
* Fixed #255 . Enhanced element identification algorithm again to maintain improved performance and close the logical gap that led to the bug. . Added tests to cover this code properly Signed-off-by: Mohab Mohie <[email protected]> * Possible fix for #256 . Couldn't reproduce issue locally (two windows machines) or remotely (GitHub Actions ubuntu) . Attempted Blind fix based on attached exception, and will wait for confirmation of the fix effectiveness on the target system. Signed-off-by: Mohab Mohie <[email protected]> * Change Extent Reports directory to enable automated cleaning Signed-off-by: Mohab Mohie <[email protected]> * Fixes #259 Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Separate method logs for test and configuration methods . Standerdize naming convention to use className.methodName and then add Description as a log entry . Disable Gif embedding in the extentreport to save space Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Removed logs for configuration methods to reduce report size . Created method to force pass tests on extent reports . Removed logic to force fail or force pass assertions logs to better map with allure report numbers . Applied Codacy comments to enhance readability . Created private method to enhance performance for extent report logging Signed-off-by: Mohab Mohie <[email protected]> * Restoring Exceptions view to extent reports Signed-off-by: Mohab Mohie <[email protected]>
MohabMohie
added a commit
that referenced
this issue
Feb 17, 2021
* Fixed #255 . Enhanced element identification algorithm again to maintain improved performance and close the logical gap that led to the bug. . Added tests to cover this code properly Signed-off-by: Mohab Mohie <[email protected]> * Possible fix for #256 . Couldn't reproduce issue locally (two windows machines) or remotely (GitHub Actions ubuntu) . Attempted Blind fix based on attached exception, and will wait for confirmation of the fix effectiveness on the target system. Signed-off-by: Mohab Mohie <[email protected]> * Change Extent Reports directory to enable automated cleaning Signed-off-by: Mohab Mohie <[email protected]> * Fixes #259 Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Separate method logs for test and configuration methods . Standerdize naming convention to use className.methodName and then add Description as a log entry . Disable Gif embedding in the extentreport to save space Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Removed logs for configuration methods to reduce report size . Created method to force pass tests on extent reports . Removed logic to force fail or force pass assertions logs to better map with allure report numbers . Applied Codacy comments to enhance readability . Created private method to enhance performance for extent report logging Signed-off-by: Mohab Mohie <[email protected]> * Restoring Exceptions view to extent reports Signed-off-by: Mohab Mohie <[email protected]> * extent extras . cleanExtentReportsDirectoryBeforeExecution . reattach animated gif to extent . use test description when available * extent extras . version bump to 5.1.20210217 . fix cursed file issue... don't ask... Signed-off-by: Mohab Mohie <[email protected]> * extent extras . minor typo fix Signed-off-by: Mohab Mohie <[email protected]>
MohabMohie
added a commit
that referenced
this issue
Nov 18, 2022
* Fixed #255 . Enhanced element identification algorithm again to maintain improved performance and close the logical gap that led to the bug. . Added tests to cover this code properly Signed-off-by: Mohab Mohie <[email protected]> * Possible fix for #256 . Couldn't reproduce issue locally (two windows machines) or remotely (GitHub Actions ubuntu) . Attempted Blind fix based on attached exception, and will wait for confirmation of the fix effectiveness on the target system. Signed-off-by: Mohab Mohie <[email protected]> * Change Extent Reports directory to enable automated cleaning Signed-off-by: Mohab Mohie <[email protected]> * Fixes #259 Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Separate method logs for test and configuration methods . Standerdize naming convention to use className.methodName and then add Description as a log entry . Disable Gif embedding in the extentreport to save space Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Removed logs for configuration methods to reduce report size . Created method to force pass tests on extent reports . Removed logic to force fail or force pass assertions logs to better map with allure report numbers . Applied Codacy comments to enhance readability . Created private method to enhance performance for extent report logging Signed-off-by: Mohab Mohie <[email protected]> * Restoring Exceptions view to extent reports Signed-off-by: Mohab Mohie <[email protected]>
MohabMohie
added a commit
that referenced
this issue
Nov 18, 2022
* Fixed #255 . Enhanced element identification algorithm again to maintain improved performance and close the logical gap that led to the bug. . Added tests to cover this code properly Signed-off-by: Mohab Mohie <[email protected]> * Possible fix for #256 . Couldn't reproduce issue locally (two windows machines) or remotely (GitHub Actions ubuntu) . Attempted Blind fix based on attached exception, and will wait for confirmation of the fix effectiveness on the target system. Signed-off-by: Mohab Mohie <[email protected]> * Change Extent Reports directory to enable automated cleaning Signed-off-by: Mohab Mohie <[email protected]> * Fixes #259 Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Separate method logs for test and configuration methods . Standerdize naming convention to use className.methodName and then add Description as a log entry . Disable Gif embedding in the extentreport to save space Signed-off-by: Mohab Mohie <[email protected]> * Fixes #257 . Removed logs for configuration methods to reduce report size . Created method to force pass tests on extent reports . Removed logic to force fail or force pass assertions logs to better map with allure report numbers . Applied Codacy comments to enhance readability . Created private method to enhance performance for extent report logging Signed-off-by: Mohab Mohie <[email protected]> * Restoring Exceptions view to extent reports Signed-off-by: Mohab Mohie <[email protected]> * extent extras . cleanExtentReportsDirectoryBeforeExecution . reattach animated gif to extent . use test description when available * extent extras . version bump to 5.1.20210217 . fix cursed file issue... don't ask... Signed-off-by: Mohab Mohie <[email protected]> * extent extras . minor typo fix Signed-off-by: Mohab Mohie <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
In the ExtentReports emailable HTML report - The @BeforeMethod annotation steps are listed in the previous test logs info at the end
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The steps of the @BeforeMethod should be inside of it's own test case info steps at first
Screenshots
Test Case No.1 (No @BeforeMethod steps)
Then after it (Test Case No.2)
Desktop:
Additional context
N/A
The text was updated successfully, but these errors were encountered: