-
Notifications
You must be signed in to change notification settings - Fork 7
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
finish step undefined status #94
Conversation
@AmsterGet ping? |
Hi guys! |
Hi @AmsterGet! |
Hi @AmsterGet. How are you? I'm really interested to use reportportal in my company with Postman and Azure Pipelines, but the issues 58 and 64 are resulting in undefined status in some cases. Is there any deadline? |
@@ -441,7 +456,7 @@ class Reporter { | |||
finishSuite () { | |||
const currentSuiteTempId = this.getCurrentSuiteTempId(); | |||
const status = this.collectionRunOptions.collection && | |||
utils.getCollectionVariablesByKey('status', this.collectionRunOptions.collection.variables); | |||
utils.getCollectionVariablesByKey('status', this.collectionRunOptions.collection.variables) || 'PASSED'; |
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.
The ReportPortal API will calculate the final suite status based on the statuses of its descendants, so setting here PASSED
as a fallback may lead to wrong suite status in the system.
Hi guys! Thanks @dgosantos89 for the solution. Please check my comments above. @brunotanoue Good to hear that. We will keep you updated in this topic. |
Hi,@dgosantos89. I hope you are doing well. This issue occurred because we didn't support conditional tests. |
Closed due to irrelevance |
@dgosantos89, FYI: The new version of the agent has been released. You can test it out with your tests. |
I had the same problem that was reported at Issue 58 and Issue 64
Some steps was marked with In progress, because the step was undefined, so my solution is finish all steps that was undefined status as SKIPPED status