-
Notifications
You must be signed in to change notification settings - Fork 66
EPMRPP-95621 || Pattern-Analysis doesn't work on Launch finish when AA finished working #2093
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…A finished working
@@ -67,7 +67,7 @@ public void shouldAnalyzeWhenEnabled() { | |||
runner.handle(event, mapping); | |||
|
|||
verify(launchPatternAnalyzer, times(1)).analyzeLaunch(launch, | |||
Sets.newHashSet(AnalyzeItemsMode.TO_INVESTIGATE, AnalyzeItemsMode.IGNORE_IMMEDIATE)); | |||
Sets.newHashSet(AnalyzeItemsMode.TO_INVESTIGATE, AnalyzeItemsMode.AUTO_ANALYZED, AnalyzeItemsMode.IGNORE_IMMEDIATE)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line is longer than 100 characters (found 125).
|
||
/** | ||
* @author <a href="mailto:[email protected]">Ivan Budayeu</a> | ||
*/ | ||
public class ProjectConfigDelegatingSubscriber<T extends ProjectIdAwareEvent> implements | ||
EventSubscriber<T> { | ||
|
||
private static final Logger LOGGER = LoggerFactory.getLogger(ProjectConfigDelegatingSubscriber.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line is longer than 100 characters (found 104).
LOGGER.error("Handle event: " + event); | ||
h.handle(event, projectConfig); | ||
} catch (Exception e) { | ||
LOGGER.debug("Error while processing event: " +e.getMessage() ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WhitespaceAround: '+' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
LOGGER.error("Handle event: " + event); | ||
h.handle(event, projectConfig); | ||
} catch (Exception e) { | ||
LOGGER.debug("Error while processing event: " +e.getMessage() ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
')' is preceded with whitespace.
No description provided.