Continuum

continuum plugin doesn't fails to add maven two projects

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.2
  • Component/s: Maven Plugin
  • Labels:
    None
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

If something doesn't work while adding the project, there is no error on the client side. The message appear only in continuum logs (maybe a probleme on the server side ?).

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Olivier Lamy added a comment - 
AddingResult addingResult = client.addMavenTwoProject( "");
addingResult.getErrorsAsString();

It is not enough ?

Show
Olivier Lamy added a comment - AddingResult addingResult = client.addMavenTwoProject( ""); addingResult.getErrorsAsString(); It is not enough ?
Hide
Permalink
Benoit Decherf added a comment -

well, I was speaking about AddMavenTwoProject class in maven-continuum-project.

in the execute method, the AddingResult is not checked, so the plugin never fails, even if the call to continuum fails.

It seems that there is the same issue for all features.

Show
Benoit Decherf added a comment - well, I was speaking about AddMavenTwoProject class in maven-continuum-project. in the execute method, the AddingResult is not checked, so the plugin never fails, even if the call to continuum fails. It seems that there is the same issue for all features.
Hide
Permalink
Olivier Lamy added a comment -

When you say "well, I was speaking about AddMavenTwoProject class in maven-continuum-project. " which class in which package ?
And how are you using it ? (If you have some spare time we can have a chat on irc.codehaus.org channel continuum)

Show
Olivier Lamy added a comment - When you say "well, I was speaking about AddMavenTwoProject class in maven-continuum-project. " which class in which package ? And how are you using it ? (If you have some spare time we can have a chat on irc.codehaus.org channel continuum)
Hide
Permalink
Olivier Lamy added a comment -

fixed in rev 648029

Show
Olivier Lamy added a comment - fixed in rev 648029

People

  • Assignee:
    Olivier Lamy
    Reporter:
    Benoit Decherf
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: