From 953f59941514ddc3582e51179e20eda9f124684c Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Mon, 8 Apr 2013 16:58:51 +0200 Subject: Fix unit tests https://fedorahosted.org/freeipa/ticket/3236 --- install/ui/test/aci_tests.js | 8 +- install/ui/test/details_tests.js | 25 +-- install/ui/test/entity_tests.js | 24 +-- install/ui/test/navigation_tests.js | 374 ++++++++++++++++++------------------ 4 files changed, 201 insertions(+), 230 deletions(-) (limited to 'install/ui/test') diff --git a/install/ui/test/aci_tests.js b/install/ui/test/aci_tests.js index fa1e971b1..2b8285794 100644 --- a/install/ui/test/aci_tests.js +++ b/install/ui/test/aci_tests.js @@ -101,10 +101,10 @@ module('aci', { IPA.permission_target_policy('target') ] }); - entity.add_facet('details', target_facet); - - target_container = $('
').appendTo(document.body); - target_facet.create(target_container); + entity.add_facet('details', target_facet); + target_container = $('
').appendTo(document.body); + target_facet.container_node = target_container[0]; + target_facet.create(); target_widget = target_facet.widgets.get_widget('target'); }, teardown: function() { diff --git a/install/ui/test/details_tests.js b/install/ui/test/details_tests.js index 05308be60..7326d553c 100644 --- a/install/ui/test/details_tests.js +++ b/install/ui/test/details_tests.js @@ -229,33 +229,20 @@ test("Testing details lifecycle: create, load.", function(){ }); var entity = IPA.get_entity('user'); - - var entity_container = $('
', { - name: 'user', - title: 'User', - 'class': 'entity' - }).appendTo(details_container); - - entity.create(entity_container); - + var container = $('
', {}).appendTo(details_container); var facet = entity.get_facet('details'); - - var facet_container = $('
', { - name: facet.name, - 'class': 'facet' - }); - - facet.create(facet_container); + facet.container_node = container[0]; + facet.create(); facet.load(data); - var contact = $('.details-section[name=contact]', facet_container); + var contact = $('.details-section[name=contact]', facet.domNode); ok( contact.length, 'Verifying section for contact is created'); - var identity = $('.details-section[name=identity]', facet_container); + var identity = $('.details-section[name=identity]', facet.domNode); ok( identity.length, @@ -267,8 +254,6 @@ test("Testing details lifecycle: create, load.", function(){ rows.length, 6, 'Verifying rows for identity'); - facet_container.attr('id','user'); - ok (load_called, 'load manager called'); var field = facet.fields.get_field('test'); diff --git a/install/ui/test/entity_tests.js b/install/ui/test/entity_tests.js index f34060f79..26d73090c 100644 --- a/install/ui/test/entity_tests.js +++ b/install/ui/test/entity_tests.js @@ -23,7 +23,7 @@ define(['freeipa/ipa', 'freeipa/jquery', 'freeipa/entity', 'freeipa/search', 'freeipa/details'], function(IPA, $) { return function() { -var entities_container; +var container; module('entity',{ setup: function() { @@ -54,11 +54,11 @@ module('entity',{ } }); - entities_container = $('
').appendTo(document.body); + container = $('
').appendTo(document.body); }, teardown: function() { - entities_container.remove(); + container.remove(); } }); @@ -70,23 +70,9 @@ test('Testing IPA.entity_set_search_definition().', function() { }; var entity = IPA.get_entity('user'); - - var entity_container = $('
', { - name: 'user', - title: 'User', - 'class': 'entity' - }).appendTo(entities_container); - - entity.create(entity_container); - var facet = entity.get_facet('search'); - - var facet_container = $('
', { - name: facet.name, - 'class': 'facet' - }); - - facet.create(facet_container); + facet.container_node = container[0]; + facet.create(); var column = facet.get_columns()[0]; ok( diff --git a/install/ui/test/navigation_tests.js b/install/ui/test/navigation_tests.js index 309a0a634..797cc2e1c 100644 --- a/install/ui/test/navigation_tests.js +++ b/install/ui/test/navigation_tests.js @@ -24,202 +24,202 @@ define(['freeipa/ipa', 'freeipa/jquery', 'freeipa/navigation', 'freeipa/entity'] module('navigation', { - setup: function() { - IPA.ajax_options.async = false; - IPA.init({ - url: 'data', - on_error: function(xhr, text_status, error_thrown) { - ok(false, 'ipa_init() failed: '+error_thrown); - } - }); - } +// setup: function() { +// IPA.ajax_options.async = false; +// IPA.init({ +// url: 'data', +// on_error: function(xhr, text_status, error_thrown) { +// ok(false, 'ipa_init() failed: '+error_thrown); +// } +// }); +// } }); test("Testing IPA.navigation.create().", function() { - var entity; - var user_mock_called = false; - var group_mock_called = false; - //Force reset of entities - IPA.entities = $.ordered_map(); - - IPA.register('user', function(spec) { - - var that = IPA.entity({ - name: 'user', - metadata: IPA.metadata.objects.user, - facets: [ - { - type: 'search' - } - ] - }); - - that.display = function(container){ - user_mock_called = true; - same(container.attr('name'), 'user', 'user container name'); - same(container[0].nodeName, 'DIV', 'user container element'); - }; - - return that; - }); - - IPA.register('group', function(spec) { - - var that = IPA.entity({ - name: 'group', - metadata: IPA.metadata.objects.group - }); - - that.display = function(container){ - group_mock_called = true; - same(container.attr('name'), 'group','user container name'); - same(container[0].nodeName, 'DIV', 'user container element'); - }; - - return that; - }); - - var navigation_container = $('