diff --git a/Uri/UriFactory.js b/Uri/UriFactory.js index b5c533f..6cfd4e1 100644 --- a/Uri/UriFactory.js +++ b/Uri/UriFactory.js @@ -45,7 +45,7 @@ * * @param {string} key * - * @return {string} + * @return {null|string} * * @method * @@ -53,11 +53,7 @@ */ static getQuery (key) { - if (!jsOMS.Uri.UriFactory.uri.hasOwnProperty(key)) { - return ''; - } - - return jsOMS.Uri.UriFactory.uri[key]; + return jsOMS.Uri.UriFactory.uri.hasOwnProperty(key) ? jsOMS.Uri.UriFactory.uri[key] : null; }; /** diff --git a/tests/SpecRunner.html b/tests/SpecRunner.html index 144f7b6..b06158e 100644 --- a/tests/SpecRunner.html +++ b/tests/SpecRunner.html @@ -4,25 +4,25 @@ Framework Test Suit - - + + - - - + + + - + - - + + - - + + - + diff --git a/tests/Uri/UriFactoryTest.js b/tests/Uri/UriFactoryTest.js index 01b01a7..504c16a 100644 --- a/tests/Uri/UriFactoryTest.js +++ b/tests/Uri/UriFactoryTest.js @@ -41,6 +41,11 @@ describe('UriFactoryTest', function () { it('Testing query clearing', function () { + jsOMS.Uri.UriFactory.setQuery('Valid', 'query1'); + jsOMS.Uri.UriFactory.setQuery('Valid', 'query2', true); + jsOMS.Uri.UriFactory.setQuery('Valid', 'query3', false); + jsOMS.Uri.UriFactory.setQuery('/valid2', 'query4'); + expect(jsOMS.Uri.UriFactory.clear('Valid')).toBeTruthy(); expect(jsOMS.Uri.UriFactory.clear('Valid')).toBeFalsy(); expect(jsOMS.Uri.UriFactory.getQuery('Valid')).toBe(null);