This is an automated email from the ASF dual-hosted git repository.

heneveld pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/brooklyn-server.git


The following commit(s) were added to refs/heads/master by this push:
     new c10b07b151 fix for intermittent initializer test failure, now that it 
might run async with creation
     new b9ade38c0e Merge branch 'master' of 
https://gitbox.apache.org/repos/asf/brooklyn-server
c10b07b151 is described below

commit c10b07b15102a31ce3e02c5eab2682be8db4a1f4
Author: Alex Heneveld <g...@alex.heneveld.org>
AuthorDate: Mon Jul 29 12:43:54 2024 +0100

    fix for intermittent initializer test failure, now that it might run async 
with creation
    
    async initializer might not complete until after startup completes.
    notes on possible future enhancements.
---
 .../java/org/apache/brooklyn/camp/brooklyn/WorkflowYamlTest.java     | 5 +++++
 1 file changed, 5 insertions(+)

diff --git 
a/camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/WorkflowYamlTest.java
 
b/camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/WorkflowYamlTest.java
index 7b3ea599a2..b4c87bca07 100644
--- 
a/camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/WorkflowYamlTest.java
+++ 
b/camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/WorkflowYamlTest.java
@@ -217,6 +217,11 @@ public class WorkflowYamlTest extends AbstractYamlTest {
         waitForApplicationTasks(app);
 
         Entity entity = Iterables.getOnlyElement(app.getChildren());
+
+        // may need to wait for workflow initializer. TBC whether this should 
be part of a REST application start, or another REST method.
+        // (maybe the default rest application create started should wait for 
these prior to invoking start? maybe they should be tagged as initialization?)
+        waitForApplicationTasks(entity);
+
         EntityAsserts.assertAttribute(entity, Sensors.newSensor(Object.class, 
"foo"), v -> {
             Asserts.assertInstanceOf(v, SpecialMap.class);
             Asserts.assertEquals( ((SpecialMap)v).x, "bar" );

Reply via email to