Skip to content

Commit 0ee090b

Browse files
authored
Merge pull request #2335 from github/starcke/rename-sup-modelled
Rename supported to modelled.
2 parents 3c0b3e4 + fb4f39d commit 0ee090b

File tree

5 files changed

+32
-34
lines changed

5 files changed

+32
-34
lines changed

extensions/ql-vscode/src/view/data-extensions-editor/DataExtensionsEditor.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import { ModeledMethod } from "../../data-extensions-editor/modeled-method";
1616
import { MethodRow } from "./MethodRow";
1717
import { assertNever } from "../../pure/helpers-pure";
1818
import { vscode } from "../vscode-api";
19-
import { calculateSupportedPercentage } from "./supported";
19+
import { calculateModeledPercentage } from "./modeled";
2020

2121
export const DataExtensionsEditorContainer = styled.div`
2222
margin-top: 1rem;
@@ -97,12 +97,12 @@ export function DataExtensionsEditor({
9797
};
9898
}, []);
9999

100-
const supportedPercentage = useMemo(
101-
() => calculateSupportedPercentage(externalApiUsages),
100+
const modeledPercentage = useMemo(
101+
() => calculateModeledPercentage(externalApiUsages),
102102
[externalApiUsages],
103103
);
104104

105-
const unsupportedPercentage = 100 - supportedPercentage;
105+
const unModeledPercentage = 100 - modeledPercentage;
106106

107107
const onChange = useCallback(
108108
(method: ExternalApiUsage, model: ModeledMethod) => {
@@ -140,10 +140,10 @@ export function DataExtensionsEditor({
140140
{externalApiUsages.length > 0 && (
141141
<>
142142
<div>
143-
<h3>External API support stats</h3>
143+
<h3>External API model stats</h3>
144144
<ul>
145-
<li>Supported: {supportedPercentage.toFixed(2)}%</li>
146-
<li>Unsupported: {unsupportedPercentage.toFixed(2)}%</li>
145+
<li>Modeled: {modeledPercentage.toFixed(2)}%</li>
146+
<li>Unmodeled: {unModeledPercentage.toFixed(2)}%</li>
147147
</ul>
148148
</div>
149149
<div>

extensions/ql-vscode/src/view/data-extensions-editor/MethodRow.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ export const MethodRow = ({
155155
value={modeledMethod?.type ?? "none"}
156156
onInput={handleTypeInput}
157157
>
158-
<VSCodeOption value="none">Unmodelled</VSCodeOption>
158+
<VSCodeOption value="none">Unmodeled</VSCodeOption>
159159
<VSCodeOption value="source">Source</VSCodeOption>
160160
<VSCodeOption value="sink">Sink</VSCodeOption>
161161
<VSCodeOption value="summary">Flow summary</VSCodeOption>

extensions/ql-vscode/src/view/data-extensions-editor/__tests__/supported.spec.ts renamed to extensions/ql-vscode/src/view/data-extensions-editor/__tests__/modeled.spec.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,33 +1,33 @@
1-
import { calculateSupportedPercentage } from "../supported";
1+
import { calculateModeledPercentage } from "../modeled";
22

3-
describe("calculateSupportedPercentage", () => {
3+
describe("calculateModeledPercentage", () => {
44
it("when there are no external API usages", () => {
5-
expect(calculateSupportedPercentage([])).toBe(0);
5+
expect(calculateModeledPercentage([])).toBe(0);
66
});
77

8-
it("when there are is 1 supported external API usage", () => {
8+
it("when there are is 1 modeled external API usage", () => {
99
expect(
10-
calculateSupportedPercentage([
10+
calculateModeledPercentage([
1111
{
1212
supported: true,
1313
},
1414
]),
1515
).toBe(100);
1616
});
1717

18-
it("when there are is 1 unsupported external API usage", () => {
18+
it("when there are is 1 unmodeled external API usage", () => {
1919
expect(
20-
calculateSupportedPercentage([
20+
calculateModeledPercentage([
2121
{
2222
supported: false,
2323
},
2424
]),
2525
).toBe(0);
2626
});
2727

28-
it("when there are multiple supporte and unsupported external API usage", () => {
28+
it("when there are multiple modeled and unmodeled external API usage", () => {
2929
expect(
30-
calculateSupportedPercentage([
30+
calculateModeledPercentage([
3131
{
3232
supported: false,
3333
},
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
import { ExternalApiUsage } from "../../data-extensions-editor/external-api-usage";
2+
3+
export function calculateModeledPercentage(
4+
externalApiUsages: Array<Pick<ExternalApiUsage, "supported">>,
5+
): number {
6+
if (externalApiUsages.length === 0) {
7+
return 0;
8+
}
9+
10+
const modeledExternalApiUsages = externalApiUsages.filter((m) => m.supported);
11+
12+
const modeledRatio =
13+
modeledExternalApiUsages.length / externalApiUsages.length;
14+
return modeledRatio * 100;
15+
}

extensions/ql-vscode/src/view/data-extensions-editor/supported.ts

Lines changed: 0 additions & 17 deletions
This file was deleted.

0 commit comments

Comments
 (0)