Merge pull request #111 from rdeits/rd/wait

work-around for race condition in `wait()`

src/visualizer.jl

@@ -114,7 +114,19 @@ function send(c::CoreVisualizer, cmd::AbstractCommand)
     nothing
 end
 
-Base.wait(c::CoreVisualizer) = WebIO.ensure_connection(c.scope.pool)
+function Base.wait(c::CoreVisualizer)
+    pool = c.scope.pool
+    while true
+        while isready(pool.new_connections)
+            push!(pool.connections, take!(pool.new_connections))
+        end
+        if !isempty(pool.connections)
+            break
+        else
+            sleep(0.25)
+        end
+    end
+end
 
 """
     vis = Visualizer()

test/runtests.jl

@@ -12,6 +12,7 @@ using MeshIO, FileIO
     include("visualizer.jl")
     include("notebook.jl")
     include("scenes.jl")
+    include("wait.jl")
 end
 
 sleep(10)

test/wait.jl

@@ -0,0 +1,20 @@
+# https://github.com/rdeits/MeshCat.jl/pull/99#issuecomment-509067108
+
+@testset "wait" begin
+    @testset "with setobject" begin
+        vis = Visualizer()
+        setobject!(vis, Triad(1.0))
+        if !haskey(ENV, "CI")
+            open(vis)
+            wait(vis)
+        end
+    end
+
+    @testset "without setobject" begin
+        vis = Visualizer()
+        if !haskey(ENV, "CI")
+            open(vis)
+            wait(vis)
+        end
+    end
+end