Skip to content

Commit a8404a5

Browse files
committed
Throw from getModelsAsDataLanguage
1 parent edbc658 commit a8404a5

File tree

4 files changed

+6
-14
lines changed

4 files changed

+6
-14
lines changed

extensions/ql-vscode/src/model-editor/flow-model-queries.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -144,9 +144,6 @@ async function runSingleFlowQuery(
144144

145145
// Interpret the results
146146
const modelsAsDataLanguage = getModelsAsDataLanguage(language);
147-
if (!modelsAsDataLanguage) {
148-
throw new Error(`No models-as-data definition for ${language}`);
149-
}
150147

151148
const definition = modelsAsDataLanguage[type];
152149

extensions/ql-vscode/src/model-editor/languages/languages.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@ const languages: Partial<Record<QueryLanguage, ModelsAsDataLanguage>> = {
99

1010
export function getModelsAsDataLanguage(
1111
language: QueryLanguage,
12-
): ModelsAsDataLanguage | undefined {
13-
return languages[language];
12+
): ModelsAsDataLanguage {
13+
const definition = languages[language];
14+
if (!definition) {
15+
throw new Error(`No models-as-data definition for ${language}`);
16+
}
17+
return definition;
1418
}

extensions/ql-vscode/src/model-editor/yaml.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,6 @@ export function createDataExtensionYaml(
3939
modeledMethods: readonly ModeledMethod[],
4040
) {
4141
const modelsAsDataLanguage = getModelsAsDataLanguage(language);
42-
if (!modelsAsDataLanguage) {
43-
throw new Error(`No models as data language for ${language}`);
44-
}
4542

4643
const methodsByType: Record<
4744
Exclude<ModeledMethodType, "none">,
@@ -253,9 +250,6 @@ export function loadDataExtensionYaml(
253250
}
254251

255252
const modelsAsDataLanguage = getModelsAsDataLanguage(language);
256-
if (!modelsAsDataLanguage) {
257-
throw new Error(`No models as data language for ${language}`);
258-
}
259253

260254
const extensions = data.extensions;
261255

extensions/ql-vscode/src/view/model-editor/ModelKindDropdown.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,6 @@ export const ModelKindDropdown = ({
2424
}: Props) => {
2525
const predicate = useMemo(() => {
2626
const modelsAsDataLanguage = getModelsAsDataLanguage(language);
27-
if (!modelsAsDataLanguage) {
28-
throw new Error(`No models for language ${language}`);
29-
}
3027

3128
return modeledMethod?.type && modeledMethod.type !== "none"
3229
? modelsAsDataLanguage[modeledMethod.type]

0 commit comments

Comments
 (0)