When deleting a for_topic bookmark, we were calling bookmark.attachedTo() for the bookmarks:changed event, but the bookmark was not always a Bookmark model instance, so sometimes that call would error. Now we make sure that the bookmarks in the topic.bookmarks JS array are all bookmark model instances, and added a test to cover this deleting for_topic bookmark case as well. |
||
|---|---|---|
| .. | ||
| acceptance | ||
| fixtures | ||
| helpers | ||
| integration | ||
| unit | ||
| core_plugins_tests.js | ||
| index.html | ||
| plugin_tests.js.erb | ||
| setup-tests.js | ||
| test_helper.js | ||
| test_starter.js | ||
| test-helper.js | ||
| theme_qunit_ember_jquery.js | ||
| theme_qunit_helper.js | ||
| theme_qunit_tests_vendor.js | ||
| theme_qunit_vendor.js | ||