After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 345723 - Block the running script if it error.
Block the running script if it error.
Status: RESOLVED FIXED
Product: LDTP
Classification: Other
Component: ldtp
0.4.x
Other All
: Normal blocker
: ---
Assigned To: LDTP maintainers
LDTP Development Mailing List
Depends on:
Blocks:
 
 
Reported: 2006-06-23 07:44 UTC by lavi.xu
Modified: 2006-08-01 05:23 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to have relative file path in log (1.17 KB, patch)
2006-07-31 20:01 UTC, Harishankaran 		none Details | Review
ldtprunner file expand patch (2.50 KB, patch)
2006-08-01 04:57 UTC, Nagappan Alagappan 		committed Details | Review

Description lavi.xu 2006-06-23 07:44:17 UTC 
Please describe the problem:
If script raises a exception and doesnt dispose own, the runing script will be blocked.

Steps to reproduce:
1. write a script case000.py:
raise 'raises a expection'

2. writes a xml test.xml(in the same folder):
<?xml version="1.0"?>
<ldtp>
	<logfileoverwrite>1</logfileoverwrite>
	<logfile>~/loggedit.xml</logfile>
	<group>
		<script>
			<name>case000.py</name>
		</script>
	</group>
</ldtp>

3. run the command under the CLI
 gldap.py test.xml


Actual results:
the script never return

Expected results:
log the exception to log file

Does this happen every time?
every

Other information:
the laster version checked by cvs -- 062306
Comment 1 Harishankaran 2006-07-31 20:01:50 UTC 
Created attachment 69998 [details] [review]
Patch to have relative file path in log
Comment 2 lavi.xu 2006-08-01 01:13:04 UTC 
Running the scritp can continuous be deal with by ldtprunner. 

But, in xml file:
============================================
<?xml version="1.0"?>
<ldtp>
        <logfileoverwrite>1</logfileoverwrite>
        <logfile>./loggedit.xml</logfile>
        <group>
                <script>
                        <name>~/work/gedit/temp/test.py</name>
                </script>
        </group>
</ldtp>
============================================
The script file 'test.py' can not be recognized yet.

Check again plz.

Thank.
Comment 3 Nagappan Alagappan 2006-08-01 04:57:08 UTC 
Created attachment 70013 [details] [review]
ldtprunner file expand patch
Comment 4 Nagappan Alagappan 2006-08-01 05:23:59 UTC 
Lavi: Fixed in CVS. If you still have any issues, please reopen the bug.