This change fixes a few issues that SonarQube surfaced:

In a couple places, we were iterating over object fields without testing

hasOwnProperty.
The calls to testCloseDescription were passing too many arguments.
We were testing whether indexOf returned a value greater than zero, but

  zero is a valid return value.

Ran js-tests.