[test] follow npm recommendations

nodejitsu · Mar 14, 2013 · 4b8ca0c · 4b8ca0c
1 parent e9765aa
commit 4b8ca0c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/jitsu/package.js b/lib/jitsu/package.js
@@ -541,7 +541,7 @@ package.properties = function (dir) {
       name: 'name',
       unique: true,
       message: 'App name',
-      validator: /^[\w|\-|\.]+$/,
+      validator: /^[A-Za-z0-9][\w|\-|\.]+$/,
       default: path.basename(dir)
     },
     {

diff --git a/test/lib/package-test.js b/test/lib/package-test.js
@@ -51,7 +51,14 @@ var suite = vows.describe('jitsu/package').addBatch({
       topic: setupPackage({ name: '.example-app' }),
 
       'should be invalid': function (topic) {
-        assert.ok(isValid('name', topic.name));
+        assert.ok(!isValid('name', topic.name));
+      }
+    },
+    'starting with _': {
+      topic: setupPackage({ name: '_example-app' }),
+
+      'should be invalid': function (topic) {
+        assert.ok(!isValid('name', topic.name));
       }
     },
     'containing -': {