From 63f7575cb38982575342a6856b7d322c14fc83f6 Mon Sep 17 00:00:00 2001
From: Pavel Vomacka <pvomacka@redhat.com>
Date: Aug 31 2017 08:35:03 +0000
Subject: Fixes for aci_tests.js


Aci_widget doesn't use table elements anymore. Whole widget is created from
list element. Change in tests needed.

https://pagure.io/freeipa/issue/6974

Reviewed-By: Petr Vobornik <pvoborni@redhat.com>

---

diff --git a/install/ui/test/aci_tests.js b/install/ui/test/aci_tests.js
index cdc8aeb..a821949 100644
--- a/install/ui/test/aci_tests.js
+++ b/install/ui/test/aci_tests.js
@@ -152,17 +152,19 @@ test("aci.attributes_widget", function() {
 
     widget.create(container);
 
-    var table = $('table', container);
+    var list = $('ul', container);
 
     ok(
-        table,
-        'Widget contains table');
+        list,
+        'Widget contains list');
 
-    widget.update({});
-    var tr = $('tbody tr', table);
+    widget.update([]);
+
+    list = $('ul', container); // reload the DOM node which contains options
+    var li = $('li', list);
 
     same(
-        tr.length, aciattrs.length,
+        li.length, aciattrs.length,
         'Widget contains all user ACI attributes');
 
     var record = {
@@ -175,14 +177,15 @@ test("aci.attributes_widget", function() {
 
     same(
         widget.save(), [],
-        'Widget has no initial values');
+        'Widget has no initialy checked values');
 
     widget.update(record.attrs);
 
-    tr = $('tbody tr', table);
+    list = $('ul', container); // reload the DOM node which contains options
+    li = $('li', list);
 
     same(
-        tr.length, aciattrs.length+1,
+        li.length, aciattrs.length+1,
         'Widget contains all user ACI attributes plus 1 unmatched attribute');
 
     same(