From 5fd270c9e4e1698ddecab7a44a80273b5901290c Mon Sep 17 00:00:00 2001 From: Nadim Afana Date: Sun, 8 May 2016 19:32:41 -0700 Subject: [PATCH] Added an integration test to ensure issue does not come back in the future. --- tests/integration/jquery-calls.js | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/tests/integration/jquery-calls.js b/tests/integration/jquery-calls.js index 78d969d2..a8083b91 100644 --- a/tests/integration/jquery-calls.js +++ b/tests/integration/jquery-calls.js @@ -25,4 +25,32 @@ test('multiple elements with arguments works', function (assert) { '2', 'The call should also change the value on the second element' ); +}); + +test('initializes correctly when jQuery $.data contains cyclic reference object', function (assert) { + var $ = require('jquery'); + require('jquery.select2'); + + var $select = $( + '' + ); + + // Add a circular reference object using jQuery. + var recursiveObject = {}; + + recursiveObject['same'] = recursiveObject; + + $select.data('same', recursiveObject); + + $select.select2(); + + assert.equal( + $select.val(), + '3', + 'The option value should be pulled correctly' + ); }); \ No newline at end of file