CU-86drpndk1 - Change how to add a method to the permissions

boa3_test/test_sc/metadata_test/MetadataInfoPermissions.py

@@ -10,7 +10,7 @@ def permissions_manifest() -> NeoMetadata:
     meta = NeoMetadata()
 
     # the contract needs permission to call this method from any contract
-    meta.add_permission(methods=['onNEP17Payment'])
+    meta.add_permission(methods='onNEP17Payment')
 
     # the contract needs permission to call this method from a specific contract
     meta.add_permission(contract='0x3846a4aa420d9831044396dd3a56011514cd10e3', methods=['get_object'])

boa3_test/test_sc/metadata_test/MetadataInfoPermissionsEmptyList.py

@@ -0,0 +1,14 @@
+from boa3.builtin.compile_time import NeoMetadata, public
+
+
+@public
+def main() -> int:
+    return 5
+
+
+def permissions_manifest() -> NeoMetadata:
+    meta = NeoMetadata()
+
+    meta.add_permission(contract='*', methods=[])
+
+    return meta

boa3_test/test_sc/metadata_test/MetadataInfoPermissionsMismatchedType.py

@@ -15,13 +15,14 @@ def permissions_manifest() -> NeoMetadata:
     meta.add_permission(methods=b'onNEP17Payment')
     meta.add_permission(methods=123)
     meta.add_permission(methods=True)
-    meta.add_permission(methods='onNEP17Payment')
+    meta.add_permission(methods=(1, 2, 3))
+    meta.add_permission(methods=(b'onNEP17Payment', 123, True))
 
     meta.add_permission(contract=b'12345678901234567890')
     meta.add_permission(contract=123)
     meta.add_permission(contract=True)
     meta.add_permission(contract=['0x3846a4aa420d9831044396dd3a56011514cd10e3'])
 
-    meta.add_permission(contract=123, methods='onNEP17Payment')
+    meta.add_permission(contract=123, methods=b'onNEP17Payment')
 
     return meta

boa3_test/test_sc/metadata_test/MetadataInfoPermissionsTuple.py

@@ -0,0 +1,14 @@
+from boa3.builtin.compile_time import NeoMetadata, public
+
+
+@public
+def main() -> int:
+    return 5
+
+
+def permissions_manifest() -> NeoMetadata:
+    meta = NeoMetadata()
+
+    meta.add_permission(contract='*', methods=('total_supply', 'onNEP17Payment'))
+
+    return meta

boa3_test/test_sc/metadata_test/MetadataInfoPermissionsWildcardList.py

@@ -0,0 +1,14 @@
+from boa3.builtin.compile_time import NeoMetadata, public
+
+
+@public
+def main() -> int:
+    return 5
+
+
+def permissions_manifest() -> NeoMetadata:
+    meta = NeoMetadata()
+
+    meta.add_permission(contract='*', methods=['*', 'onNEP17Payment'])
+
+    return meta

boa3_test/test_sc/metadata_test/MetadataInfoPermissionsWildcardListSingleElement.py

@@ -0,0 +1,14 @@
+from boa3.builtin.compile_time import NeoMetadata, public
+
+
+@public
+def main() -> int:
+    return 5
+
+
+def permissions_manifest() -> NeoMetadata:
+    meta = NeoMetadata()
+
+    meta.add_permission(contract='*', methods=['*'])
+
+    return meta

boa3_test/test_sc/metadata_test/MetadataInfoPermissionsWildcardTuple.py

@@ -0,0 +1,14 @@
+from boa3.builtin.compile_time import NeoMetadata, public
+
+
+@public
+def main() -> int:
+    return 5
+
+
+def permissions_manifest() -> NeoMetadata:
+    meta = NeoMetadata()
+
+    meta.add_permission(contract='*', methods=('*', 'onNEP17Payment'))
+
+    return meta

boa3_test/tests/compiler_tests/test_metadata.py

@@ -438,6 +438,23 @@ def test_metadata_info_permissions_default(self):
         self.assertIsInstance(manifest['permissions'], list)
         self.assertEqual(len(manifest['permissions']), 0)
 
+    def test_metadata_info_permissions_empty_list(self):
+        path = self.get_contract_path('MetadataInfoPermissionsEmptyList.py')
+        output, manifest = self.compile_and_save(path)
+
+        self.assertIn('permissions', manifest)
+        self.assertIsInstance(manifest['permissions'], list)
+        self.assertEqual(len(manifest['permissions']), 0)
+
+    def test_metadata_info_permissions_tuple(self):
+        path = self.get_contract_path('MetadataInfoPermissionsTuple.py')
+        output, manifest = self.compile_and_save(path)
+
+        self.assertIn('permissions', manifest)
+        self.assertIsInstance(manifest['permissions'], list)
+        self.assertEqual(len(manifest['permissions']), 1)
+        self.assertIn({"contract": "*", "methods": ['total_supply', 'onNEP17Payment']}, manifest['permissions'])
+
     async def test_metadata_info_permissions_wildcard(self):
         path = self.get_contract_path('MetadataInfoPermissionsWildcard.py')
         output, manifest = self.compile_and_save(path)
@@ -452,6 +469,33 @@ async def test_metadata_info_permissions_wildcard(self):
         result, _ = await self.call('main', [], return_type=int)
         self.assertEqual(result, 100_000_000)    # NEO total supply
 
+    async def test_metadata_info_permissions_wildcard_list_single_element(self):
+        path = self.get_contract_path('MetadataInfoPermissionsWildcardListSingleElement.py')
+        output, manifest = self.compile_and_save(path)
+
+        self.assertIn('permissions', manifest)
+        self.assertIsInstance(manifest['permissions'], list)
+        self.assertEqual(len(manifest['permissions']), 1)
+        self.assertIn({"contract": "*", "methods": "*"}, manifest['permissions'])
+
+    async def test_metadata_info_permissions_wildcard_list(self):
+        path = self.get_contract_path('MetadataInfoPermissionsWildcardList.py')
+        output, manifest = self.compile_and_save(path)
+
+        self.assertIn('permissions', manifest)
+        self.assertIsInstance(manifest['permissions'], list)
+        self.assertEqual(len(manifest['permissions']), 1)
+        self.assertIn({"contract": "*", "methods": "*"}, manifest['permissions'])
+
+    async def test_metadata_info_permissions_wildcard_tuple(self):
+        path = self.get_contract_path('MetadataInfoPermissionsWildcardTuple.py')
+        output, manifest = self.compile_and_save(path)
+
+        self.assertIn('permissions', manifest)
+        self.assertIsInstance(manifest['permissions'], list)
+        self.assertEqual(len(manifest['permissions']), 1)
+        self.assertIn({"contract": "*", "methods": "*"}, manifest['permissions'])
+
     def test_metadata_info_permissions_native_contract(self):
         path = self.get_contract_path('MetadataInfoPermissionsNativeContract.py')
         output, manifest = self.compile_and_save(path)