https://issues.apache.org/bugzilla/show_bug.cgi?id=48163

--- Comment #5 from Sebb <s...@apache.org> 2009-11-13 13:53:05 UTC ---
(In reply to comment #4)
> I do not understand why this test is failing for you.
> 
> As you can see, it's supposed to check that the entry in the CacheManager
> created by a call to saveDetails is used to set the connection's property 
> list:
> the "If-None-Match" set to the etag value and the "If-Modified-Since" set to
> the current time.
> 
> "It runs on my machine!" with a fresh copy of the CacheManager downloaded a 
> few
> moments ago, only changing the visibility of the CacheEntry inner class.

That should not be necessary, as I've already fixed that, so maybe you have not
downloaded the current code.

> Clearly the CacheEntry found in setHeaders has no etag. 

Not according to the way I read the code - it is checking the properties which
are obtained from this.httpUrlConnection.getRequestProperties();

> Therefore, the
> CacheEntry whose etag is set earlier in the test through reflection must not 
> be
> the same CacheEntry used by setHeaders, despite the fact that they are indexed
> by the URL.
> 
> If I could duplicate the problem, I could debug it. I wouldn't ask you to do 
> my
> work and step through the test to see where it gets off the track but I cannot
> see how to proceed otherwise. I'd be curious whether the CacheEntry 
> manipulated
> in the test's initialization is the same object as that used by setHeaders and
> since it probably is not, how the heck another got there.
> 
> Thoughts?

See above; can you try downloading the current code from trunk?

> John Samford

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscr...@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-h...@jakarta.apache.org

Reply via email to