Handle dasherized service names.

Author: Robert Jackson

__testfixtures__/ember-qunit-codemod/inject.input.js

@@ -12,3 +12,7 @@ test('it works for controllers', function(assert) {
   this.inject.controller('foo');
   this.inject.controller('foo', { as: 'bar' });
 });
+
+test('handles dasherized names', function(assert) {
+  this.inject.service('foo-bar');
+});

__testfixtures__/ember-qunit-codemod/inject.output.js

@@ -13,4 +13,8 @@ module('Unit | Service | FooBar', function(hooks) {
     this.foo = this.owner.lookup('controller:foo');
     this.bar = this.owner.lookup('controller:foo');
   });
+
+  test('handles dasherized names', function(assert) {
+    this['foo-bar'] = this.owner.lookup('service:foo-bar');
+  });
 });

ember-qunit-codemod.js

@@ -491,9 +491,16 @@ module.exports = function(file, api, options) {
             localName = as.value.value;
           }
         }
+        let property = j.identifier(localName);
+        // rudimentary attempt to confirm the property name is valid
+        // as `this.propertyName`
+        if (!localName.match(/^[a-zA-Z_][a-zA-Z0-9]+$/)) {
+          // if not, use `this['property-name']`
+          property = j.literal(localName);
+        }
         let assignment = j.assignmentExpression(
           '=',
-          j.memberExpression(j.thisExpression(), j.identifier(localName)),
+          j.memberExpression(j.thisExpression(), property),
           j.callExpression(
             j.memberExpression(
               j.memberExpression(j.thisExpression(), j.identifier('owner')),