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