<NME> S3FileTransferPropertiesUnitTests.java
<BEF> ADDFILE
<MSG> Make S3 strict URL checking optional and disabled by default

By default, turn off strict S3 URL checking and let arbitrary buckets/keys through.
Strict checking can be enabled via newly introduced property.

<DFF> @@ -0,0 +1,62 @@
+/*
+ *
+ *  Copyright 2016 Netflix, Inc.
+ *
+ *     Licensed under the Apache License, Version 2.0 (the "License");
+ *     you may not use this file except in compliance with the License.
+ *     You may obtain a copy of the License at
+ *
+ *         http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *     Unless required by applicable law or agreed to in writing, software
+ *     distributed under the License is distributed on an "AS IS" BASIS,
+ *     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *     See the License for the specific language governing permissions and
+ *     limitations under the License.
+ *
+ */
+package com.netflix.genie.web.properties;
+
+import com.netflix.genie.core.properties.S3FileTransferProperties;
+import com.netflix.genie.test.categories.UnitTest;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.experimental.categories.Category;
+
+/**
+ * Unit tests for S3FileTransferProperties.
+ *
+ * @author mprimi
+ * @since 3.1.0
+ */
+@Category(UnitTest.class)
+public class S3FileTransferPropertiesUnitTests {
+
+    private S3FileTransferProperties properties;
+
+    /**
+     * Setup for tests.
+     */
+    @Before
+    public void setup() {
+        this.properties = new S3FileTransferProperties();
+    }
+
+    /**
+     * Make sure constructor sets reasonable defaults.
+     */
+    @Test
+    public void canGetDefaultValues() {
+        Assert.assertFalse(this.properties.isStrictUrlCheckEnabled());
+    }
+
+    /**
+     * Make sure can enable strict URL checking.
+     */
+    @Test
+    public void canEnableStrictUrlChecking() {
+        this.properties.setStrictUrlCheckEnabled(true);
+        Assert.assertTrue(this.properties.isStrictUrlCheckEnabled());
+    }
+}

Make S3 strict URL checking optional and disabled by default