Set version to 7.0.0 and misc changes

* Set version to 7.0
* Fix tests
* Edit changelogs

Author: mirobers

BreakingChanges.txt

@@ -1,21 +1,22 @@
 Changes in 7.0.0
 
 OTHER
-* Upgraded Key Vault dependency to 1.0.
+ * Upgraded Key Vault dependency to 1.0. Users of the IKey and IKeyResolver interfaces should note the interface changes in this version.
+ * The getErrorCode method of StorageExtendedErrorInformation is now deprecated. Use the corresponding methods in RequestResult and StorageException instead.
 
 Changes in 6.0.0
 
 FILE
-* Many File service APIs can now throw a URISyntaxException.
-* Changed listShares() ShareListingDetails parameter to be an enum set like what is done for listing blobs.
+ * Many File service APIs can now throw a URISyntaxException.
+ * Changed listShares() ShareListingDetails parameter to be an enum set like what is done for listing blobs.
 
 OTHER
  * DefaultEndpointsProtocol will now be explicitly included in generated connection strings.
  * Connection string parsing has been substantially re-written and expanded. Please refer to current documentation about connection string formats.
 
 Changes in 5.1.1
 OTHER
-  * Reverted the code from 5.1.0 because it contained a regression and an accidental change.
+ * Reverted the code from 5.1.0 because it contained a regression and an accidental change.
 
 Changes in 5.0.0
 

ChangeLog.txt

@@ -1,6 +1,8 @@
-XXXX.XX.xx Version X.X.X
+2018.01.11 Version 7.0.0
  * Support for 2017-07-29 REST version. Please see our REST api documentation and blogs for information about the related added features.
  * Added support for soft delete feature. If a delete retention policy is enabled through the set service properties API, then blobs or snapshots can be deleted softly and retained for a specified number of days, before being permanently removed by garbage collection.
+ * When a storage request fails, the error code may now be retrieved directly from the RequestResult and StorageException classes. This error code is populated even in cases where there is no StorageExtendedErrorInformation available, such as in calls to FetchAttributes.
+ * Queue messages may now be inserted with infinite duration by specifying -1 as the timeToLiveInSeconds parameter to addMessage.
  * Improved performance of blob uploadFromFile APIs to avoid unnecessary buffering.
  * Improved performance when streaming directly from a FileInputStream to avoid unnecessary buffering.
  * Switched to using fixed length streaming mode in the HTTP client to avoid unnecessary buffering.

microsoft-azure-storage-test/src/com/microsoft/azure/storage/TestHelper.java

@@ -64,7 +64,7 @@ public class TestHelper {
     private static StorageCredentialsAccountAndKey premiumBlobCredentials;
     private static CloudStorageAccount premiumBlobAccount;
 
-    private final static boolean enableFiddler = true;
+    private final static boolean enableFiddler = false;
     private final static boolean requireSecondaryEndpoint = false;
 
     public static CloudBlobClient createCloudBlobClient() throws StorageException {

microsoft-azure-storage-test/src/com/microsoft/azure/storage/TestRunners.java

@@ -120,7 +120,7 @@ public static class CoreTestSuite {
     @SuiteClasses({ BlobOutputStreamTests.class, CloudBlobClientTests.class, CloudBlobContainerTests.class,
             CloudBlobDirectoryTests.class, CloudAppendBlobTests.class, CloudBlockBlobTests.class, CloudPageBlobTests.class,
             CloudBlobClientEncryptionTests.class, CloudBlobServerEncryptionTests.class, LeaseTests.class, SasTests.class,
-            PremiumBlobTests.class })
+            PremiumBlobTestSuite.class })
     public static class BlobTestSuite {
     }
 
@@ -184,6 +184,7 @@ public static class FastTestSuite {
 
     @RunWith(Categories.class)
     @IncludeCategory(PremiumBlobTests.class)
+    @SuiteClasses(AllTestSuite.class)
     public static class PremiumBlobTestSuite {
     }
 }

microsoft-azure-storage-test/src/com/microsoft/azure/storage/analytics/CloudAnalyticsClientTests.java

@@ -401,7 +401,7 @@ public void testCloudAnalyticsClientParseLogErrors() throws ParseException, URIS
             CloudAnalyticsClient.parseLogBlobs(null);
         }
         catch (IllegalArgumentException e) {
-            assertEquals(e.getMessage(), "The argument must not be null or an empty string. Argument name: logBlobs.");
+            assertEquals("The argument must not be null. Argument name: logBlobs.", e.getMessage());
         }
     }
 

microsoft-azure-storage-test/src/com/microsoft/azure/storage/table/TableBatchOperationTests.java

@@ -322,7 +322,7 @@ public void testBatchAddNullShouldThrow() {
             fail();
         }
         catch (IllegalArgumentException ex) {
-            assertEquals(ex.getMessage(), String.format(SR.ARGUMENT_NULL_OR_EMPTY, "element"));
+            assertEquals(String.format(SR.ARGUMENT_NULL, "element"), ex.getMessage());
         }
     }
 
@@ -334,8 +334,8 @@ public void testBatchRetrieveWithNullResolverShouldThrow() {
             fail();
         }
         catch (IllegalArgumentException ex) {
-            assertEquals(ex.getMessage(),
-                    String.format(SR.ARGUMENT_NULL_OR_EMPTY, SR.QUERY_REQUIRES_VALID_CLASSTYPE_OR_RESOLVER));
+            assertEquals(String.format(SR.ARGUMENT_NULL, SR.QUERY_REQUIRES_VALID_CLASSTYPE_OR_RESOLVER),
+                    ex.getMessage());
         }
     }
 

microsoft-azure-storage-test/src/com/microsoft/azure/storage/table/TableOperationTests.java

@@ -83,8 +83,8 @@ public void testRetrieveWithNullResolver() {
             TableOperation.retrieve("foo", "blah", (EntityResolver<?>) null);
         }
         catch (IllegalArgumentException ex) {
-            assertEquals(ex.getMessage(),
-                    String.format(SR.ARGUMENT_NULL_OR_EMPTY, SR.QUERY_REQUIRES_VALID_CLASSTYPE_OR_RESOLVER));
+            assertEquals(String.format(SR.ARGUMENT_NULL, SR.QUERY_REQUIRES_VALID_CLASSTYPE_OR_RESOLVER),
+                    ex.getMessage());
         }
     }
 

microsoft-azure-storage-test/src/com/microsoft/azure/storage/table/TableQueryTests.java

@@ -82,7 +82,7 @@ public void testQueryWithNullClassType() {
             TableQuery.from(null);
         }
         catch (IllegalArgumentException ex) {
-            assertEquals(ex.getMessage(), String.format(SR.ARGUMENT_NULL_OR_EMPTY, "class type"));
+            assertEquals(String.format(SR.ARGUMENT_NULL, "class type"), ex.getMessage());
         }
     }
 

microsoft-azure-storage/src/com/microsoft/azure/storage/Constants.java

@@ -670,7 +670,7 @@ public static class HeaderConstants {
         /**
          * Specifies the value to use for UserAgent header.
          */
-        public static final String USER_AGENT_VERSION = "6.1.0";
+        public static final String USER_AGENT_VERSION = "7.0.0";
 
         /**
          * The default type for content-type and accept

microsoft-azure-storage/src/com/microsoft/azure/storage/StorageExtendedErrorInformation.java

@@ -64,6 +64,7 @@ public HashMap<String, String[]> getAdditionalDetails() {
      *
      * @deprecated use the property on {@link RequestResult} instead.
      */
+    @Deprecated
     public String getErrorCode() {
         return this.errorCode;
     }