New Rule: `no-replace-test-comments`

What

• disallow ‘Replace this with your real tests’ comments

Why

• we should encourage folks to use ember-cli + generators
• for those in a rush / without strong reviews, you end up with plenty of checked in test files that only have canary tests:

  // Replace this with your real tests.
  test('it exists', function (assert) {
    let service = this.owner.lookup('service:company');
    assert.ok(service);
  });

• this rule will nudge developers to start writing a meaningful test earlier in the process

[bmish]

I actually don’t think we should have an autofixer for this. I would prefer to just update the Ember blueprint to add the // TODO: prefix in future generated comments.

[jakebixbyavalara]

Any of the folders here that have -test appended: https://github.com/emberjs/ember.js/tree/master/blueprints

Ember and Ember-Data include their own blueprints so we can change them with each respective version. The files contained in the various -files folders are the lodash templates used to generate the tests: https://github.com/emberjs/ember.js/blob/master/blueprints/controller-test/qunit-files/__root__/__testType__/__path__/__test__.js#L8