fix: don’t resolve empty meteor package.js

Refactors meteor to have a resolve function that detects/filters out empty files (like for other managers).

lib/manager/meteor/resolve.js

+const configParser = require('../../config');
+
+module.exports = {
+  resolvePackageFile,
+};
+
+async function resolvePackageFile(config, inputFile) {
+  const packageFile = configParser.mergeChildConfig(config.meteor, inputFile);
+  logger.debug(
+    `Resolving packageFile ${JSON.stringify(packageFile.packageFile)}`
+  );
+  packageFile.content = await platform.getFile(packageFile.packageFile);
+  if (!packageFile.content) {
+    logger.debug('No packageFile content');
+    return null;
+  }
+  return packageFile;
+}

lib/manager/resolve.js

@@ -4,6 +4,7 @@ const minimatch = require('minimatch');
 
 const manager = require('./index');
 const dockerResolve = require('../manager/docker/resolve');
+const meteorResolve = require('../manager/meteor/resolve');
 const nodeResolve = require('../manager/node/resolve');
 const bazelResolve = require('../manager/bazel/resolve');
 const { mergeChildConfig } = require('../config');
@@ -146,7 +147,7 @@ async function resolvePackageFiles(config) {
       packageFile.currentPackageJsonVersion = packageFile.content.version;
       return mergeChildConfig(config.npm, packageFile);
     } else if (packageFile.manager === 'meteor') {
-      return mergeChildConfig(config.meteor, packageFile);
+      return meteorResolve.resolvePackageFile(config, packageFile);
     } else if (packageFile.manager === 'docker') {
       return dockerResolve.resolvePackageFile(config, packageFile);
     } else if (packageFile.manager === 'node') {

test/manager/__snapshots__/resolve.spec.js.snap

@@ -45,6 +45,7 @@ Array [
 exports[`manager/resolve resolvePackageFiles() detects meteor and docker and travis and bazel 1`] = `
 Array [
   Object {
+    "content": "{}",
     "enabled": true,
     "manager": "meteor",
     "packageFile": "package.js",

test/manager/resolve.spec.js

@@ -97,7 +97,8 @@ describe('manager/resolve', () => {
         '.travis.yml',
         'WORKSPACE',
       ];
-      platform.getFile.mockReturnValueOnce('# comment\nFROM node:8\n'); // Dockerfile.js
+      platform.getFile.mockReturnValueOnce('{}'); // package.js
+      platform.getFile.mockReturnValueOnce('# comment\nFROM node:8\n'); // Dockerfile
       platform.getFile.mockReturnValueOnce('hello: world\n'); // Dockerfile
       platform.getFile.mockReturnValueOnce('# travis'); // .travis.yml
       platform.getFile.mockReturnValueOnce('# WORKSPACE'); // Dockerfile
@@ -110,6 +111,7 @@ describe('manager/resolve', () => {
         'other/Dockerfile',
         '.travis.yml',
         'WORKSPACE',
+        'package.js',
       ];
       platform.getFile.mockReturnValueOnce('# comment\n'); // Dockerfile
       const res = await resolvePackageFiles(config);