[JCBC-496] Add unit test for JCBC-488 Created: 22/Jul/14  Updated: 22/Jul/14

Status: Open
Project: Couchbase Java Client
Component/s: None
Affects Version/s: None
Fix Version/s: None
Security Level: Public

Type: Improvement Priority: Major
Reporter: Michael Nitschinger Assignee: Wei-Li Liu
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   
We need to add a unit test for the fix in http://www.couchbase.com/issues/browse/JCBC-488 just to make sure it works as expected. The appropriate change ishttp://review.couchbase.org/#/c/39429/ and has been merged already.

I'd propose a unit test which mocks out the underlying stuff and just pushes a HttpOperation a few times through the ViewConnection#addOp which should add the headers only 1 times instead of N.




[JCBC-495] 1.4x doesn't build if source dir contains whitespace Created: 16/Jul/14  Updated: 16/Jul/14

Status: Open
Project: Couchbase Java Client
Component/s: Core
Affects Version/s: 1.4.3
Fix Version/s: None
Security Level: Public

Type: Bug Priority: Minor
Reporter: David Haikney Assignee: David Haikney
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   
After running a git clone; git checkout of 1.4.3 I was unable to build on my local machine. "ant package" yielded the following:

 [exec] /Users/dhaikney/Documents/Google Drive/dev/couchbase-java-client/src/scripts/write-version-info.sh: line 47: ${changesfile}: ambiguous redirect

Some of the variables in the write-version-info.sh script need surrounding quotes to allow for the whitespace.

 Comments   
Comment by David Haikney [ 16/Jul/14 ]
http://review.couchbase.org/39443




Generated at Thu Jul 24 02:15:10 CDT 2014 using JIRA 5.2.4#845-sha1:c9f4cc41abe72fb236945343a1f485c2c844dac9.