Position tokens first to guarantee backward compatibility for legacy positional arguments and add a unit test for it

Author: yvonnep165

firebase_admin/_messaging_encoder.py

@@ -84,7 +84,7 @@ class MulticastMessage:
         fcm_options: An instance of ``messaging.FCMOptions`` (optional).
     """
     def __init__(
-            self, fids=None, tokens=None, data=None, notification=None, android=None,
+            self, tokens=None, fids=None, data=None, notification=None, android=None,
             webpush=None, apns=None, fcm_options=None):
         if tokens is not None:
             warnings.warn(

tests/test_messaging.py

@@ -160,6 +160,10 @@ def test_tokens_deprecation_warning(self):
         with pytest.deprecated_call():
             messaging.MulticastMessage(tokens=['token'])
 
+    def test_tokens_deprecation_warning_positional(self):
+        with pytest.deprecated_call():
+            messaging.MulticastMessage(['token'])
+
 
 class TestMessageEncoder: