Author: pmouawad
Date: Sat Sep 17 13:30:28 2016
New Revision: 1761229

URL: http://svn.apache.org/viewvc?rev=1761229&view=rev
Log:
Bug 60106 - Settings defaults : Switch "jmeter.save.saveservice.connect_time" 
to true (after 3.0)
Fix tests
Bugzilla Id: 60106

Modified:
    jmeter/trunk/test/src/org/apache/jmeter/save/TestCSVSaveService.java

Modified: jmeter/trunk/test/src/org/apache/jmeter/save/TestCSVSaveService.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/test/src/org/apache/jmeter/save/TestCSVSaveService.java?rev=1761229&r1=1761228&r2=1761229&view=diff
==============================================================================
--- jmeter/trunk/test/src/org/apache/jmeter/save/TestCSVSaveService.java 
(original)
+++ jmeter/trunk/test/src/org/apache/jmeter/save/TestCSVSaveService.java Sat 
Sep 17 13:30:28 2016
@@ -155,7 +155,7 @@ public class TestCSVSaveService extends
     // header text should not change unexpectedly
     // if this test fails, check whether the default was intentionally changed 
or not
     public void testHeader() {
-        final String HDR = 
"timeStamp,elapsed,label,responseCode,responseMessage,threadName,dataType,success,failureMessage,bytes,grpThreads,allThreads,Latency,IdleTime";
+        final String HDR = 
"timeStamp,elapsed,label,responseCode,responseMessage,threadName,dataType,success,failureMessage,bytes,grpThreads,allThreads,Latency,IdleTime,Connect";
         assertEquals("Header text has changed", HDR, 
CSVSaveService.printableFieldNamesToString());
     }
 
@@ -163,7 +163,7 @@ public class TestCSVSaveService extends
     // sample format should not change unexpectedly
     // if this test fails, check whether the default was intentionally changed 
or not
     public void testSample() {
-        final String RESULT = "1,2,3,4,5,6,7,true,,8,9,10,11,12";
+        final String RESULT = "1,2,3,4,5,6,7,true,,8,9,10,11,12,13";
         SampleResult result = new SampleResult();
         result.setSaveConfig(new SampleSaveConfiguration());
         result.setStampAndTime(1, 2);
@@ -178,7 +178,7 @@ public class TestCSVSaveService extends
         result.setAllThreads(10);
         result.setLatency(11);
         result.setIdleTime(12);
-//        result.setConnectTime(13);
+        result.setConnectTime(13);
         assertEquals("Result text has changed", RESULT, 
CSVSaveService.resultToDelimitedString(new SampleEvent(result,"")));
     }
 }


Reply via email to