diff options
author | HampusM <hampus@hampusmat.com> | 2021-07-22 12:17:14 +0200 |
---|---|---|
committer | HampusM <hampus@hampusmat.com> | 2021-07-22 12:17:14 +0200 |
commit | 64fe51d449785c5f443c9adac20df274cd6a2284 (patch) | |
tree | f027f702d7318c83083cbe4a2b6d56177eaa3cee /test/unit | |
parent | f932f998452ab1fe25d3938444c0222453489bf7 (diff) |
Improved test names
Diffstat (limited to 'test/unit')
-rw-r--r-- | test/unit/commit.unit.test.ts | 14 | ||||
-rw-r--r-- | test/unit/repository.unit.test.ts | 20 |
2 files changed, 17 insertions, 17 deletions
diff --git a/test/unit/commit.unit.test.ts b/test/unit/commit.unit.test.ts index 5c4d8c7..904a67a 100644 --- a/test/unit/commit.unit.test.ts +++ b/test/unit/commit.unit.test.ts @@ -15,7 +15,7 @@ describe("Commit", () => { repository = await Repository.open(env.BASE_DIR, env.AVAIL_REPO); }); - it("Looks up a commit", async () => { + it("Should look up a commit", async () => { expect.assertions(8); const commit = await Commit.lookup(repository, env.AVAIL_COMMIT); @@ -26,19 +26,19 @@ describe("Commit", () => { expectCommitProperties(commit); }); - it("Looks up a nonexistant commit and throws", async () => { + it("Should look up a nonexistant commit and throw", async () => { expect.assertions(1); await expect(Commit.lookup(repository, env.UNAVAIL_COMMIT)).rejects.toThrow(); }); - it("Looks up if an commit that exists exist", async () => { + it("Should look up if an existent commit exists and respond true", async () => { expect.assertions(1); await expect(Commit.lookupExists(repository, env.AVAIL_COMMIT)).resolves.toBeTruthy(); }); - it("Looks up if an nonexistant commit exists", async () => { + it("Should look up if an nonexistant commit exists and respond false", async () => { expect.assertions(1); await expect(Commit.lookupExists(repository, env.UNAVAIL_COMMIT)).resolves.toBeFalsy(); @@ -51,7 +51,7 @@ describe("Commit", () => { commit = await repository.masterCommit(); }); - it("Gets the stats", async () => { + it("Should get the stats", async () => { expect.assertions(4); const stats = await commit.stats(); @@ -63,7 +63,7 @@ describe("Commit", () => { expect(stats).toHaveProperty("files_changed"); }); - it("Gets the diff", async () => { + it("Should get the diff", async () => { expect.assertions(2); const diff = await commit.diff(); @@ -72,7 +72,7 @@ describe("Commit", () => { expect(diff).toBeInstanceOf(Diff); }); - it("Gets the tree", async () => { + it("Should get the tree", async () => { expect.assertions(2); const tree = await commit.tree(); diff --git a/test/unit/repository.unit.test.ts b/test/unit/repository.unit.test.ts index 32a82e8..4fa851b 100644 --- a/test/unit/repository.unit.test.ts +++ b/test/unit/repository.unit.test.ts @@ -18,7 +18,7 @@ function expectRepositoryProperties(repository: Repository) { } describe("Repository", () => { - it("Opens a repository successfully", async () => { + it("Should open a repository", async () => { expect.assertions(8); const repository = await Repository.open(env.BASE_DIR, env.AVAIL_REPO); @@ -29,13 +29,13 @@ describe("Repository", () => { expectRepositoryProperties(repository); }); - it("Fails to open a nonexistant repository", async () => { + it("Should fail to open a nonexistant repository", async () => { expect.assertions(1); await expect(Repository.open(env.BASE_DIR, env.UNAVAIL_REPO)).rejects.toBeInstanceOf(BaseError); }); - it("Opens all repositories", async () => { + it("Should open all repositories", async () => { expect.hasAssertions(); const all_repositories = await Repository.openAll(env.BASE_DIR); @@ -57,19 +57,19 @@ describe("Repository", () => { repository = await Repository.open(env.BASE_DIR, env.AVAIL_REPO); }); - it("Looks up if an object that exists exist", async () => { + it("Should look up if an existent object exists and respond true", async () => { expect.assertions(1); await expect(repository.lookupExists(env.AVAIL_OBJECT)).resolves.toBeTruthy(); }); - it("Looks up if an nonexistant object exists", async () => { + it("Should look up if an nonexistant object exists and respond false", async () => { expect.assertions(1); await expect(repository.lookupExists(env.UNAVAIL_OBJECT)).resolves.toBeFalsy(); }); - it("Gets the master commit", async () => { + it("Should get the master commit", async () => { expect.assertions(8); const master_commit = await repository.masterCommit(); @@ -80,7 +80,7 @@ describe("Repository", () => { expectCommitProperties(master_commit); }); - it("Gets the commits", async () => { + it("Should get the commits", async () => { expect.hasAssertions(); const commits = await repository.commits(); @@ -95,7 +95,7 @@ describe("Repository", () => { } }); - it("Gets the tree", async () => { + it("Should get the tree", async () => { expect.assertions(2); const tree = await repository.tree(); @@ -104,7 +104,7 @@ describe("Repository", () => { expect(tree).toBeInstanceOf(Tree); }); - it("Gets the branches", async () => { + it("Should get the branches", async () => { expect.hasAssertions(); const branches = await repository.branches(); @@ -120,7 +120,7 @@ describe("Repository", () => { } }); - it("Gets the tags", async () => { + it("Should get the tags", async () => { expect.hasAssertions(); const tags = await repository.tags(); |