<html>
<head>
<style><!--
  body {background-color:#ffffff;}
  .file {border:1px solid #eeeeee;margin-top:1em;margin-bottom:1em;}
  .pathname {font-family:monospace; float:right;}
  .fileheader {margin-bottom:.5em;}
  .diff {margin:0;}
  .tasklist {padding:4px;border:1px dashed #000000;margin-top:1em;}
  .tasklist ul {margin-top:0;margin-bottom:0;}
  tr.alt {background-color:#eeeeee}
  #added {background-color:#ddffdd;}
  #addedchars {background-color:#99ff99;font-weight:bolder;}
  tr.alt #added {background-color:#ccf7cc;}
  #removed {background-color:#ffdddd;}
  #removedchars {background-color:#ff9999;font-weight:bolder;}
  tr.alt #removed {background-color:#f7cccc;}
  #copied {background-color:#ccccff;}
  tr.alt #copied {background-color:#bbbbf7;}
  #info {color:#888888;}
  #context {background-color:#eeeeee;}
  td {padding-left:.3em;padding-right:.3em;}
  tr.head {border-bottom-width:1px;border-bottom-style:solid;}
  tr.head td {padding:0;padding-top:.2em;}
  .task {background-color:#ffff00;}
  .comment {white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;white-space:pre-wrap;word-wrap:break-word;padding:4px;border:1px dashed #000000;background-color:#ffffdd}
  .error {color:red;}
  hr {border-width:0px;height:2px;background:black;}
--></style>
</head>
<body>
<table cellspacing="0" cellpadding="0" border="0" rules="cols">
<tr class="head"><td colspan="5">Commit in <b><tt>metaas/trunk/src/test/java/uk/co/badgersinfoil/metaas</tt></b></td></tr>
<tr><td><tt><a href="#file1">ASMethodTests.java</a></tt> </td><td></td><td align="right" id="added">+39</td><td align="right" id="removed">-15</td><td nowrap="nowrap" align="center">559 -&gt; 560</td></tr>
</table>
<pre class="comment">
increase code coverage in ModifierUtils.java
</pre>
<hr /><a name="file1" /><div class="file">
<span class="pathname">metaas/trunk/src/test/java/uk/co/badgersinfoil/metaas</span><br />
<div class="fileheader"><big><b>ASMethodTests.java</b></big> <small id="info">559 -&gt; 560</small></div>
<pre class="diff"><small id="info">--- trunk/src/test/java/uk/co/badgersinfoil/metaas/ASMethodTests.java        2007-12-03 00:10:37 UTC (rev 559)
+++ trunk/src/test/java/uk/co/badgersinfoil/metaas/ASMethodTests.java        2007-12-03 00:28:12 UTC (rev 560)
@@ -13,11 +13,12 @@
</small></pre><pre class="diff" id="context"> public class ASMethodTests extends TestCase {
         private ActionScriptFactory fact = new ActionScriptFactory();
         private ASCompilationUnit unit;
</pre><pre class="diff" id="added">+        private ASClassType clazz;
</pre><pre class="diff" id="context">         private ASMethod meth;
 
         protected void setUp() {
                 unit = fact.newClass("Test");
</pre><pre class="diff" id="removed">-                <span id="removedchars">ASClassType&nbsp;</span>clazz = (ASClassType)unit.getType();
</pre><pre class="diff" id="added">+                clazz = (ASClassType)unit.getType();
</pre><pre class="diff" id="context">                 meth = clazz.newMethod("test", Visibility.PUBLIC, null);
         }
 
</pre><pre class="diff"><small id="info">@@ -96,26 +97,21 @@
</small></pre><pre class="diff" id="context">         }
 
         public void testProtection() {
</pre><pre class="diff" id="removed">-                assertSame(Visibility.PUBLIC, meth.getVisibility());
</pre><pre class="diff" id="added">+                assertVisibility(Visibility.PUBLIC, meth);
</pre><pre class="diff" id="context">                 meth.setVisibility(Visibility.PRIVATE);
</pre><pre class="diff" id="removed">-                assertSame(Visibility.PRIVATE, meth.getVisibility());
</pre><pre class="diff" id="added">+                assertVisibility(Visibility.PRIVATE, meth);
</pre><pre class="diff" id="context">                 meth.setVisibility(Visibility.PROTECTED);
</pre><pre class="diff" id="removed">-                assertSame(Visibility.PROTECTED, meth.getVisibility());
</pre><pre class="diff" id="added">+                assertVisibility(Visibility.PROTECTED, meth);
</pre><pre class="diff" id="context">                 meth.setVisibility(Visibility.INTERNAL);
</pre><pre class="diff" id="removed">-                assertSame(Visibility.INTERNAL, meth.getVisibility());
</pre><pre class="diff" id="added">+                assertVisibility(Visibility.INTERNAL, meth);
</pre><pre class="diff" id="context">                 meth.setVisibility(Visibility.PUBLIC);
</pre><pre class="diff" id="removed">-                assertSame(Visibility.PUBLIC, meth.getVisibility());
</pre><pre class="diff" id="added">+                assertVisibility(Visibility.PUBLIC, meth);
</pre><pre class="diff" id="context"> 
</pre><pre class="diff" id="removed">-                meth.setVisibility(Visibility.DEFAULT);
-                assertSame(Visibility.DEFAULT, meth.getVisibility());
-                meth.setVisibility(Visibility.PRIVATE);
-                assertSame(Visibility.PRIVATE, meth.getVisibility());
</pre><pre class="diff" id="added">+                checkSetVisibilityAfterDefault(meth, Visibility.PRIVATE);
+                checkSetVisibilityAfterDefault(meth, Visibility.PUBLIC);
+                checkSetVisibilityAfterDefault(meth, Visibility.PROTECTED);
+                checkSetVisibilityAfterDefault(meth, Visibility.INTERNAL);
</pre><pre class="diff" id="context"> 
</pre><pre class="diff" id="removed">-                meth.setVisibility(Visibility.DEFAULT);
-                assertSame(Visibility.DEFAULT, meth.getVisibility());
-                meth.setVisibility(Visibility.PUBLIC);
-                assertSame(Visibility.PUBLIC, meth.getVisibility());
-
</pre><pre class="diff" id="context">                 // squeeze out the last bit of code coverage,
                 assertEquals("public", Visibility.PUBLIC.toString());
                 assertEquals("private", Visibility.PRIVATE.toString());
</pre><pre class="diff"><small id="info">@@ -123,7 +119,35 @@
</small></pre><pre class="diff" id="context">                 assertEquals("internal", Visibility.INTERNAL.toString());
                 assertEquals("[default]", Visibility.DEFAULT.toString());
         }
</pre><pre class="diff" id="added">+        
+        private static void assertVisibility(Visibility expectedVisibility,
+                                             ASMethod method)
+        {
+                assertSame(expectedVisibility, method.getVisibility());
+        }
</pre><pre class="diff" id="context"> 
</pre><pre class="diff" id="added">+        private static void checkSetVisibilityAfterDefault(ASMethod method,
+                                                    Visibility visibility)
+        {
+                method.setVisibility(Visibility.DEFAULT);
+                assertVisibility(Visibility.DEFAULT, method);
+                method.setVisibility(visibility);
+                assertVisibility(visibility, method);
+        }
+        
+        public void testProtectionOnCreate() {
+                meth = clazz.newMethod("testDefault", Visibility.DEFAULT, null);
+                assertVisibility(Visibility.DEFAULT, meth);
+                meth = clazz.newMethod("testPublic", Visibility.PUBLIC, null);
+                assertVisibility(Visibility.PUBLIC, meth);
+                meth = clazz.newMethod("testPrivate", Visibility.PRIVATE, null);
+                assertVisibility(Visibility.PRIVATE, meth);
+                meth = clazz.newMethod("testProtected", Visibility.PROTECTED, null);
+                assertVisibility(Visibility.PROTECTED, meth);
+                meth = clazz.newMethod("testInternal", Visibility.INTERNAL, null);
+                assertVisibility(Visibility.INTERNAL, meth);
+        }
+
</pre><pre class="diff" id="context">         public void testArgs() throws IOException {
                 ASArg foo = meth.addParam("foo", "Number");
                 assertEquals("foo", foo.getName());
</pre></div>
<center><small><a href="http://www.badgers-in-foil.co.uk/projects/cvsspam/" title="commit -&gt; email">CVSspam</a> 0.2.12</small></center>
</body></html>