Chore: Move comment to make tests more organized

[magurotuna]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[X] Other, please explain: Move a comment in the getter-return test, so that the orders of tests will make sense.

What changes did you make? (Give an overview)

Currently, a part of the getter-return tests look like:

eslint/tests/lib/rules/getter-return.js

Lines 221 to 226 in 30b76c9

	{ code: "Object.defineProperties(foo, { bar: { get: function () {}} });", options, errors: [{ messageId: "expected" }] },
	{ code: "Object.defineProperties(foo, { bar: { get: function (){if(bar) {return true;}}}});", options, errors: [{ messageId: "expectedAlways" }] },
	{ code: "Object.defineProperties(foo, { bar: { get: function () {~function () { return true; }()}} });", options, errors: [{ messageId: "expected" }] },
	// option: {allowImplicit: true}
	{ code: "Object.defineProperty(foo, \"bar\", { get: function (){}});", options, errors: [{ messageId: "expected" }] },

Seeing this, it is quite natural that we think {allowImplicit: true} is applied only to line 226. But in fact, {allowImplicit: true} is also applied to line 221-223 - it's really confusing.

So I have moved this comment up, which indicates that this comment is applied to these 4 test cases.

Is there anything you'd like reviewers to focus on?

no

[jsf-clabot]

All committers have signed the CLA.

[mdjermanovic]

Thanks for the PR, nice catch!

[mdjermanovic]

LGTM, thanks!

[mdjermanovic]

Thanks for contributing!