Skip to content

Add mock candidates to election creation tests #218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Esar1212 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add mock candidates to election creation tests #218

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c977bf6
Add mock candidates to election creation tests
Esar1212 Jan 7, 2026
caa625d
Fix candidateID values in ElectionFactory tests
Esar1212 Jan 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 21 additions & 6 deletions blockchain/test/ElectionFactory.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,21 @@ describe("ElectionFactory", function () {
maxVotes: 3,
options: ["Option 1", "Option 2", "Option 3"]
};
// candidateID is required for tests, although the contract overwrites it during initialization
// This ensures consistent test behavior
const mockCandidates = [
{
name: "Candidate 1",
description: "First candidate",
candidateID: 0
},
{
name: "Candidate 2",
description: "Second candidate",
candidateID: 1
}
];


beforeEach(async function () {
[owner, user1, user2, router] = await ethers.getSigners();
Expand All @@ -35,26 +50,26 @@ describe("ElectionFactory", function () {

describe("Election Creation", function () {
it("Should create a new election", async function () {
await factory.createElection(mockElectionInfo, 0, 0);
await factory.createElection(mockElectionInfo,mockCandidates, 0, 0);
expect(await factory.electionCount()).to.equal(1);

const openElections = await factory.getOpenElections();
expect(openElections.length).to.equal(1);
});

it("Should allow multiple elections creation", async function () {
await factory.createElection(mockElectionInfo, 0, 0);
await factory.createElection(mockElectionInfo, 0, 0);
await factory.createElection(mockElectionInfo,mockCandidates, 0, 0);
await factory.createElection(mockElectionInfo,mockCandidates, 0, 0);

expect(await factory.electionCount()).to.equal(2);
const openElections = await factory.getOpenElections();
expect(openElections.length).to.equal(2);
});
});

describe("Election Deletion", function () {
beforeEach(async function () {
await factory.createElection(mockElectionInfo, 0, 0);
await factory.createElection(mockElectionInfo, mockCandidates,0, 0);
});

it("Should allow owner to delete their election", async function () {
Expand Down Expand Up @@ -89,4 +104,4 @@ describe("ElectionFactory", function () {
).to.be.revertedWithCustomError(factory, "OwnerRestricted");
});
});
});
});