You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2014/09/08 00:48:33 UTC

svn commit: r1623241 - /tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java

Author: markt
Date: Sun Sep  7 22:48:33 2014
New Revision: 1623241

URL: http://svn.apache.org/r1623241
Log:
Code format. No functional change.
Remove unnecessary blocks.

Modified:
    tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java

Modified: tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java?rev=1623241&r1=1623240&r2=1623241&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java (original)
+++ tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java Sun Sep  7 22:48:33 2014
@@ -80,102 +80,80 @@ public class TestExpiresFilter extends T
         tomcat.start();
         try {
             // VERIFY EXCLUDED RESPONSE STATUS CODES
-            {
-                int[] excludedResponseStatusCodes = expiresFilter.getExcludedResponseStatusCodesAsInts();
-                Assert.assertEquals(2, excludedResponseStatusCodes.length);
-                Assert.assertEquals(304, excludedResponseStatusCodes[0]);
-                Assert.assertEquals(503, excludedResponseStatusCodes[1]);
-            }
+            int[] excludedResponseStatusCodes = expiresFilter.getExcludedResponseStatusCodesAsInts();
+            Assert.assertEquals(2, excludedResponseStatusCodes.length);
+            Assert.assertEquals(304, excludedResponseStatusCodes[0]);
+            Assert.assertEquals(503, excludedResponseStatusCodes[1]);
 
             // VERIFY DEFAULT CONFIGURATION
-            {
-                ExpiresConfiguration expiresConfiguration = expiresFilter.getDefaultExpiresConfiguration();
-                Assert.assertEquals(StartingPoint.ACCESS_TIME,
-                        expiresConfiguration.getStartingPoint());
-                Assert.assertEquals(1,
-                        expiresConfiguration.getDurations().size());
-                Assert.assertEquals(DurationUnit.MONTH,
-                        expiresConfiguration.getDurations().get(0).getUnit());
-                Assert.assertEquals(1, expiresConfiguration.getDurations().get(
-                        0).getAmount());
-            }
+            ExpiresConfiguration expiresConfigurationDefault =
+                    expiresFilter.getDefaultExpiresConfiguration();
+            Assert.assertEquals(StartingPoint.ACCESS_TIME,
+                    expiresConfigurationDefault.getStartingPoint());
+            Assert.assertEquals(1, expiresConfigurationDefault.getDurations().size());
+            Assert.assertEquals(DurationUnit.MONTH,
+                    expiresConfigurationDefault.getDurations().get(0).getUnit());
+            Assert.assertEquals(1, expiresConfigurationDefault.getDurations().get(0).getAmount());
 
             // VERIFY TEXT/HTML
-            {
-                ExpiresConfiguration expiresConfiguration = expiresFilter.getExpiresConfigurationByContentType().get(
-                        "text/html");
-                Assert.assertEquals(StartingPoint.ACCESS_TIME,
-                        expiresConfiguration.getStartingPoint());
-
-                Assert.assertEquals(3,
-                        expiresConfiguration.getDurations().size());
-
-                Duration oneMonth = expiresConfiguration.getDurations().get(0);
-                Assert.assertEquals(DurationUnit.MONTH, oneMonth.getUnit());
-                Assert.assertEquals(1, oneMonth.getAmount());
-
-                Duration fifteenDays = expiresConfiguration.getDurations().get(
-                        1);
-                Assert.assertEquals(DurationUnit.DAY, fifteenDays.getUnit());
-                Assert.assertEquals(15, fifteenDays.getAmount());
-
-                Duration twoHours = expiresConfiguration.getDurations().get(2);
-                Assert.assertEquals(DurationUnit.HOUR, twoHours.getUnit());
-                Assert.assertEquals(2, twoHours.getAmount());
-            }
+            ExpiresConfiguration expiresConfigurationTextHtml =
+                    expiresFilter.getExpiresConfigurationByContentType().get("text/html");
+            Assert.assertEquals(StartingPoint.ACCESS_TIME,
+                    expiresConfigurationTextHtml.getStartingPoint());
+
+            Assert.assertEquals(3, expiresConfigurationTextHtml.getDurations().size());
+
+            Duration oneMonth = expiresConfigurationTextHtml.getDurations().get(0);
+            Assert.assertEquals(DurationUnit.MONTH, oneMonth.getUnit());
+            Assert.assertEquals(1, oneMonth.getAmount());
+
+            Duration fifteenDays = expiresConfigurationTextHtml.getDurations().get(1);
+            Assert.assertEquals(DurationUnit.DAY, fifteenDays.getUnit());
+            Assert.assertEquals(15, fifteenDays.getAmount());
+
+            Duration twoHours = expiresConfigurationTextHtml.getDurations().get(2);
+            Assert.assertEquals(DurationUnit.HOUR, twoHours.getUnit());
+            Assert.assertEquals(2, twoHours.getAmount());
+
             // VERIFY IMAGE/GIF
-            {
-                ExpiresConfiguration expiresConfiguration = expiresFilter.getExpiresConfigurationByContentType().get(
-                        "image/gif");
-                Assert.assertEquals(StartingPoint.LAST_MODIFICATION_TIME,
-                        expiresConfiguration.getStartingPoint());
-
-                Assert.assertEquals(2,
-                        expiresConfiguration.getDurations().size());
-
-                Duration fiveHours = expiresConfiguration.getDurations().get(0);
-                Assert.assertEquals(DurationUnit.HOUR, fiveHours.getUnit());
-                Assert.assertEquals(5, fiveHours.getAmount());
-
-                Duration threeMinutes = expiresConfiguration.getDurations().get(
-                        1);
-                Assert.assertEquals(DurationUnit.MINUTE, threeMinutes.getUnit());
-                Assert.assertEquals(3, threeMinutes.getAmount());
+            ExpiresConfiguration expiresConfigurationImageGif =
+                    expiresFilter.getExpiresConfigurationByContentType().get("image/gif");
+            Assert.assertEquals(StartingPoint.LAST_MODIFICATION_TIME,
+                    expiresConfigurationImageGif.getStartingPoint());
+
+            Assert.assertEquals(2, expiresConfigurationImageGif.getDurations().size());
+
+            Duration fiveHours = expiresConfigurationImageGif.getDurations().get(0);
+            Assert.assertEquals(DurationUnit.HOUR, fiveHours.getUnit());
+            Assert.assertEquals(5, fiveHours.getAmount());
+
+            Duration threeMinutes = expiresConfigurationImageGif.getDurations().get(1);
+            Assert.assertEquals(DurationUnit.MINUTE, threeMinutes.getUnit());
+            Assert.assertEquals(3, threeMinutes.getAmount());
 
-            }
             // VERIFY IMAGE/JPG
-            {
-                ExpiresConfiguration expiresConfiguration = expiresFilter.getExpiresConfigurationByContentType().get(
-                        "image/jpg");
-                Assert.assertEquals(StartingPoint.ACCESS_TIME,
-                        expiresConfiguration.getStartingPoint());
-
-                Assert.assertEquals(1,
-                        expiresConfiguration.getDurations().size());
-
-                Duration tenThousandSeconds = expiresConfiguration.getDurations().get(
-                        0);
-                Assert.assertEquals(DurationUnit.SECOND,
-                        tenThousandSeconds.getUnit());
-                Assert.assertEquals(10000, tenThousandSeconds.getAmount());
+            ExpiresConfiguration expiresConfigurationImageJpg =
+                    expiresFilter.getExpiresConfigurationByContentType().get("image/jpg");
+            Assert.assertEquals(StartingPoint.ACCESS_TIME,
+                    expiresConfigurationImageJpg.getStartingPoint());
+
+            Assert.assertEquals(1, expiresConfigurationImageJpg.getDurations().size());
+
+            Duration tenThousandSeconds = expiresConfigurationImageJpg.getDurations().get(0);
+            Assert.assertEquals(DurationUnit.SECOND, tenThousandSeconds.getUnit());
+            Assert.assertEquals(10000, tenThousandSeconds.getAmount());
 
-            }
             // VERIFY VIDEO/MPEG
-            {
-                ExpiresConfiguration expiresConfiguration = expiresFilter.getExpiresConfigurationByContentType().get(
-                        "video/mpeg");
-                Assert.assertEquals(StartingPoint.LAST_MODIFICATION_TIME,
-                        expiresConfiguration.getStartingPoint());
-
-                Assert.assertEquals(1,
-                        expiresConfiguration.getDurations().size());
-
-                Duration twentyThousandSeconds = expiresConfiguration.getDurations().get(
-                        0);
-                Assert.assertEquals(DurationUnit.SECOND,
-                        twentyThousandSeconds.getUnit());
-                Assert.assertEquals(20000, twentyThousandSeconds.getAmount());
-            }
+            ExpiresConfiguration expiresConfiguration =
+                    expiresFilter.getExpiresConfigurationByContentType().get("video/mpeg");
+            Assert.assertEquals(StartingPoint.LAST_MODIFICATION_TIME,
+                    expiresConfiguration.getStartingPoint());
+
+            Assert.assertEquals(1, expiresConfiguration.getDurations().size());
+
+            Duration twentyThousandSeconds = expiresConfiguration.getDurations().get(0);
+            Assert.assertEquals(DurationUnit.SECOND, twentyThousandSeconds.getUnit());
+            Assert.assertEquals(20000, twentyThousandSeconds.getAmount());
         } finally {
             tomcat.stop();
         }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org