Removed hello-world rule. Bumped version to 0.1.0. Refactored tests to use eslint-tester.

Author: benmosher

files/foo.js

@@ -0,0 +1 @@
+var foo = "foo";

files/tall.coffee

@@ -1,4 +1,7 @@
 exports.rules = {
-	"test-import-visitor": require("./lib/rules/test-import-visitor"),
   "valid-relative-path": require("./lib/rules/valid-relative-path")
 };
+
+exports.rulesConfig = {
+  "valid-relative-path": [2, [".js", ".coffee", ".es6"]]
+};

index.js

@@ -21,15 +21,20 @@ function resolveImportPath(context, importPath) {
   return path.resolve(path.dirname(context.getFilename()), importPath);
 }
 
-function exists(resolvedPath) {
-  return EXTENSIONS.some(function (ext) { return fs.existsSync(resolvedPath + ext); });
+function exists(resolvedPath, extensions) {
+  return extensions.some(function (ext) { return fs.existsSync(resolvedPath + ext); });
+}
+
+function extensionsFromOptions(options) {
+  return options[0];
 }
 
 module.exports = function (context) {
+  var extensions = extensionsFromOptions(context.options) || EXTENSIONS;
   return {
     "ImportDeclaration": function (node) {
       var importPath = node.source.value;
-      if (isRelative(importPath) && !exists(resolveImportPath(context, importPath))) {
+      if (isRelative(importPath) && !exists(resolveImportPath(context, importPath), extensions)) {
         context.report(node, "Imported file does not exist.");
       }
     }

lib/rules/test-import-visitor.js

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-import",
-  "version": "0.0.1",
+  "version": "0.1.0",
   "description": "Import with sanity.",
   "main": "index.js",
   "directories": {
@@ -31,5 +31,8 @@
     "eslint-tester": "^0.6.0",
     "mocha": "^2.2.1",
     "object-assign": "^2.0.0"
+  },
+  "peerDependencies": {
+    "eslint": ">=0.16.0"
   }
 }

lib/rules/valid-relative-path.js

@@ -1,62 +1,47 @@
 "use strict";
 
 var assign = require("object-assign");
-
-var expect = require("chai").expect;
-
 var path = require("path");
 
 var linter = require("eslint").linter,
-    CLIEngine = require("eslint").CLIEngine,
     ESLintTester = require("eslint-tester");
 
 var eslintTester = new ESLintTester(linter);
 
-function valid(c) {
-  return {
-    code: c,
-    ecmaFeatures: { modules: true}
-  };
-}
 
-function invalid(c, errors) {
-  return assign(valid(c), {errors: errors});
+var ecmaFeatures = {ecmaFeatures: { modules: true}};
+
+function filename(f) {
+  return path.join(process.cwd(), "./files", f);
 }
 
 eslintTester.addRuleTest("lib/rules/valid-relative-path", {
   valid: [
-    valid("import foo from 'bar';")
+    assign({
+      code: "import * as foo from 'bar';" // not relative
+    }, ecmaFeatures),
+    assign({
+      code: "import foo from './bar';",
+      filename: filename("foo.js")
+    }, ecmaFeatures),
+    assign({
+      code: "import bar from './bar.js';",
+      args: [1, [""]],
+      filename: filename("foo.js"),
+      errors: [{message: "Imported file does not exist."}]
+    }, ecmaFeatures)
   ],
 
   invalid: [
-    invalid("import foo from './bar/baz'",
-      [ { message: "Imported file does not exist." } ])
+    assign({
+      code: "import bar from './bar.js';",
+      filename: filename("foo.js"),
+      errors: [{message: "Imported file does not exist."}]
+    }, ecmaFeatures),
+    assign({
+      code: "import bar from './baz';",
+      filename: filename("foo.js"),
+      errors: [{message: "Imported file does not exist."}]
+    }, ecmaFeatures)
   ]
 });
-
-function executeOnFile(filename) {
-  return this.executeOnFiles([path.join(process.cwd(), "./files", filename)]);
-}
-
-function checkErrorMessage(report, message) {
-  return report.results[0].messages.some(function (result) { return result.message === message; });
-}
-
-describe("valid-relative-path: against actual files", function () {
-  var cli = new CLIEngine({
-    rulePaths: [path.resolve(process.cwd(), "lib/rules")],
-    rules: { "valid-relative-path": 2 }
-  });
-  cli.executeOnFile = executeOnFile;
-
-  it("should follow relative paths.", function () {
-    var report = cli.executeOnFiles([path.join(__dirname, "../../../files/foo.js")]);
-    expect(report.errorCount).to.equal(0);
-  });
-
-  it("should fail for non-existent imports.", function () {
-    var report = cli.executeOnFile("bar.js");
-    expect(report.errorCount).to.equal(1);
-    expect(checkErrorMessage(report, "Imported file does not exist.")).to.be.true;
-  });
-});