Handle umbrella projects when deciding if test file

Author: mauricerkelly

lib/init.js

@@ -31,12 +31,12 @@ const findElixirProjectPath = async (editorPath) => {
   const editorDir = dirname(editorPath);
   const mixexsPath = find(editorDir, 'mix.exs');
   if (mixexsPath !== null) {
-    const pathArray = mixexsPath.split('/');
+    const pathArray = mixexsPath.split(sep);
     if (pathArray.length > 3 && pathArray[pathArray.length - 3] === 'apps') {
       //  Treat this as an umbrella app. This may be wrong -
       //  If you happen to keep your code in a directory called 'apps'
       pathArray.splice((pathArray.length - 3), 3);
-      const umbrellaProjectPath = pathArray.join('/');
+      const umbrellaProjectPath = pathArray.join(sep);
 
       //  Safety check by looking for a `mix.exs` file in the same directory as
       //  'apps'. If it exists, then it's likely an umbrella project
@@ -68,11 +68,21 @@ const isMixProject = async (filePath) => {
   return existsSync(join(project, 'mix.exs'));
 };
 
+const isUmbrellaProject = async (filePath) => {
+  const project = await elixirProjectPath(filePath);
+  return existsSync(join(project, 'apps'));
+}
+
 const isTestFile = async (filePath) => {
   const project = await elixirProjectPath(filePath);
   const relativePath = relative(project, filePath);
-  // Is the first directory of the relative path "test"?
-  return relativePath.split(sep)[0] === 'test';
+  if (isUmbrellaProject()) {
+    // Is the structure "apps/app_name/test/..."
+    return relativePath.split(sep)[2] === 'test';
+  } else {
+    // Is the structure "test/..."
+    return relativePath.split(sep)[0] === 'test';
+  }
 };
 
 const isForcedElixirc = () => forceElixirc;