woolfel     2005/09/13 11:46:36

  Modified:    src/junit/org/apache/jmeter/protocol/java/sampler Tag:
                        rel-2-1 JUnitSampler.java
  Log:
  fixed a bug with the sampleresults not getting saved. the sampler wasn't 
setting the response
  data, so the resultCollector was getting a NPE.
  peter
  
  Revision  Changes    Path
  No                   revision
  No                   revision
  1.11.2.5  +6 -1      
jakarta-jmeter/src/junit/org/apache/jmeter/protocol/java/sampler/JUnitSampler.java
  
  Index: JUnitSampler.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/junit/org/apache/jmeter/protocol/java/sampler/JUnitSampler.java,v
  retrieving revision 1.11.2.4
  retrieving revision 1.11.2.5
  diff -u -r1.11.2.4 -r1.11.2.5
  --- JUnitSampler.java 5 Sep 2005 19:45:03 -0000       1.11.2.4
  +++ JUnitSampler.java 13 Sep 2005 18:46:36 -0000      1.11.2.5
  @@ -288,7 +288,7 @@
               rlabel = JUnitSampler.class.getName();
           }
           sresult.setSampleLabel(rlabel);
  -        sresult.setSamplerData(getClassname() + "." + getMethod());
  +        sresult.setSamplerData(getClassname() + "." + getMethod());   
           // check to see if the test class is null. if it is, we create
           // a new instance. this should only happen at the start of a
           // test run
  @@ -371,6 +371,7 @@
                   sresult.setSuccessful(true);
                   sresult.setResponseMessage(getSuccess());
                   sresult.setResponseCode(getSuccessCode());
  +                sresult.setResponseData("Not Applicable".getBytes());
               }
           } else {
               // we should log a warning, but allow the test to keep running
  @@ -378,6 +379,10 @@
               // this should be externalized to the properties
               sresult.setResponseMessage("failed to create an instance of the 
class");
           }
  +        sresult.setBytes(0);
  +        sresult.setContentType("text");
  +        sresult.setDataType("Not Applicable");
  +        sresult.setRequestHeaders("Not Applicable");
                return sresult;
        }
   
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to