fix: Check reserve is listed when setting as collateral (#684)

* fix: reserve existance check when set as collateral

* fix: gas snapshots

Author: avniculae

snapshots/Spoke.Operations.json

@@ -5,10 +5,10 @@
   "liquidationCall: partial": "384217",
   "permitReserve + repay (multicall)": "269705",
   "permitReserve + supply (multicall)": "140588",
-  "permitReserve + supply + enable collateral (multicall)": "173977",
+  "permitReserve + supply + enable collateral (multicall)": "174118",
   "repay: full": "173610",
   "repay: partial": "228487",
-  "supply + enable collateral (multicall)": "152410",
+  "supply + enable collateral (multicall)": "152551",
   "supply: 0 borrows, collateral disabled": "116422",
   "supply: 0 borrows, collateral enabled": "119569",
   "supply: 1 borrow": "119558",
@@ -17,8 +17,8 @@
   "updateUserDynamicConfig: 2 collaterals": "58377",
   "updateUserRiskPremium: 1 borrow": "122190",
   "updateUserRiskPremium: 2 borrows": "158175",
-  "usingAsCollateral: 0 borrows, enable": "53930",
-  "usingAsCollateral: 1 borrow, disable": "132392",
+  "usingAsCollateral: 0 borrows, enable": "54071",
+  "usingAsCollateral: 1 borrow, disable": "132533",
   "usingAsCollateral: 1 borrow, enable": "24831",
   "withdraw: 0 borrows, full": "136817",
   "withdraw: 0 borrows, partial": "137740",

src/contracts/Spoke.sol

@@ -739,6 +739,7 @@ contract Spoke is ISpoke, Multicall, AccessManaged, EIP712 {
     DataTypes.Reserve storage reserve,
     bool usingAsCollateral
   ) internal view {
+    require(address(reserve.hub) != address(0), ReserveNotListed());
     require(!reserve.paused, ReservePaused());
     // deactivation should be allowed
     require(!usingAsCollateral || !reserve.frozen, ReserveFrozen());

tests/unit/Spoke/Spoke.SetUsingAsCollateral.t.sol

@@ -7,6 +7,13 @@ import 'tests/unit/Spoke/SpokeBase.t.sol';
 contract SpokeConfigTest is SpokeBase {
   using SafeCast for uint256;
 
+  function test_setUsingAsCollateral_revertsWith_ReserveNotListed() public {
+    uint256 reserveCount = spoke1.getReserveCount();
+    vm.prank(alice);
+    vm.expectRevert(ISpoke.ReserveNotListed.selector);
+    spoke1.setUsingAsCollateral(reserveCount, true, alice);
+  }
+
   function test_setUsingAsCollateral_revertsWith_ReserveFrozen() public {
     uint256 daiReserveId = _daiReserveId(spoke1);