fixing tests: first message is empty due to env build. Try to do it in beforeAll()

Author: takbal

lib/julia-server.jl

@@ -490,9 +490,7 @@ try
     # this is looping until Atom dies
     while true
         conn = accept(server)
-        # there isn't too much point in @spawn this: only the server generation is slow, and that is MT anyway
-        # you would also need to add more threads that are not going do too much all of the time
-        # but you may get weird race conditions
+        # the pipe is fifo anyway and cannot deal with multiple connections
         @async handle_connection(conn)
     end
 

spec/linter-julia-spec.js

@@ -20,6 +20,13 @@ describe('The Julia StaticLint.jl provider for Linter', () => {
     await atom.packages.activatePackage('linter-julia');
   });
 
+  // avoid first pass empty messages by linting any files in the environment, then
+  // waiting for the symbols to build
+  beforeAll(function (done) {
+    await atom.workspace.open(badFile);
+    setTimeout(done, 15000);
+  });
+
   it('checks a file with syntax error and reports the correct message', async () => {
     const excerpt = 'Missing reference';
     const editor = await atom.workspace.open(badFile);