Hey @vatsa287 @amarts !
All checks have passed..
Please review it.

@hardik-pratap-singh Even though the tests have passed, I doubt that else branch is actually being executed. Since initially asset type has been set to MF. And in if condition it is being checked if type is of any 3 valid types. So if part is executed and not else. So the error code is not being checked. Please add a log and check.

@vatsa287 ! I was also thinking the same. But everywhere I can find asset_type being initialized with MF. How can I actually get the asset_type ?

@hardik-pratap-singh Even though the tests have passed, I doubt that else branch is actually being executed. Since initially asset type has been set to MF. And in if condition it is being checked if type is of any 3 valid types. So if part is executed and not else. So the error code is not being checked. Please add a log and check.

@vatsa287 ! I was also thinking the same. But everywhere I can find asset_type being initialized with MF. How can I actually get the asset_type ?

Check pallets/src/types.rs where we define AssetTypeOf to be ,

pub enum AssetTypeOf {
	ART,
	BOND,
	MF,
}

@hardik-pratap-singh Even though the tests have passed, I doubt that else branch is actually being executed. Since initially asset type has been set to MF. And in if condition it is being checked if type is of any 3 valid types. So if part is executed and not else. So the error code is not being checked. Please add a log and check.

@vatsa287 ! I was also thinking the same. But everywhere I can find asset_type being initialized with MF. How can I actually get the asset_type ?

Check pallets/src/types.rs where we define AssetTypeOf to be ,

pub enum AssetTypeOf {
	ART,
	BOND,
	MF,
}

I wanted to ask that all of the values i.e. asset_desc , asset_tag , asset_meta , asset_qty , asset_value and asset_type are being hard coded right ? Or we are getting them dynamically from somewhere ?
If we don't have to hard code the value just like we did for asset_type as MF, what can we write for asset_type ?
Currently we have this : let asset_type = AssetTypeOf::MF

We can add a new asset type called Invalid and pass AssetTypeOf::Invalid to get the else part in function. But I don't think that will be a good practice.

@vatsa287 Can you help me with it ?

Hey @vatsa287 @NiranjanAP @amarts !
I am stuck at this issue.

Can you please guide me ?

@hardik-pratap-singh Regarding this, It is not possible to send a value/option which is not defined in the enum of valid asset types.
Moreover there are checks present in SDK through with the data is in bound to the chain. And on the chain too there is validity check present.
Since RUST is a compiled time language you cannot pass AssetTypeOf::Invalid in tests too which will throw it is not a part of enum declared.