Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[aspectj-dev] finding fixed files for each bug from bugzilla for aspectj

Hello,
I am using the export facility of the bugzilla, where one can export information about bugs. I need the fixed files also along with all the other information. But there is no way to extract that

Here is an example bug from the bugzilla repository, where is the fixed file list encoded? Even though the bug is "fixed" or "resolved" i do not see that information in an xml exported output by bugzilla.

[CODE]
<bug>
          <bug_id>43030</bug_id>
         
          <creation_ts>2003-09-12 12:59:00 -0400</creation_ts>
          <short_desc>unit tests failing due to misuse of getAbsolutePath()</short_desc>
          <delta_ts>2003-09-17 03:06:00 -0400</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Tools</classification>
          <product>AspectJ</product>
          <component>IDE</component>
          <version>1.1.1</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
         
         
          <bug_file_loc/>
          <status_whiteboard/>
          <keywords/>
          <priority>P1</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
         
         
          <everconfirmed>1</everconfirmed>
          <reporter name="Jim Hugunin">jim-aj</reporter>
          <assigned_to name="Adrian Colyer">adrian.colyer</assigned_to>
         
         
          <votes>0</votes>

     

     

     

          <long_desc isprivate="0">
            <commentid>185952</commentid>
            <who name="Jim Hugunin">jim-aj</who>
            <bug_when>2003-09-12 12:59:40 -0400</bug_when>
            <thetext>Many of the unit tests in org.aspectj.ajdt.ajc.BuildArgParserTestCase and in
org.aspectj.ajde.StructureModelTest are failing for me.  These failures appear
to be due to the behavior of File.getAbsolutePath().  My suspicion is that these
failures are caused by recent changes to the structure model code to remove
calls to File.getCanonicalPath().

These failures all come under Win XP using either jdk 1.4.2 or jdk 1.3.x from
SUN.  To reproduce the failures, you need to launch eclipse using "-data
c:/path_to_workspace".  Note the lowercase 'c' in the path.  If I change this to
"-data C:/path_to_workspace", then all of the tests will pass.


Here are the traces from two of the failures:

java.lang.IndexOutOfBoundsException: Index: 2, Size: 1
    at java.util.ArrayList.RangeCheck(ArrayList.java:486)
    at java.util.ArrayList.get(ArrayList.java:302)
    at
org.aspectj.ajde.StructureModelTest.testPointcutName(StructureModelTest.java:93)
-----------------------------------------
junit.framework.ComparisonFailure: expected:&lt;c...&gt; but was:&lt;C...&gt;
    at junit.framework.Assert.assertEquals(Assert.java:81)
    at junit.framework.Assert.assertEquals(Assert.java:87)
    at
org.aspectj.ajdt.ajc.BuildArgParserTestCase.testSourceRootDirWithFiles(BuildArgParserTestCase.java:221)

The filenames that are failing to compare above differ only in the initial
capitalization of the 'c', e.g.
c:\aspectj\workspace\org.aspectj.ajdt.core\testdata\ajc\pkg
    and
C:\aspectj\workspace\org.aspectj.ajdt.core\testdata\ajc\pkg</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>185953</commentid>
            <who name="Jim Hugunin">jim-aj</who>
            <bug_when>2003-09-12 13:01:51 -0400</bug_when>
            <thetext>I'm raising this to a P1 bug because someone needs to investigate this bug
before the 1.1.1 release.  It's possibly just an issue with the test code;
however, it might also indicate a subtle and dangerous bug in the structure
model code.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>186043</commentid>
            <who name="Adrian Colyer">adrian.colyer</who>
            <bug_when>2003-09-12 15:00:18 -0400</bug_when>
            <thetext>I confirm I can reproduce the failures. I've investigated all the
BuildArgParserTestCase ones and they were harmless, but easy to fix - I hate
failing test cases.

Now looking at the StructureModelTest failures - potentially more serious.

(Just appending progress to save anyone else duplicating work).</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>186059</commentid>
            <who name="Adrian Colyer">adrian.colyer</who>
            <bug_when>2003-09-12 15:37:21 -0400</bug_when>
            <thetext>Now fixed. The StructureModel test cases were indeed hiding a more subtle and
serious bug - findNodeForSourceFile and findNodeForSourceLine were using
absolute path instead of canonical path to look for matches - and so not finding
corresponding nodes unless the path happened to match. This is the same problem
that caused Andy to lose his outline view from AJDT.

I changed three files. If anyone has the time to buddy-check my changes
(sensible precaution this close to ship) I'd appreciate it. The files are:

org.aspectj.asm.internal.AspectJElementHierarchy
org.aspectj.ajdt.ajc.BuildArgParserTestCase
org.aspectj.util.ConfigParser</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>186060</commentid>
            <who name="Adrian Colyer">adrian.colyer</who>
            <bug_when>2003-09-12 15:37:55 -0400</bug_when>
            <thetext>-</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>186741</commentid>
            <who name="Mik Kersten">mik.kersten</who>
            <bug_when>2003-09-17 03:06:00 -0400</bug_when>
            <thetext>At one point in the past those find methodhs did use canonical paths, but my
over-optimistic migration to absolute paths changed that. 

I reviewed these changes and they are all correct.  </thetext>
          </long_desc>
     
[/CODE]

Shivani

--
Research Scholar,
School of Electrical and Computer Engineering
Purdue University
West Lafayette IN
web.ics.purdue.edu/~sgrao

Back to the top