Merge pull request #119 from meetjn/election-removeCandidate

Bug: added edge case. closes #118

Author: Ronnieraj37

blockchain/contracts/Election.sol

@@ -11,6 +11,7 @@ contract Election is Initializable {
     error GetVotes();
     error ElectionIncomplete();
     error ElectionInactive();
+    error InvalidCandidateID();
 
     mapping(address user => bool isVoted) public userVoted;
 
@@ -120,10 +121,10 @@ contract Election is Initializable {
     }
 
     function removeCandidate(uint _id) external onlyOwner electionStarted {
-        candidates[_id] = candidates[candidates.length - 1];
-        candidates[_id].candidateID = _id;
-        candidates.pop();
-    }
+    if (_id >= candidates.length) revert InvalidCandidateID();
+    candidates[_id] = candidates[candidates.length - 1]; // Replace with last element
+    candidates.pop(); 
+}
 
     function getCandidateList() external view returns (Candidate[] memory) {
         return candidates;