atilaneves / cmake-ide

Use Emacs as a C/C++ IDE
BSD 3-Clause "New" or "Revised" License
716 stars 92 forks source link

Fix unescaped character literals #171

Closed jeffkowalski closed 6 years ago

jeffkowalski commented 6 years ago
coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 273


Files with Coverage Reduction New Missed Lines %
cmake-ide.el 62 68.49%
<!-- Total: 62 -->
Totals Coverage Status
Change from base Build 272: 0.0%
Covered Lines: 363
Relevant Lines: 530

💛 - Coveralls
atilaneves commented 6 years ago

Please write tests for this.

jeffkowalski commented 6 years ago

@atilaneves, Coveralls declares my change is already tested, and indeed it's covered by

(ert-deftest test-unquote ()
  (should (equal (cide--unquote "\"foo\"") "foo")))

in test\utils-test.el.

What more would you need?