Improve testing.

Handle cases when for gadget_data_source and gadget_data_handler we have
empty strings.

OfficeJS/src/renderjs/renderjs.js

@@ -150,7 +150,7 @@ var RenderJs = (function () {
                     data_source = gadget.attr("data-gadget:data-source");
                     data_handler = gadget.attr("data-gadget:data-handler");
                     // acquire data and pass it to method handler
-                    if (data_source!==undefined){
+                    if (data_source!==undefined && data_source!==""){
                       $.ajax({url:data_source,
                               dataType: "json",
                               yourCustomData: {"data_handler": data_handler},

OfficeJS/src/renderjs/test/renderjs_test.html

@@ -14,8 +14,7 @@
     <h1 id="qunit-header">QUnit RenderJS test suite</h1>
     <h2 id="qunit-banner"></h2>
     <h2 id="qunit-userAgent"></h2>
-    <ol id="qunit-tests">
-    </ol>
+    <ol id="qunit-tests"></ol>
     <div id="qunit-fixture"> </div>
 
     <script type="text/javascript">

OfficeJS/src/renderjs/test/renderjs_test.js

@@ -12,14 +12,6 @@ function setupRenderJSTest(){
                 deepEqual(data, RenderJs.Cache.get(cache_id));
   });
 
- 
-  module("TabularGadget");
-   test('addNewTabGadget', function(){
-               RenderJs.TabbularGadget.addNewTabGadget("qunit-fixture", "test-gadget.xhtml");
-               equal($("#qunit-fixture").children(".gadget").length, 1);
-               equal(RenderJs.GadgetIndex.getGadgetList().length, 1);
-   });
-
   module("GadgetIndex");
   test('GadgetIndex', function(){
                // re-init GadgetIndex
@@ -37,12 +29,19 @@ function setupRenderJSTest(){
               equal($("#qunit-fixture").attr("id"), RenderJs.GadgetIndex.getRootGadget().getDom().attr("id"));
               equal(RenderJs.GadgetIndex.getGadgetById("qunit-fixture"), RenderJs.GadgetIndex.getRootGadget());
 
-              // unregister gadget
+              // unregister gadget all gadgets from this test not to mess with rest of tests
               RenderJs.GadgetIndex.unregisterGadget(RenderJs.GadgetIndex.getGadgetById("qunit-fixture"));
               equal(RenderJs.GadgetIndex.getGadgetList().length, 1);
               equal(RenderJs.GadgetIndex.getGadgetById("new"), RenderJs.GadgetIndex.getRootGadget());
+              RenderJs.GadgetIndex.unregisterGadget(RenderJs.GadgetIndex.getGadgetById("new"));
+              equal(RenderJs.GadgetIndex.getGadgetList().length, 0);
+   });
 
-
+   module("TabularGadget");
+   test('addNewTabGadget', function(){
+               RenderJs.TabbularGadget.addNewTabGadget("qunit-fixture", "test-gadget.html", "", "");
+               equal($("#qunit-fixture").children(".gadget").length, 1);
+               equal(RenderJs.GadgetIndex.getGadgetList().length, 1);
    });
 
 };

OfficeJS/src/renderjs/test/test-gadget.html

+test!!!
\ No newline at end of file

OfficeJS/src/renderjs/test/test-gadget.xhtml

-test!!!
-<div id="test-container">
-</div>
\ No newline at end of file