[Fix-7777-7776]: Generate valid code if comment exists before closing brace

[shekhar-patil]

closes #7777 and #7776

---

Before submitting the PR make sure the following are checked:

• Wrote good commit messages.
• Commit message starts with [Fix #issue-number] (if the related issue exists).
• Feature branch is up-to-date with master (if not - rebase it).
• Squashed related commits together.
• Added tests.
• Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
• The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
• Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

[jonas054]

Just one comment from me. Looks good otherwise!

[jonas054]

If I remove this line, all tests still pass. I suspect that this is because the call to remove is not needed, rather than being related to poor test coverage. But I don't see why it's not needed. 😕

Anyway, it's better if the method only selects a certain source range, like its name says, and not does anything else (single responsibility principle).

[shekhar-patil]

@jonas054 , Thanks for reviewing, Yes, The mentioned line is needed. I wrote the wrong test case which I have corrected now. Please review now.

[bbatsov]

Thanks!