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

Daniel Shahaf commented on SVN-4891:
------------------------------------

> This is expecting to be able to revert some replacements. The WC was created 
> by upgrade, and the repo is not available. Is the problem that the upgrade 
> should have left the pristines in place for these files, but actually it 
> removed them?

 This currently XPASSes under f31 and XFAILs under f32.

I think that's the correct behaviour for f32 {_}as currently defined{_}. 
However, this is a form of data loss, so once SVN-4889 is implemented, it would 
be reasonable to have {{svn upgrade}} default pristines-on-demand behaviour to 
off for upgraded wc's, and then this test should PASS under the new format as 
well.

> Pristines-on-demand: fix disabled tests
> ---------------------------------------
>
>                 Key: SVN-4891
>                 URL: https://issues.apache.org/jira/browse/SVN-4891
>             Project: Subversion
>          Issue Type: Bug
>            Reporter: Julian Foad
>            Assignee: Julian Foad
>            Priority: Major
>
> The following tests had been changed in the pristines-on-demand development.
> Most were disabled with the '@Wimp' decorator. (This caused some of them to 
> give an XPASS and some an XFAIL result when run against the older WC format, 
> and all XFAIL against the newer WC format.)
> They need to be re-enabled and fixed.
> (I am removing them from this list as and when fixed.)
> {code:java}
>     basic_tests.py 8 basic_commit_corruption(): basic corruption detection on 
> commit
>       [[Relies on wc.text_base_path()]]
>     basic_tests.py 9 basic_update_corruption(): basic corruption detection on 
> update
>       [[Relies on wc.text_base_path()]]
>     externals_tests.py ... ... ...:
>       update_modify_file_external(),
>       remap_file_external_with_prop_del(),
>       file_external_recorded_info():
>         existing issue (Windows only)
>     revert_tests.py 2 revert_reexpand_keyword(): revert reexpands manually 
> contracted keyword
>       "See r1101730 and r1101817"
>     trans_tests.py 1 keywords_from_birth(): commit new files with keywords 
> active from birth
>       [[Relies on wc.text_base_path()]]
>     trans_tests.py 3 eol_change_is_text_mod(): committing eol-style change 
> forces text send
>       [[Relies on wc.text_base_path()]]
>     update_tests.py 57 skip_access_denied(): access denied paths should be 
> skipped
>       [[Need to update the access denied handling in update_editor.c]]
>     upgrade_tests.py 16 replaced_files(): upgrade with base and working 
> replaced files
>       [[Can't fetch pristines: the working copy points to file:///tmp/repo]]
>     {code}
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to