Skip to content

Commit f21117e

Browse files
committed
Extract read and write to store
1 parent fcc9376 commit f21117e

File tree

2 files changed

+23
-8
lines changed

2 files changed

+23
-8
lines changed
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
import { outputJson, readJson } from "fs-extra";
2+
import { join } from "path";
3+
import { VariantAnalysisRepositoryTask } from "../shared/variant-analysis";
4+
5+
export const REPO_TASK_FILENAME = "repo_task.json";
6+
7+
export function writeRepoTask(
8+
storageDirectory: string,
9+
repoTask: VariantAnalysisRepositoryTask,
10+
): Promise<void> {
11+
return outputJson(join(storageDirectory, REPO_TASK_FILENAME), repoTask);
12+
}
13+
14+
export function readRepoTask(
15+
storageDirectory: string,
16+
): Promise<VariantAnalysisRepositoryTask> {
17+
return readJson(join(storageDirectory, REPO_TASK_FILENAME));
18+
}

extensions/ql-vscode/src/variant-analysis/variant-analysis-results-manager.ts

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { appendFile, pathExists, mkdir, outputJson, readJson } from "fs-extra";
1+
import { appendFile, pathExists, mkdir } from "fs-extra";
22
import fetch from "node-fetch";
33
import { EOL } from "os";
44
import { join } from "path";
@@ -17,6 +17,7 @@ import {
1717
import { DisposableObject, DisposeHandler } from "../pure/disposable-object";
1818
import { EventEmitter } from "vscode";
1919
import { unzipFile } from "../pure/zip";
20+
import { readRepoTask, writeRepoTask } from "./store/repo-task-store";
2021

2122
type CacheKey = `${number}/${string}`;
2223

@@ -37,7 +38,6 @@ export type LoadResultsOptions = {
3738
};
3839

3940
export class VariantAnalysisResultsManager extends DisposableObject {
40-
private static readonly REPO_TASK_FILENAME = "repo_task.json";
4141
private static readonly RESULTS_DIRECTORY = "results";
4242

4343
private readonly cachedResults: Map<
@@ -82,10 +82,7 @@ export class VariantAnalysisResultsManager extends DisposableObject {
8282
await mkdir(resultDirectory, { recursive: true });
8383
}
8484

85-
await outputJson(
86-
join(resultDirectory, VariantAnalysisResultsManager.REPO_TASK_FILENAME),
87-
repoTask,
88-
);
85+
await writeRepoTask(resultDirectory, repoTask);
8986

9087
const zipFilePath = join(resultDirectory, "results.zip");
9188

@@ -184,8 +181,8 @@ export class VariantAnalysisResultsManager extends DisposableObject {
184181
repositoryFullName,
185182
);
186183

187-
const repoTask: VariantAnalysisRepositoryTask = await readJson(
188-
join(storageDirectory, VariantAnalysisResultsManager.REPO_TASK_FILENAME),
184+
const repoTask: VariantAnalysisRepositoryTask = await readRepoTask(
185+
storageDirectory,
189186
);
190187

191188
if (!repoTask.databaseCommitSha || !repoTask.sourceLocationPrefix) {

0 commit comments

Comments
 (0)