Hi Poornima,<div><br></div><div>Generic I meant, because the test doesn&#39;t have any report at end, like how any unit test framework does. Consider creating screenshot incase of test failure. If everything goes fine, the test will not quit Firefox, I think.</div>

<div><br></div><div>Still some of the window names or object names have hard coded text, they are not reused from common variables.</div><div><br></div><div>Thanks</div><div>Nagappan</div><div><br><div class="gmail_quote">

On Mon, Apr 16, 2012 at 6:59 AM, Poornima Nayak <span dir="ltr">&lt;<a href="mailto:mpnayak@linux.vnet.ibm.com">mpnayak@linux.vnet.ibm.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Incorporated all the comments given by Nagappan. Modified header of all<br>
the files.<br>
<a href="https://github.com/PoornimaNayak/gnomeautomation" target="_blank">https://github.com/PoornimaNayak/gnomeautomation</a><br>
<div class="HOEnZb"><div class="h5">On Sat, 2012-04-14 at 22:35 +0530, Poornima Nayak wrote:<br>
&gt; On Wed, 2012-04-11 at 19:24 -0700, Nagappan Alagappan wrote:<br>
&gt; &gt; Hi Poornima,<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; I&#39;m not convinced with firefox.py. I see the following issues:<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; * its not generic<br>
&gt; I missed this line while replying earlier. Not very sure what you meant<br>
&gt; by &#39;not generic&#39;<br>
&gt; &gt; * It doesn&#39;t handle any exception, say if you have test script 1 to<br>
&gt; &gt; 10, if the test fails for some reason and throws any exception in test<br>
&gt; &gt; 2, tests starting 3 to 10 won&#39;t be executed<br>
&gt; &gt; * The test doesn&#39;t put back to the old state, I mean if some exception<br>
&gt; &gt; happens, all the current window should be closed and should be smooth<br>
&gt; &gt; for the next test<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; Other comments from IRC:<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; * Create a readme file with version of Firefox, Linux distribution<br>
&gt; &gt; * Add respective license / author files<br>
&gt; &gt; * Move &#39;frm*MozillaFirefox&#39; and other object names to some common<br>
&gt; &gt; file, so changing things will be at one place, will be really helpful<br>
&gt; &gt; at later point, if you are planing for i18n / l10n. Just change that<br>
&gt; &gt; file content with appropriate locale text and your scripts are ready<br>
&gt; &gt; for localization testing<br>
&gt; &gt; * time module imported couple of times in navigation.py<br>
&gt; &gt; * Comment navigation.py line#57, verifyabout.py line#46,<br>
&gt; &gt; verifynewtab.py line#40<br>
&gt; &gt; * Use wait instead of time.sleep<br>
&gt; &gt; * Additional space between url and = in verifynewtab.py<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; Additional comments:<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; * Don&#39;t use space between the function name and open bracket, ex:<br>
&gt; &gt; instead of &#39;log (&#39; use &#39;log(&#39;<br>
&gt; &gt; * Maybe you can use &#39;if not waittillguinotexist(&#39;window&#39;)&#39; instead<br>
&gt; &gt; of &#39;if waittillguinotexist(&#39;window&#39;) == 0&#39;<br>
&gt; &gt; * At end of the complete test execution, application should exit<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; Thanks<br>
&gt; &gt; Nagappan<br>
&gt; &gt;<br>
&gt; &gt; On Wed, Apr 11, 2012 at 12:06 PM, Poornima Nayak<br>
&gt; &gt; &lt;<a href="mailto:mpnayak@linux.vnet.ibm.com">mpnayak@linux.vnet.ibm.com</a>&gt; wrote:<br>
&gt; &gt;         Hi All<br>
&gt; &gt;<br>
&gt; &gt;         Sheetal &amp; I work in IBM Linux Technology Center, we are<br>
&gt; &gt;         testing<br>
&gt; &gt;         Accessibility. We are using LDTP test script written for<br>
&gt; &gt;         automating<br>
&gt; &gt;         Firefox testing. These scripts where written based on Firefox<br>
&gt; &gt;         version in<br>
&gt; &gt;         2004/2006 and LDTP supported API&#39;s in 2004/2006.<br>
&gt; &gt;         So we had to do lot of modification and come out with some<br>
&gt; &gt;         work arounds.<br>
&gt; &gt;<br>
&gt; &gt;         Nagappan already suggested lot of changes, those will be done.<br>
&gt; &gt;         I specially need comment on firefox.py which triggeres test<br>
&gt; &gt;         one by one.<br>
&gt; &gt;<br>
&gt; &gt;         I have uploaded scripts at<br>
&gt; &gt;         <a href="https://github.com/PoornimaNayak/gnomeautomation" target="_blank">https://github.com/PoornimaNayak/gnomeautomation</a>.<br>
&gt; &gt;<br>
&gt; &gt;         We are planning to modify around 23 more Firefox test scripts<br>
&gt; &gt;         and<br>
&gt; &gt;         publish in the above repository after we learn the expectation<br>
&gt; &gt;         from LDTP<br>
&gt; &gt;         maintainer and the community.<br>
&gt; &gt;<br>
&gt; &gt;         Regards<br>
&gt; &gt;         Poornima<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;         _______________________________________________<br>
&gt; &gt;         LDTP-dev mailing list<br>
&gt; &gt;         <a href="mailto:LDTP-dev@lists.freedesktop.org">LDTP-dev@lists.freedesktop.org</a><br>
&gt; &gt;         <a href="http://lists.freedesktop.org/mailman/listinfo/ldtp-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/ldtp-dev</a><br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; --<br>
&gt; &gt; Linux Desktop (GUI Application) Testing Project -<br>
&gt; &gt; <a href="http://ldtp.freedesktop.org" target="_blank">http://ldtp.freedesktop.org</a><br>
&gt; &gt; <a href="http://nagappanal.blogspot.com" target="_blank">http://nagappanal.blogspot.com</a><br>
&gt; &gt;<br>
&gt;<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; LDTP-dev mailing list<br>
&gt; <a href="mailto:LDTP-dev@lists.freedesktop.org">LDTP-dev@lists.freedesktop.org</a><br>
&gt; <a href="http://lists.freedesktop.org/mailman/listinfo/ldtp-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/ldtp-dev</a><br>
&gt;<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Linux Desktop (GUI Application) Testing Project - <a href="http://ldtp.freedesktop.org">http://ldtp.freedesktop.org</a><br><a href="http://nagappanal.blogspot.com">http://nagappanal.blogspot.com</a><br>


</div>