You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by Cassie <do...@apache.org> on 2008/05/14 14:08:14 UTC

Re: svn commit: r656222 - in /incubator/shindig/trunk: java/gadgets/src/test/java/org/apache/shindig/gadgets/AllTests.java pom.xml site/index.html

On Wed, May 14, 2008 at 12:56 PM, Vincent Siveton
<vi...@gmail.com> wrote:
> Hi,
>
> 2008/5/14, doll@apache.org <do...@apache.org>:
>> Author: doll
>>  Date: Wed May 14 03:24:23 2008
>>  New Revision: 656222
>>
>>  URL: http://svn.apache.org/viewvc?rev=656222&view=rev
>>  Log:
>>  SHINDIG-221
>>  Patch from Dave Smith. Added an AllTests target for the gadgets java unit tests. Also adds instructions for running this in eclipse to get code coverage results.
>
> [SNIP]
>
> http://svn.apache.org/viewvc/incubator/shindig/trunk/site/index.html?rev=656222&r1=656221&r2=656222&view=diff
>>  ==============================================================================
>>  --- incubator/shindig/trunk/site/index.html (original)
>>  +++ incubator/shindig/trunk/site/index.html Wed May 14 03:24:23 2008
>>  @@ -293,8 +293,8 @@
>>            </li>
>>            <li>File -&gt; New -&gt; Java Project
>>            <ol>
>>  -            <li>Select 'Create project from existing source' and navigate to:<br>
>>  -            ~/src/shindig/java</li>
>>  +            <li>Name the project. The instructions below will assume "SHINDIG".</li>
>>  +            <li>Select 'Create project from existing source' and navigate to <code>.../src/shindig/java</code>
>>              <li>Click Finish</li>
>>              <li>If you see a dialog for "Open Associated Perspective", click Ok. Don't worry about the errors after
>>              loading as they will be fixed in the next step.</li>
>>  @@ -307,9 +307,26 @@
>>            debugging, you need the source jars. Right-click the project, select <code>Maven : Download Sources</code> and
>>            Eclipse will automatically know about these sources when debugging. You can browse them under <code>Maven
>>            Dependencies</code> in your project.</li>
>>  +         <li>If you'll be using AllTests to run tests or generate code coverage stats, adjust the project's output folders.
>>  +           <ol>
>>  +             <li>Project -&gt; Properties -&gt; Java Build Path -&gt; Source</li>
>>  +             <li>Locate and open <code>SHINDIG/gadgets/src/test/java</code></li>
>>  +             <li>Select <code>Output Folder: (Default Output Folder)</code> and click Edit...</li>
>>  +             <li>Select <code>Specific Output Folder</code></li>
>>  +             <li>Enter <code>target/test-classes</code> and click OK.</li>
>>  +             <li>Repeat for <code>SHINDIG/social-api/src/test/java</code></li>
>
> I think that SHINDIG_HOME will be more conventional

I believe it is being called SHINDIG instead of SHINDIG_HOME because
this is the name of the eclipse project.. rather than the root of the
src tree where the code is located.


>
>>  +           </ol>
>>  +         </li>
>>          </ol>
>>        </ul>
>>
>>  +      <h4><font size="3"><a name="Code_coverage_in_Eclipse"> Generating Code Coverage in Eclipse </a></font></h4>
>>  +      <p>To generate code coverage statistics inside of Eclipse, install the <a href="http://www.eclema.org">EclEmma</a> plugin. Then</p>
>
> Typo: should be www.eclemma.org, NOT eclema.org

fixing this.
thanks.

- cassie


>
> Cheers,
>
> Vincent
>
>>  +      <ul>
>>  +        <li>Open <code>org.apache.shindig.gadgets.AllTests</code></li>
>>  +        <li>Right-click in the class, and select <code>Coverage as -&gt; JUnit Test</code></li>
>>  +      </ul>
>>  +
>>        <h4><font size="3"><a name="Running_inside_Eclipse"> Running inside Eclipse </a></font></h4>
>>        <p>To debug the server in Eclipse, follow the last two steps <a
>>          href="http://cwiki.apache.org/WICKET/maven-jetty-plugin.html" target="_top">here</a> (takes a few minutes to set up):</p>
>>
>>
>>
>

Re: svn commit: r656222 - in /incubator/shindig/trunk: java/gadgets/src/test/java/org/apache/shindig/gadgets/AllTests.java pom.xml site/index.html

Posted by Dave Smith <da...@gmail.com>.
On Wed, May 14, 2008 at 5:08 AM, Cassie <do...@apache.org> wrote:

> On Wed, May 14, 2008 at 12:56 PM, Vincent Siveton
> <vi...@gmail.com> wrote:
> > Hi,
> >
> > 2008/5/14, doll@apache.org <do...@apache.org>:
> >> Author: doll
> >>  Date: Wed May 14 03:24:23 2008
> >>  New Revision: 656222
> >>
> >>  URL: http://svn.apache.org/viewvc?rev=656222&view=rev
> >>  Log:
> >>  SHINDIG-221
> >>  Patch from Dave Smith. Added an AllTests target for the gadgets java
> unit tests. Also adds instructions for running this in eclipse to get code
> coverage results.
> >
> > [SNIP]
> >
> >
> http://svn.apache.org/viewvc/incubator/shindig/trunk/site/index.html?rev=656222&r1=656221&r2=656222&view=diff
> >>
>  ==============================================================================
> >>  --- incubator/shindig/trunk/site/index.html (original)
> >>  +++ incubator/shindig/trunk/site/index.html Wed May 14 03:24:23 2008
> >>  @@ -293,8 +293,8 @@
> >>            </li>
> >>            <li>File -&gt; New -&gt; Java Project
> >>            <ol>
> >>  -            <li>Select 'Create project from existing source' and
> navigate to:<br>
> >>  -            ~/src/shindig/java</li>
> >>  +            <li>Name the project. The instructions below will assume
> "SHINDIG".</li>
> >>  +            <li>Select 'Create project from existing source' and
> navigate to <code>.../src/shindig/java</code>
> >>              <li>Click Finish</li>
> >>              <li>If you see a dialog for "Open Associated Perspective",
> click Ok. Don't worry about the errors after
> >>              loading as they will be fixed in the next step.</li>
> >>  @@ -307,9 +307,26 @@
> >>            debugging, you need the source jars. Right-click the project,
> select <code>Maven : Download Sources</code> and
> >>            Eclipse will automatically know about these sources when
> debugging. You can browse them under <code>Maven
> >>            Dependencies</code> in your project.</li>
> >>  +         <li>If you'll be using AllTests to run tests or generate code
> coverage stats, adjust the project's output folders.
> >>  +           <ol>
> >>  +             <li>Project -&gt; Properties -&gt; Java Build Path -&gt;
> Source</li>
> >>  +             <li>Locate and open
> <code>SHINDIG/gadgets/src/test/java</code></li>
> >>  +             <li>Select <code>Output Folder: (Default Output
> Folder)</code> and click Edit...</li>
> >>  +             <li>Select <code>Specific Output Folder</code></li>
> >>  +             <li>Enter <code>target/test-classes</code> and click
> OK.</li>
> >>  +             <li>Repeat for
> <code>SHINDIG/social-api/src/test/java</code></li>
> >
> > I think that SHINDIG_HOME will be more conventional
>
> I believe it is being called SHINDIG instead of SHINDIG_HOME because
> this is the name of the eclipse project.. rather than the root of the
> src tree where the code is located.


That was my thought.


>
>
>
> >
> >>  +           </ol>
> >>  +         </li>
> >>          </ol>
> >>        </ul>
> >>
> >>  +      <h4><font size="3"><a name="Code_coverage_in_Eclipse">
> Generating Code Coverage in Eclipse </a></font></h4>
> >>  +      <p>To generate code coverage statistics inside of Eclipse,
> install the <a href="http://www.eclema.org">EclEmma</a> plugin. Then</p>
> >
>


>
> > Typo: should be www.eclemma.org, NOT eclema.org
>

Indeed. Thanks for the quick catch and fix.

<http://eclema.org>
>
> fixing this.
> thanks.
>
> - cassie
>
>
> >
> > Cheers,
> >
> > Vincent
> >
> >>  +      <ul>
> >>  +        <li>Open <code>org.apache.shindig.gadgets.AllTests</code></li>
> >>  +        <li>Right-click in the class, and select <code>Coverage as
> -&gt; JUnit Test</code></li>
> >>  +      </ul>
> >>  +
> >>        <h4><font size="3"><a name="Running_inside_Eclipse"> Running
> inside Eclipse </a></font></h4>
> >>        <p>To debug the server in Eclipse, follow the last two steps <a
> >>          href="http://cwiki.apache.org/WICKET/maven-jetty-plugin.html"
> target="_top">here</a> (takes a few minutes to set up):</p>
> >>
> >>
> >>
> >
>