[
https://issues.apache.org/jira/browse/FLINK-10733?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Chesnay Schepler closed FLINK-10733.
------------------------------------
Resolution: Fixed
Fix Version/s: 1.7.0
master: 89e59d8f398abf3597eb268c7e34927e35d8f383
1.7: 6d138a84e3feab488d679716e1475f95508d2662
> Misleading clean_log_files() in common.sh
> -----------------------------------------
>
> Key: FLINK-10733
> URL: https://issues.apache.org/jira/browse/FLINK-10733
> Project: Flink
> Issue Type: Bug
> Components: E2E Tests
> Affects Versions: 1.6.2
> Reporter: Kostas Kloudas
> Assignee: vinoyang
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.7.0
>
>
> In the `common.sh` base script of the end-to-end tests, there is a
> `clean_stdout_files` which cleans only the `*.out` files and a
> `clean_log_files` which cleans *both* `*.log` and `*.out` files.
> Given the current behavior that at the end of a test, the logs are checked
> and if there are exceptions (even expected ones but not whitelisted), the
> tests fails, some tests chose to call the `clean_log_files` so that
> exceptions are ignored. In this case, also `*.out` files are cleaned so if a
> test was checking for errors in the `.out` files, then the test will falsely
> pass.
> The solution is as simple as renaming the method to something more
> descriptive like `clean_logs_and_output_files`, but doing so, also includes
> checking if any existing tests were falsely passing.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
