Use view functions in FlowManager (onflow#228)

NtTestAlert · Apr 10, 2024 · 1d66db1 · 1d66db1
1 parent 6a11947
commit 1d66db1
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 8 deletions.
diff --git a/.changeset/late-trees-ring.md b/.changeset/late-trees-ring.md
@@ -0,0 +1,5 @@
+---
+"@onflow/flow-js-testing": patch
+---
+
+Use `view` functions in FlowManager
diff --git a/cadence/contracts/FlowManager.cdc b/cadence/contracts/FlowManager.cdc
@@ -6,7 +6,7 @@ access(all) contract FlowManager {
     access(all) struct Mapper {
         access(all) let accounts: {String: Address}
 
-        access(all) fun getAddress(_ name: String): Address? {
+        access(all) view fun getAddress(_ name: String): Address? {
             return self.accounts[name]
         }
 
@@ -20,7 +20,7 @@ access(all) contract FlowManager {
         }
     }
 
-    access(all) fun getAccountAddress(_ name: String): Address?{
+    access(all) view fun getAccountAddress(_ name: String): Address?{
         let accountManager = self.account
             .capabilities.borrow<&FlowManager.Mapper>(self.accountManagerPath)!
 
@@ -67,12 +67,12 @@ access(all) contract FlowManager {
         emit FlowManager.TimestampOffsetChanged(offset: offset)
     }
 
-    access(all) fun getBlockHeight(): UInt64 {
+    access(all) view fun getBlockHeight(): UInt64 {
         var block = getCurrentBlock()
         return block.height + self.blockOffset
     }
 
-    access(all) fun getBlockTimestamp(): UFix64 {
+    access(all) view fun getBlockTimestamp(): UFix64 {
         var block = getCurrentBlock()
         return block.timestamp + self.timestampOffset
     }

diff --git a/src/generated/contracts/FlowManager.js b/src/generated/contracts/FlowManager.js
@@ -17,7 +17,7 @@ access(all) contract FlowManager {
     access(all) struct Mapper {
         access(all) let accounts: {String: Address}
 
-        access(all) fun getAddress(_ name: String): Address? {
+        access(all) view fun getAddress(_ name: String): Address? {
             return self.accounts[name]
         }
 
@@ -31,7 +31,7 @@ access(all) contract FlowManager {
         }
     }
 
-    access(all) fun getAccountAddress(_ name: String): Address?{
+    access(all) view fun getAccountAddress(_ name: String): Address?{
         let accountManager = self.account
             .capabilities.borrow<&FlowManager.Mapper>(self.accountManagerPath)!
 
@@ -78,12 +78,12 @@ access(all) contract FlowManager {
         emit FlowManager.TimestampOffsetChanged(offset: offset)
     }
 
-    access(all) fun getBlockHeight(): UInt64 {
+    access(all) view fun getBlockHeight(): UInt64 {
         var block = getCurrentBlock()
         return block.height + self.blockOffset
     }
 
-    access(all) fun getBlockTimestamp(): UFix64 {
+    access(all) view fun getBlockTimestamp(): UFix64 {
         var block = getCurrentBlock()
         return block.timestamp + self.timestampOffset
     }