Create voucher with , VoucherHub address, Set account as owner, expiration and deal with Authorization

contracts/VoucherHub.sol

@@ -158,8 +158,6 @@ contract VoucherHub is OwnableUpgradeable, UUPSUpgradeable, IVoucherHub {
      * @dev Note: the same account could have 2 voucher instances if the "beaconAddress"
      * changes, but this should not happen since the beacon is upgradeable, hence the
      * address should never be changed.
-     *
-
      * @param owner The address of the voucher owner.
      * @param voucherType The ID of the voucher type.
      * @return voucherAddress The address of the created voucher contract.

contracts/beacon/IVoucher.sol

@@ -7,15 +7,18 @@ interface IVoucher {
     event AccountAuthorized(address indexed account);
     event AccountUnauthorized(address indexed account);
 
-    function getExpiration() external view returns (uint256);
-
-    function getType() external view returns (uint256);
+    function initialize(
+        address owner,
+        address voucherHub,
+        uint256 expiration,
+        uint256 voucherTypeId
+    ) external;
 
     function getVoucherHub() external view returns (address);
+    function getExpiration() external view returns (uint256);
+    function getType() external view returns (uint256);
 
     function authorizeAccount(address account) external;
-
     function unauthorizeAccount(address account) external;
-
     function isAccountAuthorized(address account) external view returns (bool);
 }

contracts/beacon/Voucher.sol

@@ -38,21 +38,21 @@ contract Voucher is OwnableUpgradeable, IVoucher {
     /**
      * Initialize implementation contract.
      * @param owner The owner of the contract.
-     * @param voucherType The type of the voucher.
+     * @param voucherTypeId The type Id of the voucher.
      * @param expiration The expiration timestamp of the voucher.
      * @param voucherHub The address of the voucher hub.
      */
     function initialize(
         address owner,
         address voucherHub,
         uint256 expiration,
-        uint256 voucherType
+        uint256 voucherTypeId
     ) external initializer {
         __Ownable_init(owner);
         VoucherStorage storage $ = _getVoucherStorage();
         $._voucherHub = voucherHub;
         $._expiration = expiration;
-        $._type = voucherType;
+        $._type = voucherTypeId;
         // TODO: deposit sRLC.
     }
 

test/beacon/Voucher.test.ts

@@ -182,11 +182,15 @@ describe('Voucher', function () {
             await createVoucherTx.wait();
             const voucherAddress = await voucherHub.getVoucher(voucherOwner1);
             const voucher: Voucher = await commonUtils.getVoucher(voucherAddress);
-
-
+            await voucher.connect(voucherOwner1).authorizeAccount(anyone.address);
+            const anyoneIsAuthorized = await voucher.isAccountAuthorized(anyone.address);
+            // unauthorize the account
             await expect(
                 voucher.connect(anyone).unauthorizeAccount(anyone.address),
             ).to.be.revertedWithCustomError(voucher, 'OwnableUnauthorizedAccount');
+            // Check that the state of mapping is not modified from.
+            expect(await voucher.isAccountAuthorized(anyone.address)).to.equal(anyoneIsAuthorized)
+                .to.be.true;
         });
 
         it('Should not authorize owner account', async () => {