fix(config): normalizeConfig broke

NullVoxPopuli · Mar 27, 2020 · 8988541 · 8988541
1 parent 48c2eea
commit 8988541
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/lib/rules/decorator-position.js b/lib/rules/decorator-position.js
@@ -179,6 +179,9 @@ function positionDecorator(context, node, options) {
 
 function placeDecoratorsBesideProperty(context, node, options) {
   for (const decoratorConfig of options.overrides[PREFER_INLINE]) {
+    if (!decoratorConfig) {
+      continue;
+    }
     const config = normalizeConfig(decoratorConfig, INTENT.SAME_LINE);
     const info = decoratorInfo(node, config);
 
@@ -206,6 +209,9 @@ function placeDecoratorsBesideProperty(context, node, options) {
 
 function placeDecoratorsAboveProperty(context, node, options) {
   for (const decoratorConfig of options.overrides[ABOVE]) {
+    if (!decoratorConfig) {
+      continue;
+    }
     const config = normalizeConfig(decoratorConfig, INTENT.DIFFERENT_LINES);
     const info = decoratorInfo(node, config);
 

diff --git a/smoke-tests/ember/general.js b/smoke-tests/ember/general.js
@@ -13,4 +13,9 @@ export class Foo {
   @hasMany('relation', {
     someConfig: true,
   }) relationship;
+
+  @(task(function* () {
+    // some task
+  }).drop())
+  someTask;
 }