Rename mapping functions to follow same patterns and conventions
This commit is contained in:
@@ -1,8 +1,8 @@
|
||||
import { outputJson, readJson } from "fs-extra";
|
||||
import { join } from "path";
|
||||
import { VariantAnalysisRepositoryTask } from "../shared/variant-analysis";
|
||||
import { mapRepoTaskToData } from "./repo-task-to-data-mapper";
|
||||
import { mapRepoTaskToDomain } from "./repo-task-to-domain-mapper";
|
||||
import { mapRepoTaskToDto } from "./repo-task-to-data-mapper";
|
||||
import { mapRepoTaskToDomainModel } from "./repo-task-to-domain-mapper";
|
||||
|
||||
export const REPO_TASK_FILENAME = "repo_task.json";
|
||||
|
||||
@@ -10,7 +10,7 @@ export async function writeRepoTask(
|
||||
storageDirectory: string,
|
||||
repoTask: VariantAnalysisRepositoryTask,
|
||||
): Promise<void> {
|
||||
const repoTaskData = mapRepoTaskToData(repoTask);
|
||||
const repoTaskData = mapRepoTaskToDto(repoTask);
|
||||
await outputJson(join(storageDirectory, REPO_TASK_FILENAME), repoTaskData);
|
||||
}
|
||||
|
||||
@@ -20,5 +20,5 @@ export async function readRepoTask(
|
||||
const repoTaskData = await readJson(
|
||||
join(storageDirectory, REPO_TASK_FILENAME),
|
||||
);
|
||||
return mapRepoTaskToDomain(repoTaskData);
|
||||
return mapRepoTaskToDomainModel(repoTaskData);
|
||||
}
|
||||
|
||||
@@ -8,7 +8,7 @@ import {
|
||||
VariantAnalysisRepoStatusDto,
|
||||
} from "./repo-tasks-dto";
|
||||
|
||||
export function mapRepoTaskToData(
|
||||
export function mapRepoTaskToDto(
|
||||
repoTask: VariantAnalysisRepositoryTask,
|
||||
): VariantAnalysisRepositoryTaskDto {
|
||||
return {
|
||||
@@ -17,7 +17,7 @@ export function mapRepoTaskToData(
|
||||
fullName: repoTask.repository.fullName,
|
||||
private: repoTask.repository.private,
|
||||
},
|
||||
analysisStatus: mapRepoTaskAnalysisStatusToData(repoTask.analysisStatus),
|
||||
analysisStatus: mapRepoTaskAnalysisStatusToDto(repoTask.analysisStatus),
|
||||
resultCount: repoTask.resultCount,
|
||||
artifactSizeInBytes: repoTask.artifactSizeInBytes,
|
||||
failureMessage: repoTask.failureMessage,
|
||||
@@ -27,7 +27,7 @@ export function mapRepoTaskToData(
|
||||
};
|
||||
}
|
||||
|
||||
function mapRepoTaskAnalysisStatusToData(
|
||||
function mapRepoTaskAnalysisStatusToDto(
|
||||
analysisStatus: VariantAnalysisRepoStatus,
|
||||
): VariantAnalysisRepoStatusDto {
|
||||
switch (analysisStatus) {
|
||||
|
||||
@@ -8,7 +8,7 @@ import {
|
||||
VariantAnalysisRepoStatusDto,
|
||||
} from "./repo-tasks-dto";
|
||||
|
||||
export function mapRepoTaskToDomain(
|
||||
export function mapRepoTaskToDomainModel(
|
||||
repoTask: VariantAnalysisRepositoryTaskDto,
|
||||
): VariantAnalysisRepositoryTask {
|
||||
return {
|
||||
@@ -17,7 +17,9 @@ export function mapRepoTaskToDomain(
|
||||
fullName: repoTask.repository.fullName,
|
||||
private: repoTask.repository.private,
|
||||
},
|
||||
analysisStatus: mapRepoTaskAnalysisStatusToDomain(repoTask.analysisStatus),
|
||||
analysisStatus: mapRepoTaskAnalysisStatusToDomainModel(
|
||||
repoTask.analysisStatus,
|
||||
),
|
||||
resultCount: repoTask.resultCount,
|
||||
artifactSizeInBytes: repoTask.artifactSizeInBytes,
|
||||
failureMessage: repoTask.failureMessage,
|
||||
@@ -27,7 +29,7 @@ export function mapRepoTaskToDomain(
|
||||
};
|
||||
}
|
||||
|
||||
function mapRepoTaskAnalysisStatusToDomain(
|
||||
function mapRepoTaskAnalysisStatusToDomainModel(
|
||||
analysisStatus: VariantAnalysisRepoStatusDto,
|
||||
): VariantAnalysisRepoStatus {
|
||||
switch (analysisStatus) {
|
||||
|
||||
Reference in New Issue
Block a user