On 09/29/2010 12:44 AM, Endi Sukma Dewata wrote:
----- "Adam Young"<ayo...@redhat.com>  wrote:

tab objects
Convert the tab lists to arrays of objects with four potential fields:

tab[0] ->  tab.name
tab[1] ->  tab.label
tab[2] ->  tab.setup or tab.children
Added unit tests
ACK, but could you add the unit tests into the index.html and all_tests.html?
I'll add unit tests too and rebase my patch on your patch. Thanks.

diff --git a/install/static/test/all_tests.html 
b/install/static/test/all_tests.html
index 687bea1..93c4de2 100644
--- a/install/static/test/all_tests.html
+++ b/install/static/test/all_tests.html
@@ -5,15 +5,19 @@
      <link rel="stylesheet" href="qunit.css" type="text/css" media="screen">
      <script type="text/javascript" src="qunit.js"></script>
      <script type="text/javascript" src="../jquery.js"></script>
+<script type="text/javascript" src="../jquery.ba-bbq.js"></script>
+<script type="text/javascript" src="../jquery-ui.js"></script>
      <script type="text/javascript" src="../ipa.js"></script>
      <script type="text/javascript" src="../details.js"></script>
      <script type="text/javascript" src="../search.js"></script>
      <script type="text/javascript" src="../add.js"></script>
      <script type="text/javascript" src="../entity.js"></script>
      <script type="text/javascript" src="../associate.js"></script>
+<script type="text/javascript" src="../navigation.js"></script>
      <script type="text/javascript" src="ipa_tests.js"></script>
      <script type="text/javascript" src="entity_tests.js"></script>
      <script type="text/javascript" src="association_tests.js"></script>
+<script type="text/javascript" src="navigation_tests.js"></script>
  </head>
  <body>
      <h1 id="qunit-header">Complete Test Suite</h1>
diff --git a/install/static/test/index.html b/install/static/test/index.html
index 14ca7f0..581be24 100644
--- a/install/static/test/index.html
+++ b/install/static/test/index.html
@@ -27,6 +27,7 @@
          <li><a href="ipa_tests.html">Core Test Suite</a>
          <li><a href="entity_tests.html">Entity Test Suite</a>
          <li><a href="association_tests.html">Association Test Suite</a>
+<li><a href="navigation_tests.html">Navigation Test Suite</a>
          </ul>
      </div>

--
Endi S. Dewata
Yeah.  I'll do that before checkin

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to