Bug fix - Changing variable from "MemoryCost" to "cpuMemCost" in standard_scrypt method (#643)

* Changing 'memoryCost' to CpuMemCost

RELEASE NOTE: Fixed an incorrect key used to set MemoryCost config in standard_scrypt() API

* Update Test for standard_scrypt

* Update _user_import.py

Author: pragatimodi

firebase_admin/_user_import.py

@@ -454,7 +454,7 @@ def standard_scrypt(cls, memory_cost, parallelization, block_size, derived_key_l
         """Creates a new standard Scrypt algorithm instance.
 
         Args:
-            memory_cost: Memory cost as a non-negaive integer.
+            memory_cost: CPU Memory cost as a non-negative integer.
             parallelization: Parallelization as a non-negative integer.
             block_size: Block size as a non-negative integer.
             derived_key_length: Derived key length as a non-negative integer.
@@ -463,7 +463,7 @@ def standard_scrypt(cls, memory_cost, parallelization, block_size, derived_key_l
             UserImportHash: A new ``UserImportHash``.
         """
         data = {
-            'memoryCost': _auth_utils.validate_int(memory_cost, 'memory_cost', low=0),
+            'cpuMemCost': _auth_utils.validate_int(memory_cost, 'memory_cost', low=0),
             'parallelization': _auth_utils.validate_int(parallelization, 'parallelization', low=0),
             'blockSize': _auth_utils.validate_int(block_size, 'block_size', low=0),
             'dkLen': _auth_utils.validate_int(derived_key_length, 'derived_key_length', low=0),

tests/test_user_mgt.py

@@ -1212,7 +1212,7 @@ def test_standard_scrypt(self):
             memory_cost=14, parallelization=2, block_size=10, derived_key_length=128)
         expected = {
             'hashAlgorithm': 'STANDARD_SCRYPT',
-            'memoryCost': 14,
+            'cpuMemCost': 14,
             'parallelization': 2,
             'blockSize': 10,
             'dkLen': 128,