[ 
https://issues.apache.org/jira/browse/KAFKA-20900?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18102337#comment-18102337
 ] 

Gaurav Narula commented on KAFKA-20900:
---------------------------------------

Thanks for reporting this Chia! I had found the same last week but had covered 
it under a minor PR at https://github.com/apache/kafka/pull/22987

I've edited its title now and given it this issue number

> docker_sanity_test.py should not ignore "Error" count
> -----------------------------------------------------
>
>                 Key: KAFKA-20900
>                 URL: https://issues.apache.org/jira/browse/KAFKA-20900
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Chia-Ping Tsai
>            Assignee: Gaurav Narula
>            Priority: Minor
>
> The `failure_count` does not account for environment or execution errors. 
> This leads to a misleading console output stating "all tests passed 
> successfully", even when no tests were actually executed
>  
> {code:py}
>     runner = HTMLTestRunner.HTMLTestRunner(
>                 stream=outfile,
>                 title=f'Test Report: Apache Kafka {mode.capitalize()} Docker 
> Image',
>                 description='This demonstrates the report output.'
>                 )
>     result = runner.run(combined_suite)
>     return result.failure_count
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to