#5: no such file or directory

Summary: Will try to use the filename of the source to search for the root, will print a more useful error message on failure

index.js

@@ -24,7 +24,8 @@ module.exports = {
  },
  create(context) {
  const { resolvedTemplate, mustMatch, chars, onNonMatchingHeader, nonMatchingTolerance } = resolveOptions(
- context.options[0]
+ context.options[0],
+ context.getFilename()
  );
  const sourceCode = context.getSourceCode();
  const text = sourceCode.getText().substring(0, chars);
@@ -42,16 +43,16 @@ module.exports = {
  topNode = node;
  }
  let headerMatches = false;
- if(!headerMatches && mustMatch && text){
+ if (!headerMatches && mustMatch && text) {
  headerMatches = !!String(text).match(mustMatch);
  //If the header matches, return early
- if(headerMatches) return;
- } 
+ if (headerMatches) return;
+ }
  //If chars doesn't match, a header comment/template exists and nonMatchingTolerance is set, try calculating string distance
  if (!headerMatches && hasHeaderComment && resolvedTemplate && _.isNumber(nonMatchingTolerance)) {
- const dist = metriclcs(resolvedTemplate,firstComment.value);
+ const dist = metriclcs(resolvedTemplate, firstComment.value);
  //Return early, mark as true for future work if needed
- if(nonMatchingTolerance <= dist){
+ if (nonMatchingTolerance <= dist) {
  headerMatches = true;
  return;
  }

package.json

@@ -1,6 +1,6 @@
 {
  "name": "eslint-plugin-notice",
- "version": "0.5.6",
+ "version": "0.6.6",
  "description": "An eslint rule that checks the top of files and --fix them too!",
  "main": "index.js",
  "directories": {
@@ -25,6 +25,7 @@
  "eslint": "^4.10.0"
  },
  "dependencies": {
+ "find-root": "^1.1.0",
  "lodash": ">=2.4.0",
  "metric-lcs": "^0.1.2"
  },

tests/lib/rules/fix-result-1.js

@@ -1,6 +1,6 @@
 
 /**
- * Copyright (c) 2017, Nick Deis
+ * Copyright (c) 2018, Nick Deis
  */
 
 function noStyle(){

tests/lib/rules/fix-result-2.js

@@ -1,6 +1,6 @@
 
 /**
- * Copyright (c) 2017, Nick Deis
+ * Copyright (c) 2018, Nick Deis
  */
 
 /**

tests/lib/rules/fix-result-3.js

@@ -1,6 +1,6 @@
 
 /**
- * Copyright (c) 2017, Nick Deis
+ * Copyright (c) 2018, Nick Deis
  */
 
 

utils.js

@@ -3,7 +3,9 @@
  */
 
 const _ = require("lodash"),
- fs = require("fs");
+ fs = require("fs"),
+ findRoot = require("find-root"),
+ path = require("path");
 
 const COULD_NOT_FIND = `Could not find a match for the mustMatch pattern`;
 const REPORT_AND_SKIP = `Found a header comment which did not match the mustMatch pattern, skipping fix and reporting`;
@@ -26,16 +28,35 @@ function regexpizeTemplate({ template, varRegexps }) {
  return new RegExp(_.template(escapeRegExp(template))(allPatternVars));
 }
 
-function resolveOptions({
- mustMatch,
- templateFile,
- template,
- templateVars,
- chars,
- onNonMatchingHeader,
- varRegexps,
- nonMatchingTolerance
-}) {
+function resolveTemplate({ templateFile, template, fileName }) {
+
+ if(template) return template;
+ if(!templateFile){
+ throw new Error(`Either template or templateFile must be set`);
+ }
+ //Naively look for the templateFile first
+ if(fs.existsSync(templateFile)){
+ return fs.readFileSync(templateFile, "utf8");
+ }
+ if(!fs.existsSync(fileName)){
+ throw new Error(`Could not find the file name ${fileName}. This is necessary to find the root`);
+ }
+ const root = findRoot(fileName);
+ const rootTemplateFile = path.join(root,templateFile);
+ if(fs.existsSync(rootTemplateFile)){
+ return fs.readFileSync(rootTemplateFile, "utf8");
+ }
+ const absRootTemplateFile = path.resolve(rootTemplateFile);
+ if(fs.existsSync(absRootTemplateFile)){
+ return fs.readFileSync(absRootTemplateFile,"utf8");
+ }
+ throw new Error(`Can't find templateFile @ ${absRootTemplateFile}`);
+}
+
+function resolveOptions(
+ { mustMatch, templateFile, template, templateVars, chars, onNonMatchingHeader, varRegexps, nonMatchingTolerance },
+ fileName
+) {
  onNonMatchingHeader = onNonMatchingHeader || "prepend";
  templateVars = templateVars || {};
  varRegexps = varRegexps || {};
@@ -48,10 +69,7 @@ function resolveOptions({
  } else if (!(mustMatch instanceof RegExp)) {
  mustMatch = new RegExp(mustMatch);
  }
-
- if (!template && templateFile) {
- template = fs.readFileSync(templateFile, "utf8");
- }
+ template = resolveTemplate({ templateFile, template, fileName });
 
  const YEAR = new Date().getFullYear();
  const allVars = Object.assign({}, { YEAR }, templateVars);