[ 
https://issues.apache.org/jira/browse/CALCITE-3595?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Danny Chen resolved CALCITE-3595.
---------------------------------
    Fix Version/s: 1.22.0
         Assignee: Danny Chen
       Resolution: Fixed

Fixed in 
[edc386d|https://github.com/apache/calcite/commit/edc386d960b12642cdca40abb692beab6f75ce72],
 thanks for your PR, [~yanlin-Lynn] !

> Test infrastructure overwrites reference log with wrong results
> ---------------------------------------------------------------
>
>                 Key: CALCITE-3595
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3595
>             Project: Calcite
>          Issue Type: Bug
>          Components: build
>            Reporter: Julian Hyde
>            Assignee: Danny Chen
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.22.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> When I add a test case to a test that is based on DiffRepository (such as 
> SqlPrettyWriterTest), and run it in Intellij, the first time gives an error 
> because the actual output does not match the expected output. This is good.
> I would expect that if I run it again, it will fail again, but it does not. 
> I'm not sure exactly what is going on, but maybe the test is reading from an 
> output file instead of a reference log. This was probably broken during the 
> migration to gradle, because the paths for reference logs have all changed.
> This bug prevents me from using the workflow where I add a test case, run it, 
> and when it fails, change the code until the output looks right, then 
> overwrite the reference log.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to