vijayaguru <vi...@collab.net> writes:

> +#----------------------------------------------------------------------
> +# Test for issue #3471 'svn up touches file w/ lock & svn:keywords property'
> +#
> +# Marked as XFail until that issue is fixed.
> +def update_with_file_lock_and_keywords_property_set(sbox):
> +  """update with file lock & keywords property set"""
> +  sbox.build()
> +  
> +  wc_dir = sbox.wc_dir
>  
> +  mu_path = os.path.join(wc_dir, 'A', 'mu')
> +  svntest.main.file_append(mu_path, '$Id$')
> +  svntest.main.run_svn(None, 'ps', 'svn:keywords', 'Id', mu_path)
> +  svntest.main.run_svn(None, 'lock', mu_path)
> +  mu_ts_before_update = os.path.getmtime(mu_path)

The keyword is present but not expanded.

> +  
> +  # Issue #3471 manifests itself here; The timestamp of 'mu' gets updated 
> +  # to the time of the last "svn up".
> +  sbox.simple_update()

That update expands the keyword, so I would expect the timestamp to
change.

> +  mu_ts_after_update = os.path.getmtime(mu_path)
> +  if (mu_ts_before_update != mu_ts_after_update):
> +    print("The timestamp of 'mu' before and after update does not match.")
> +    raise svntest.Failure

Why is this an failure?

Perhaps there should be a second call to update, with a check that the
second doesn't change the timestamp.  With two update
sleep_for_timestamps could be used to introduce the necessary delay to
trigger the bug.

-- 
Philip

Reply via email to