Fix ListEditView JS tests.

Review Request #12815 — Created Jan. 27, 2023 and submitted — Latest diff uploaded

Information

Djblets
release-4.x

Reviewers

The JS tests for the ListEditView had a couple bugs. First was that we
had a mix of string characters, with both backtick strings and
single-quote strings that were concatenated. This meant that the
generated <input> element had some garbage characters in it with extra
quotes and + signs.

Second was that the default row HTML we were creating was only being passed
to the template, and not also to the view.

Ran js-tests and saw all djblets tests pass.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Fix ListEditVied JS tests.
The JS tests for the ListEditView had a couple bugs. First was that we had a mix of string characters, with both backtick strings and single-quote strings that were concatenated. This meant that the generated `<input>` element had some garbage characters in it with extra quotes and + signs. Second was that the default row HTML we were creating was only being passed to the template, and not also to the view. Testing Done: Ran js-tests and saw all djblets tests pass.
a5079560c5a0a5cfc3360638b0d1f5baed599a79 David Trowbridge
djblets/static/djblets/js/forms/views/tests/listEditViewTests.es6.js
Loading...