Fix RecordPlaintext.owner() for private records and correct OwnedRecord property names in record-provider

Author: marshacb

sdk/src/record-provider.ts

@@ -262,9 +262,9 @@ class NetworkRecordProvider implements RecordProvider {
         const recordsPts = await this.networkClient.findRecords(startHeight, endHeight, searchParameters.unspent, ["credits.aleo"], microcredits, maxAmount, searchParameters.nonces, this.account.privateKey());
         return recordsPts.map((record) => ({
                 owner: record.owner().toString(),
-                programName: 'credits.aleo',
-                recordName: 'credits',
-                recordPlaintext: record.toString(),
+                program_name: 'credits.aleo',
+                record_name: 'credits',
+                record_plaintext: record.toString(),
         }));
     }
 

wasm/src/record/record_plaintext.rs

@@ -32,6 +32,7 @@ use crate::{
             CurrentNetwork,
             FieldNative,
             IdentifierNative,
+            LiteralNative,
             PlaintextEntryNative,
             PlaintextNative,
             ProgramIDNative,
@@ -108,7 +109,10 @@ impl RecordPlaintext {
     pub fn owner(&self) -> Result<Address, String> {
         match self.0.owner() {
             Owner::<CurrentNetwork, PlaintextNative>::Public(owner) => Ok(Address::from(*owner)),
-            _ => Err("Record is not public".to_string()),
+            Owner::<CurrentNetwork, PlaintextNative>::Private(plaintext) => match plaintext {
+                PlaintextNative::Literal(LiteralNative::Address(address), _) => Ok(Address::from(*address)),
+                _ => Err("Private owner is not an address literal".to_string()),
+            },
         }
     }
 
@@ -469,6 +473,20 @@ mod tests {
         assert!(RecordPlaintext::from_string(invalid_bech32).is_err());
     }
 
+    #[wasm_bindgen_test]
+    fn test_owner_with_private_visibility() {
+        let record = RecordPlaintext::from_string(CREDITS_RECORD).unwrap();
+        let owner = record.owner().unwrap();
+        assert_eq!(owner.to_string(), "aleo1j7qxyunfldj2lp8hsvy7mw5k8zaqgjfyr72x2gh3x4ewgae8v5gscf5jh3");
+    }
+
+    #[wasm_bindgen_test]
+    fn test_owner_with_private_visibility_battleship() {
+        let record = RecordPlaintext::from_string(BATTLESHIP_RECORD).unwrap();
+        let owner = record.owner().unwrap();
+        assert_eq!(owner.to_string(), "aleo1kh5t7m30djl0ecdn4f5vuzp7dx0tcwh7ncquqjkm4matj2p2zqpqm6at48");
+    }
+
     #[wasm_bindgen_test]
     fn test_record_decrypt_record_sender_ciphertext() {
         // Get the private key corresponding to the record.