Skip to content

Commit 9d7ea8e

Browse files
Make test names clearer
1 parent 68c41d3 commit 9d7ea8e

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

extensions/ql-vscode/src/view/variant-analysis/__tests__/VariantAnalysisStats.spec.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ describe(VariantAnalysisStats.name, () => {
141141
).not.toBeInTheDocument();
142142
});
143143

144-
it("renders a view logs link when the variant analysis status is succeeded", () => {
144+
it("renders 'View logs' link when the variant analysis status is succeeded", () => {
145145
render({
146146
variantAnalysisStatus: VariantAnalysisStatus.Succeeded,
147147
completedAt: new Date(),
@@ -151,25 +151,25 @@ describe(VariantAnalysisStats.name, () => {
151151
expect(onViewLogsClick).toHaveBeenCalledTimes(1);
152152
});
153153

154-
it("renders a running text when the variant analysis status is in progress", () => {
154+
it("renders 'Running' text when the variant analysis status is in progress", () => {
155155
render({ variantAnalysisStatus: VariantAnalysisStatus.InProgress });
156156

157157
expect(screen.getByText("Running")).toBeInTheDocument();
158158
});
159159

160-
it("renders a failed text when the variant analysis status is failed", () => {
160+
it("renders 'Failed' text when the variant analysis status is failed", () => {
161161
render({ variantAnalysisStatus: VariantAnalysisStatus.Failed });
162162

163163
expect(screen.getByText("Failed")).toBeInTheDocument();
164164
});
165165

166-
it("renders a stopped text when the variant analysis status is canceled", () => {
166+
it("renders 'Stopped' text when the variant analysis status is canceled", () => {
167167
render({ variantAnalysisStatus: VariantAnalysisStatus.Canceled });
168168

169169
expect(screen.getByText("Stopped")).toBeInTheDocument();
170170
});
171171

172-
it("renders a some analyses failed text when the overall variant analysis status is succeeded but not all analyses successful", () => {
172+
it("renders 'Some analyses failed' text when the overall variant analysis status is succeeded but not all analyses successful", () => {
173173
render({
174174
variantAnalysisStatus: VariantAnalysisStatus.Succeeded,
175175
completedRepositoryCount: 10,
@@ -179,13 +179,13 @@ describe(VariantAnalysisStats.name, () => {
179179
expect(screen.getByText("Some analyses failed")).toBeInTheDocument();
180180
});
181181

182-
it("renders a succeeded text when the variant analysis status is succeeded and successful repository count omitted", () => {
182+
it("renders 'Succeeded' text when the variant analysis status is succeeded and successful repository count omitted", () => {
183183
render({ variantAnalysisStatus: VariantAnalysisStatus.Succeeded });
184184

185185
expect(screen.getByText("Succeeded")).toBeInTheDocument();
186186
});
187187

188-
it("renders a succeeded text when the variant analysis status is succeeded and successful repository count equals total repository count", () => {
188+
it("renders 'Succeeded' text when the variant analysis status is succeeded and successful repository count equals total repository count", () => {
189189
render({
190190
variantAnalysisStatus: VariantAnalysisStatus.Succeeded,
191191
completedRepositoryCount: 10,

0 commit comments

Comments
 (0)