Skip to content

Commit

Permalink
Fix expected weave messages after previous changes
Browse files Browse the repository at this point in the history
Remove leading "weaveInfo", add missing "see also:" to weave messages.

Relates to #218.

Signed-off-by: Alexander Kriegisch <[email protected]>
  • Loading branch information
kriegaex committed Apr 13, 2024
1 parent a1a700f commit afc8261
Show file tree
Hide file tree
Showing 6 changed files with 142 additions and 130 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@
options="-1.5"/>
<run class="ataspectj.DumpTest" ltw="ataspectj/aop-dump.xml"/>
</ajc-test>

<ajc-test dir="java5/ataspectj" title="LTW DumpTest before and after">
<compile
files="ataspectj/EmptyAspect.aj"
Expand Down Expand Up @@ -163,7 +163,7 @@
files="ataspectj/ltwlog/Aspect1.java"
options="-1.5 -XterminateAfterCompilation"
>
</compile>
</compile>
<run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-silent.xml">
<stdout>
<line text="execution(Main.target())"/>
Expand All @@ -182,7 +182,7 @@
files="ataspectj/ltwlog/Aspect1.java"
options="-1.5 -XterminateAfterCompilation"
>
</compile>
</compile>
<run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-verbose.xml">
<stdout>
<line text="execution(Main.target())"/>
Expand All @@ -207,7 +207,7 @@
files="ataspectj/ltwlog/Aspect1.java"
options="-1.5 -XterminateAfterCompilation"
>
</compile>
</compile>
<run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-verboseandshow.xml">
<stdout>
<line text="execution(Main.target())"/>
Expand All @@ -218,7 +218,8 @@
<line text="info using"/>
<line text="info register aspect ataspectj.ltwlog.Aspect1"/>
<line text="debug weaving 'ataspectj.ltwlog.Main'"/>
<line text="weaveinfo Join point 'method-execution(void ataspectj.ltwlog.Main.target())' in Type 'ataspectj.ltwlog.Main' (Main.java:22) advised by before advice from 'ataspectj.ltwlog.Aspect1' (Aspect1.java)"/>
<line text="Join point 'method-execution(void ataspectj.ltwlog.Main.target())' in Type 'ataspectj.ltwlog.Main' (Main.java:22) advised by before advice from 'ataspectj.ltwlog.Aspect1' (Aspect1.java)"/>
<line text="see also:"/>
<line text="debug weaving 'ataspectj.ltwlog.Aspect1'"/>
</stderr>
</run>
Expand All @@ -233,7 +234,7 @@
files="ataspectj/ltwlog/Aspect1.java"
options="-1.5 -XterminateAfterCompilation"
>
</compile>
</compile>
<run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-messagehandler.xml">
<stdout>
<line text="MessageHolder.MessageHolder()"/>
Expand All @@ -257,11 +258,16 @@
<!--<run class="ataspectj.DeclareParentsInterfaceTest" ltw="ataspectj/aop-decptest.xml"/>-->
<ant file="ajc-ant.xml" target="ltw.Decp" verbose="true">
<stderr>
<line text="weaveinfo Extending interface set for type 'ataspectj.DeclareParentsInterfaceTest$Target' (DeclareParentsInterfaceTest.java) to include 'ataspectj.DeclareParentsInterfaceTest$Marker' (DeclareParentsInterfaceTest.java)"/>
<line text="weaveinfo Join point 'method-execution(void ataspectj.DeclareParentsInterfaceTest$Target.target())' in Type 'ataspectj.DeclareParentsInterfaceTest$Target' (DeclareParentsInterfaceTest.java:27) advised by before advice from 'ataspectj.DeclareParentsInterfaceTest$TestAspect' (DeclareParentsInterfaceTest.java)"/>
<line text="weaveinfo Extending interface set for type 'ataspectj.DeclareParentsImplementsTest$Target' (DeclareParentsImplementsTest.java) to include 'ataspectj.DeclareParentsImplementsTest$Introduced' (DeclareParentsImplementsTest.java)"/>
<line text="weaveinfo Type 'ataspectj.DeclareParentsImplementsTest$Target' (DeclareParentsImplementsTest.java) has intertyped method from 'ataspectj.DeclareParentsImplementsTest$TestAspect' (DeclareParentsImplementsTest.java:'void ataspectj.DeclareParentsImplementsTest$Introduced.intro()')"/>
<line text="weaveinfo Join point 'method-execution(void ataspectj.DeclareParentsImplementsTest$Implementation.intro())' in Type 'ataspectj.DeclareParentsImplementsTest$Implementation' (DeclareParentsImplementsTest.java:47) advised by before advice from 'ataspectj.DeclareParentsImplementsTest$TestAspect' (DeclareParentsImplementsTest.java)"/>
<line text="Extending interface set for type 'ataspectj.DeclareParentsInterfaceTest$Target' (DeclareParentsInterfaceTest.java) to include 'ataspectj.DeclareParentsInterfaceTest$Marker' (DeclareParentsInterfaceTest.java)"/>
<line text="see also:"/>
<line text="Join point 'method-execution(void ataspectj.DeclareParentsInterfaceTest$Target.target())' in Type 'ataspectj.DeclareParentsInterfaceTest$Target' (DeclareParentsInterfaceTest.java:27) advised by before advice from 'ataspectj.DeclareParentsInterfaceTest$TestAspect' (DeclareParentsInterfaceTest.java)"/>
<line text="see also:"/>
<line text="Extending interface set for type 'ataspectj.DeclareParentsImplementsTest$Target' (DeclareParentsImplementsTest.java) to include 'ataspectj.DeclareParentsImplementsTest$Introduced' (DeclareParentsImplementsTest.java)"/>
<line text="see also:"/>
<line text="Type 'ataspectj.DeclareParentsImplementsTest$Target' (DeclareParentsImplementsTest.java) has intertyped method from 'ataspectj.DeclareParentsImplementsTest$TestAspect' (DeclareParentsImplementsTest.java:'void ataspectj.DeclareParentsImplementsTest$Introduced.intro()')"/>
<line text="see also:"/>
<line text="Join point 'method-execution(void ataspectj.DeclareParentsImplementsTest$Implementation.intro())' in Type 'ataspectj.DeclareParentsImplementsTest$Implementation' (DeclareParentsImplementsTest.java:47) advised by before advice from 'ataspectj.DeclareParentsImplementsTest$TestAspect' (DeclareParentsImplementsTest.java)"/>
<line text="see also:"/>
</stderr>
</ant>
</ajc-test>
Expand Down Expand Up @@ -346,4 +352,4 @@
</ajc-test>


</suite>
</suite>
40 changes: 20 additions & 20 deletions tests/src/test/resources/org/aspectj/systemtest/ajc150/ltw/ltw.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@
<line text="Main.test2" />
</stdout>
<stderr>
<line
text="weaveinfo Join point 'method-execution(void Main.test1())' in Type 'Main' (Main.java:17) advised by before advice from 'Aspect1' (Aspect1.aj:16)" />
<line
text="weaveinfo Join point 'method-execution(void Main.test2())' in Type 'Main' (Main.java:21) advised by before advice from 'Aspect2' (Aspect2.aj:16)" />
<line text="Join point 'method-execution(void Main.test1())' in Type 'Main' (Main.java:17) advised by before advice from 'Aspect1' (Aspect1.aj:16)" />
<line text="see also:" />
<line text="Join point 'method-execution(void Main.test2())' in Type 'Main' (Main.java:21) advised by before advice from 'Aspect2' (Aspect2.aj:16)" />
<line text="see also:" />
<line text="Aspect1.before_test1" />
<line text="Aspect2.before_test2" />
</stderr>
Expand Down Expand Up @@ -249,8 +249,8 @@
<line text="TestITDMethod.main" />
</stdout>
<stderr>
<line
text="weaveinfo Type 'TestITDMethod' (TestITDMethod.java) has intertyped method from 'AbstractAspect' (AbstractAspect.aj:'void TestITDMethod.test()')" />
<line text="Type 'TestITDMethod' (TestITDMethod.java) has intertyped method from 'AbstractAspect' (AbstractAspect.aj:'void TestITDMethod.test()')" />
<line text="see also:" />
<line text="AbstractAspect_TestITDMethod.test" />
</stderr>
</run>
Expand Down Expand Up @@ -359,12 +359,12 @@
<line text="T call" />
</stdout>
<stderr>
<line
text="weaveinfo Join point 'method-call(void util.A.foo())' in Type 'child.Executor' (Executor.aj:18) advised by before advice from 'child.Advisor' (Advisor.aj:20)" />
<line
text="weaveinfo Extending interface set for type 'util.A' (A.aj) to include 'util.T' (Advisor.aj)" />
<line
text="weaveinfo Extending interface set for type 'child.Advisor' (Advisor.aj) to include 'util.T' (Advisor.aj)" />
<line text="Join point 'method-call(void util.A.foo())' in Type 'child.Executor' (Executor.aj:18) advised by before advice from 'child.Advisor' (Advisor.aj:20)" />
<line text="see also:" />
<line text="Extending interface set for type 'util.A' (A.aj) to include 'util.T' (Advisor.aj)" />
<line text="see also:" />
<line text="Extending interface set for type 'child.Advisor' (Advisor.aj) to include 'util.T' (Advisor.aj)" />
<line text="see also:" />
</stderr>
</run>
</ajc-test>
Expand Down Expand Up @@ -464,10 +464,10 @@
<line text="Main.test2" />
</stdout>
<stderr>
<line
text="weaveinfo Join point 'method-execution(void Main.test1())' in Type 'Main' (Main.java:17) advised by before advice from 'Aspect1' (Aspect1.aj:16)" />
<line
text="weaveinfo Join point 'method-execution(void Main.test2())' in Type 'Main' (Main.java:21) advised by before advice from 'Aspect2' (Aspect2.aj:16)" />
<line text="Join point 'method-execution(void Main.test1())' in Type 'Main' (Main.java:17) advised by before advice from 'Aspect1' (Aspect1.aj:16)" />
<line text="see also:" />
<line text="Join point 'method-execution(void Main.test2())' in Type 'Main' (Main.java:21) advised by before advice from 'Aspect2' (Aspect2.aj:16)" />
<line text="see also:" />
<line text="Aspect1.before_test1" />
<line text="Aspect2.before_test2" />
</stderr>
Expand All @@ -491,10 +491,10 @@
<line text="Main.test2" />
</stdout>
<stderr>
<line
text="weaveinfo Join point 'method-execution(void Main.test1())' in Type 'Main' (Main.java:17) advised by before advice from 'Aspect1' (Aspect1.aj:16)" />
<line
text="weaveinfo Join point 'method-execution(void Main.test2())' in Type 'Main' (Main.java:21) advised by before advice from 'Aspect2' (Aspect2.aj:16)" />
<line text="Join point 'method-execution(void Main.test1())' in Type 'Main' (Main.java:17) advised by before advice from 'Aspect1' (Aspect1.aj:16)" />
<line text="see also:" />
<line text="Join point 'method-execution(void Main.test2())' in Type 'Main' (Main.java:21) advised by before advice from 'Aspect2' (Aspect2.aj:16)" />
<line text="see also:" />
<line text="Aspect1.before_test1" />
<line text="Aspect2.before_test2" />
</stderr>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,8 @@
<compile files="AnAspect.java" options="-1.4"/>
<run class="BigHorribleClass" ltw="aop1.xml">
<stderr>
<line text="weaveinfo Join point 'staticinitialization(void BigHorribleClass.&lt;clinit&gt;())'"/>
<line text="Join point 'staticinitialization(void BigHorribleClass.&lt;clinit&gt;())'"/>
<line text="see also:" />
<line text="SerialVersionUID is "/>
</stderr>
</run>
Expand Down
Loading

0 comments on commit afc8261

Please sign in to comment.