Merge pull request #163 from JuliaGizmos/jbj/testy-printy

Gives a descriptive name to Interact signal printers, fixes a test with latest Reactive, fixes Travis issue on macos with julia 0.5

.travis.yml

@@ -8,13 +8,10 @@ julia:
   - nightly
 notifications:
   email: false
-matrix:
-  allow_failures:
-    - julia: nightly
 script:
   - if [[ -a .git/shallow ]]; then git fetch --unshallow; fi
   - julia -e 'Pkg.clone(pwd())'
-  - julia -e 'Pkg.test("Interact"; coverage=true)'
+  - julia -e 'cov=(VERSION < v"0.5-" || !is_apple() || VERSION >= v"0.6-"); Pkg.test("Interact"; coverage=cov)'
 after_success:
   - julia --color=yes -e 'cd(Pkg.dir("Interact")); Pkg.add("Coverage"); using Coverage;
     Coveralls.submit(Coveralls.process_folder())'

src/IJulia/setup.jl

@@ -78,7 +78,7 @@ function init_comm(x::Signal)
                                  get_data_dict(value, mimes)))
             end
         end
-        preserve(map(notify, x))
+        preserve(map(notify, x; name="$(x.name) printer (Interact)"))
     else
         comm = comms[x]
     end

test/ijulia.jl

@@ -22,7 +22,7 @@ sliderWidget = slider(1:5)
     # Julia v0.6 uses spaces after commas when printing types, but
     # previous versions did not. To ensure that this test works on all
     # versions, we just remove spaces from both sides of the comparison.
-    @test removespaces("""Interact.Slider{Int64}(Signal{Int64}(3, nactions=1),\"\",3,1:5,\"horizontal\",true,\"d\",true)""") == removespaces(stringmime("text/plain", sliderWidget))
+    @test removespaces("""Interact.Slider{Int64}($(string(signal(sliderWidget))),\"\",3,1:5,\"horizontal\",true,\"d\",true)""") == removespaces(stringmime("text/plain", sliderWidget))
     @test "3" == stringmime("text/plain", signal(sliderWidget))
 
     @test "" == stringmime("text/html", sliderWidget)