Merge branch 'hkj-fcm-errors' of github.com:firebase/firebase-admin-dotnet into hkj-fcm-errors

Author: hiranya911

FirebaseAdmin/FirebaseAdmin.Tests/Messaging/FirebaseMessagingClientTest.cs

@@ -34,9 +34,9 @@ public class FirebaseMessagingClientTest
         public void NoProjectId()
         {
             var clientFactory = new HttpClientFactory();
-            Assert.Throws<FirebaseException>(
+            Assert.Throws<ArgumentException>(
                 () => new FirebaseMessagingClient(clientFactory, MockCredential, null));
-            Assert.Throws<FirebaseException>(
+            Assert.Throws<ArgumentException>(
                 () => new FirebaseMessagingClient(clientFactory, MockCredential, string.Empty));
         }
 

FirebaseAdmin/FirebaseAdmin/Messaging/AndroidConfig.cs

@@ -97,7 +97,7 @@ private string PriorityString
                         this.Priority = Messaging.Priority.High;
                         return;
                     default:
-                        throw new FirebaseException(
+                        throw new ArgumentException(
                             $"Invalid priority value: {value}. Only 'high' and 'normal'"
                             + " are allowed.");
                 }

FirebaseAdmin/FirebaseAdmin/Messaging/FirebaseMessagingClient.cs

@@ -49,7 +49,7 @@ public FirebaseMessagingClient(
         {
             if (string.IsNullOrEmpty(projectId))
             {
-                throw new FirebaseException(
+                throw new ArgumentException(
                     "Project ID is required to access messaging service. Use a service account "
                     + "credential or set the project ID explicitly via AppOptions. Alternatively "
                     + "you can set the project ID via the GOOGLE_CLOUD_PROJECT environment "
@@ -83,7 +83,7 @@ internal static string ClientVersion
         /// <exception cref="ArgumentNullException">If the message argument is null.</exception>
         /// <exception cref="ArgumentException">If the message contains any invalid
         /// fields.</exception>
-        /// <exception cref="FirebaseException">If an error occurs while sending the
+        /// <exception cref="FirebaseMessagingException">If an error occurs while sending the
         /// message.</exception>
         /// <param name="message">The message to be sent. Must not be null.</param>
         /// <param name="dryRun">A boolean indicating whether to perform a dry run (validation