diff --git a/frontend_tests/node_tests/list_render.js b/frontend_tests/node_tests/list_render.js index 9bab402917..9b7de71d64 100644 --- a/frontend_tests/node_tests/list_render.js +++ b/frontend_tests/node_tests/list_render.js @@ -453,3 +453,13 @@ run_test('clear_event_handlers', () => { assert.equal(scroll_container.cleared, true); assert.equal(filter_element.cleared, true); }); + +run_test('errors', () => { + // We don't care about actual data for this test. + const list = 'stub'; + const container = 'stub'; + + blueslip.expect('error', 'Need opts to create widget.'); + list_render.create(container, list); + blueslip.reset(); +}); diff --git a/static/js/list_render.js b/static/js/list_render.js index 3308be66bd..2d0cda2703 100644 --- a/static/js/list_render.js +++ b/static/js/list_render.js @@ -51,6 +51,11 @@ exports.validate_filter = (opts) => { // list: The list of items to progressively append. // opts: An object of random preferences. exports.create = function ($container, list, opts) { + if (!opts) { + blueslip.error('Need opts to create widget.'); + return; + } + if (opts.name && DEFAULTS.instances.get(opts.name)) { // Clear event handlers for prior widget. const old_widget = DEFAULTS.instances.get(opts.name); @@ -68,9 +73,6 @@ exports.create = function ($container, list, opts) { filter_value: '', }; - if (!opts) { - return; - } exports.validate_filter(opts); const widget = {};