Merge pull request #30 from simonihmig/fix-empty-jq-selector

Skip converting :eq() and :selected without a selector

Author: simonihmig

transforms/integration/__testfixtures__/jq-extensions.input.js

@@ -33,12 +33,14 @@ test('it renders', function(assert) {
   assert.ok(this.$('.foo:radio').length);
   assert.ok(this.$('.foo:reset').length);
   assert.ok(this.$('.foo:selected').length);
+  assert.ok(this.$(':selected').length);
   assert.ok(this.$('.foo:submit').length);
   assert.ok(this.$('.foo:text').length);
   assert.ok(this.$('.foo:visible').length);
   assert.ok(this.$(JQEXTENSION_SELECTOR_AS_LOCAL_CONST).length);
   assert.ok(this.$(ANY_SELECTOR_AS_IMPORTED_CONST).length);
 
+  assert.ok(this.$(':eq(0)').length);
   assert.ok(this.$('.foo:eq(0)').length);
   assert.ok(this.$('.foo:eq(1)').length);
   assert.ok(this.$('.foo:first-child').length);

transforms/integration/__testfixtures__/jq-extensions.output.js

@@ -34,12 +34,14 @@ test('it renders', function(assert) {
   assert.ok(this.$('.foo:radio').length);
   assert.ok(this.$('.foo:reset').length);
   assert.ok(findAll('.foo:checked').length);
+  assert.ok(this.$(':selected').length);
   assert.ok(this.$('.foo:submit').length);
   assert.ok(this.$('.foo:text').length);
   assert.ok(this.$('.foo:visible').length);
   assert.ok(this.$(JQEXTENSION_SELECTOR_AS_LOCAL_CONST).length);
   assert.ok(this.$(ANY_SELECTOR_AS_IMPORTED_CONST).length);
 
+  assert.ok(this.$(':eq(0)').length);
   assert.ok(findAll(find('.foo')).length);
   assert.ok(findAll('.foo')[1].length);
   assert.ok(findAll('.foo:first-child').length);

transforms/utils.js

@@ -28,8 +28,8 @@ const jqExtensions = [
 ];
 
 const supportedJqExtensions = [
-  /:eq/,
-  /:selected/,
+  /.+:eq/,
+  /.+:selected/,
 ];
 
 const tailEqRegex = /:eq\(\d+\)$/;