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

ASF GitHub Bot commented on GEODE-2532:
---------------------------------------

Github user mmartell commented on the issue:

    https://github.com/apache/geode-native/pull/30
  
    @pivotal-jbarrett It seems like a very good suggestion to keep all test 
files local to each test subdirectory. That would avoid potential name 
collision of these configured xml files. It looks like an attempt was made to 
avoid such file name collisions by appending a random number, but that isn't a 
guarantee.
    
    I've created a new ticket (GEODE-2540) to move these xml config files to 
each local test subdirectory. That is a much bigger task which will require 
refactoring the NUnit based tests to extract the current test name. 
    
    For now, I'd like to suggest we accept this PR which solves the problem of 
polluting the src tree with these test files. 
    
    



> .NET Integration Tests Create Temp Files in the Source Directory
> ----------------------------------------------------------------
>
>                 Key: GEODE-2532
>                 URL: https://issues.apache.org/jira/browse/GEODE-2532
>             Project: Geode
>          Issue Type: Improvement
>          Components: native client
>            Reporter: Michael Martell
>         Attachments: ExtraneousTempFiles.jpg
>
>
> The C++/CLI integration tests create temporary .xml files in the src 
> directory as shown in attachment. This pollutes your working area with files 
> that should not be added to the repo. 
> This task is to create these temp files in the runtime test folder.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to