Continuum Release should do a checkout if there is no working copy

Details

  • Complexity:
    Intermediate

Description

Rather than requiring an existing working copy, Continuum Release should be able to do a checkout if there is no working copy present for the project being released.

  1. Text File
    CONTINUUM-2458.patch
    26/Mar/10 6:13 AM
    3 kB
    Gwen Harold Autencio
  2. Text File
    unit-test-CONTINUUM-2458.patch
    28/Mar/10 9:22 PM
    3 kB
    Gwen Harold Autencio

Issue Links

is depended upon by

Bug - A problem which impairs or prevents the functions of the product. CONTINUUM-2386 Build environment selection is ignored when releasing with distributed build enabled

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Bug - A problem which impairs or prevents the functions of the product. CONTINUUM-2457 A successful build should not be required in order to release.

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Wendy Smoak added a comment -

CONTINUUM-2457 will have to be resolved first. Right now not only must there be an existing working copy, but the most recent build result must be a success before you are allowed to begin a release.

Show
Wendy Smoak added a comment - CONTINUUM-2457 will have to be resolved first. Right now not only must there be an existing working copy, but the most recent build result must be a success before you are allowed to begin a release.
Hide
Permalink
Gwen Harold Autencio added a comment -

Hi,

attaching patch for this issue.

Show
Gwen Harold Autencio added a comment - Hi, attaching patch for this issue.
Hide
Permalink
Gwen Harold Autencio added a comment -

I'll follow the unit test for this patch shortly.
Thanks

Show
Gwen Harold Autencio added a comment - I'll follow the unit test for this patch shortly. Thanks
Hide
Permalink
Jevica Arianne B. Zurbano added a comment -

Thanks Gwen!

BTW, I think it should be "checkOutScmResult" instead of "updateScmResult" for the following snippet, am I correct?

+        else
+        {
+            if( !checkOutScmResult.isSuccess() )
+            {
+                throw new ReleaseScmCommandException( "Unable to checkout project", updateScmResult );
+            }
+            
+            copyUpdated = checkOutScmResult.getCheckedOutFiles().size() > 0; 
+        }
Show
Jevica Arianne B. Zurbano added a comment - Thanks Gwen! BTW, I think it should be "checkOutScmResult" instead of "updateScmResult" for the following snippet, am I correct? 
+        else
+        {
+            if( !checkOutScmResult.isSuccess() )
+            {
+                throw new ReleaseScmCommandException( "Unable to checkout project", updateScmResult );
+            }
+            
+            copyUpdated = checkOutScmResult.getCheckedOutFiles().size() > 0; 
+        }
Hide
Permalink
Gwen Harold Autencio added a comment -

Hi Jevica,

Sorry that should have been "checkOutScmResult".
Attaching unit test for patch thanks.

Show
Gwen Harold Autencio added a comment - Hi Jevica, Sorry that should have been "checkOutScmResult". Attaching unit test for patch thanks.
Hide
Permalink
Jevica Arianne B. Zurbano added a comment -

Patch applied in r928579.

  • checkout project if working directory does not exist
  • unit test for non-existing working directory

I also added unit tests for working directory with project checkout and empty working directory.

Thanks Gwen!

Show
Jevica Arianne B. Zurbano added a comment - Patch applied in r928579.
  • checkout project if working directory does not exist
  • unit test for non-existing working directory
I also added unit tests for working directory with project checkout and empty working directory. Thanks Gwen!

People

  • Assignee:
    Jevica Arianne B. Zurbano
    Reporter:
    Wendy Smoak
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: