Provide default for name option (#396)

* Automatically set name option unless already set

Use the filename to set a component's name option, if one isn't provided. This allows more helpful debugging messages by default.

* remove unnecessary object check for __vue_options__

* add test for automatic name setting based on filenames

* remove redundant if statement from loader.js

Author: chrisvfritz

lib/loader.js

@@ -180,7 +180,9 @@ module.exports = function (content) {
     // constructor export interop
     'if (typeof __vue_options__ === "function") {\n' +
     '  __vue_options__ = __vue_options__.options\n' +
-    '}\n'
+    '}\n' +
+    // add name based on filename if not already set
+    '__vue_options__.name = __vue_options__.name || "' + fileName.replace(/(\.vue|[^a-z0-9\-])/gi, '') + '"\n'
 
   // add require for template
   var template = parts.template

test/fixtures/named.vue

@@ -0,0 +1,9 @@
+<template>
+  <p>hi</p>
+</template>
+
+<script>
+export default {
+  name: 'custom-name'
+}
+</script>

test/fixtures/unnamed-5tr@ng3_f1l3n@me$.vue

@@ -0,0 +1,3 @@
+<template>
+  <p>hi</p>
+</template>

test/fixtures/unnamed.vue

@@ -0,0 +1,3 @@
+<template>
+  <p>hi</p>
+</template>

test/test.js

@@ -108,6 +108,25 @@ describe('vue-loader', function () {
     })
   })
 
+  it('automatic name setting based on filename', function (done) {
+    test({
+      entry: './test/fixtures/unnamed.vue'
+    }, function (window, module, rawModule) {
+      expect(module.name).to.equal('unnamed')
+      test({
+        entry: './test/fixtures/named.vue'
+      }, function (window, module, rawModule) {
+        expect(module.name).to.equal('custom-name')
+        test({
+          entry: './test/fixtures/unnamed-5tr@ng3_f1l3n@me$.vue'
+        }, function (window, module, rawModule) {
+          expect(module.name).to.equal('unnamed-5trng3f1l3nme')
+          done()
+        })
+      })
+    })
+  })
+
   it('pre-processors', function (done) {
     test({
       entry: './test/fixtures/pre.vue'