Chore: fix tests, remove obsolete imports

This commit is contained in:
Pieter Vander Vennet 2025-04-22 01:45:34 +02:00
parent 3f577e7978
commit 2ea9236d37
6 changed files with 13 additions and 14 deletions

View file

@ -868,7 +868,9 @@ class LayerOverviewUtils extends Script {
id + id +
"(feat: Feature, helperFunctions: Record<ExtraFuncType, (feature: Feature) => Function>) {" "(feat: Feature, helperFunctions: Record<ExtraFuncType, (feature: Feature) => Function>) {"
) )
allCode.push(" const {" + ExtraFunctions.types.join(", ") + "} = helperFunctions") if (code?.length > 0) {
allCode.push(" const {" + ExtraFunctions.types.join(", ") + "} = helperFunctions")
}
for (const line of code) { for (const line of code) {
const firstEq = line.indexOf("=") const firstEq = line.indexOf("=")
let attributeName = line.substring(0, firstEq).trim() let attributeName = line.substring(0, firstEq).trim()

View file

@ -4,7 +4,6 @@ import { Changes } from "../Changes"
import { ChangeDescription } from "./ChangeDescription" import { ChangeDescription } from "./ChangeDescription"
import { And } from "../../Tags/And" import { And } from "../../Tags/And"
import { OsmWay } from "../OsmObject" import { OsmWay } from "../OsmObject"
import { GeoOperations } from "../../GeoOperations"
import InsertPointIntoWayAction from "./InsertPointIntoWayAction" import InsertPointIntoWayAction from "./InsertPointIntoWayAction"
export default class CreateNewNodeAction extends OsmCreateAction { export default class CreateNewNodeAction extends OsmCreateAction {

View file

@ -10,7 +10,6 @@
import SubtleButton from "../../Base/SubtleButton.svelte" import SubtleButton from "../../Base/SubtleButton.svelte"
import Translations from "../../i18n/Translations.js" import Translations from "../../i18n/Translations.js"
import TagHint from "../TagHint.svelte" import TagHint from "../TagHint.svelte"
import { And } from "../../../Logic/Tags/And.js"
import LoginToggle from "../../Base/LoginToggle.svelte" import LoginToggle from "../../Base/LoginToggle.svelte"
import Constants from "../../../Models/Constants.js" import Constants from "../../../Models/Constants.js"
import FilteredLayer from "../../../Models/FilteredLayer" import FilteredLayer from "../../../Models/FilteredLayer"
@ -37,7 +36,6 @@
import BaseUIElement from "../../BaseUIElement" import BaseUIElement from "../../BaseUIElement"
import TitledPanel from "../../Base/TitledPanel.svelte" import TitledPanel from "../../Base/TitledPanel.svelte"
import ThemeViewState from "../../../Models/ThemeViewState" import ThemeViewState from "../../../Models/ThemeViewState"
import SpecialTranslation from "../TagRendering/SpecialTranslation.svelte"
export let coordinate: { lon: number; lat: number } export let coordinate: { lon: number; lat: number }
export let state: ThemeViewState export let state: ThemeViewState

View file

@ -6,9 +6,8 @@
import type { SpecialVisualizationState } from "../SpecialVisualization" import type { SpecialVisualizationState } from "../SpecialVisualization"
import FilteredLayer from "../../Models/FilteredLayer" import FilteredLayer from "../../Models/FilteredLayer"
import FilterToggle from "./FilterToggle.svelte" import FilterToggle from "./FilterToggle.svelte"
import ToSvelte from "../Base/ToSvelte.svelte"
import Tr from "../Base/Tr.svelte" import Tr from "../Base/Tr.svelte"
import { Store, UIEventSource } from "../../Logic/UIEventSource" import { Store } from "../../Logic/UIEventSource"
import Translations from "../i18n/Translations" import Translations from "../i18n/Translations"
import type { FilterSearchResult } from "../../Logic/Search/FilterSearch" import type { FilterSearchResult } from "../../Logic/Search/FilterSearch"
import FilterSearch from "../../Logic/Search/FilterSearch" import FilterSearch from "../../Logic/Search/FilterSearch"

View file

@ -1,10 +1,9 @@
import { LayerConfigJson } from "../../../../src/Models/ThemeConfig/Json/LayerConfigJson" import { LayerConfigJson } from "../../../../src/Models/ThemeConfig/Json/LayerConfigJson"
import LineRenderingConfigJson from "../../../../src/Models/ThemeConfig/Json/LineRenderingConfigJson" import LineRenderingConfigJson from "../../../../src/Models/ThemeConfig/Json/LineRenderingConfigJson"
import { PrepareLayer, RewriteSpecial } from "../../../../src/Models/ThemeConfig/Conversion/PrepareLayer"
import { import {
PrepareLayer, QuestionableTagRenderingConfigJson
RewriteSpecial, } from "../../../../src/Models/ThemeConfig/Json/QuestionableTagRenderingConfigJson"
} from "../../../../src/Models/ThemeConfig/Conversion/PrepareLayer"
import { QuestionableTagRenderingConfigJson } from "../../../../src/Models/ThemeConfig/Json/QuestionableTagRenderingConfigJson"
import RewritableConfigJson from "../../../../src/Models/ThemeConfig/Json/RewritableConfigJson" import RewritableConfigJson from "../../../../src/Models/ThemeConfig/Json/RewritableConfigJson"
import { describe, expect, it } from "vitest" import { describe, expect, it } from "vitest"
@ -68,7 +67,7 @@ describe("PrepareLayer", () => {
const expected = { const expected = {
id: "testlayer", id: "testlayer",
source: { osmTags: "key=value" }, source: { osmTags: { "and": ["key=value"] } },
lineRendering: [ lineRendering: [
{ {
color: { color: {

View file

@ -2,7 +2,7 @@ import { ThemeConfigJson } from "../../../../src/Models/ThemeConfig/Json/ThemeCo
import { LayerConfigJson } from "../../../../src/Models/ThemeConfig/Json/LayerConfigJson" import { LayerConfigJson } from "../../../../src/Models/ThemeConfig/Json/LayerConfigJson"
import { PrepareTheme } from "../../../../src/Models/ThemeConfig/Conversion/PrepareTheme" import { PrepareTheme } from "../../../../src/Models/ThemeConfig/Conversion/PrepareTheme"
import ThemeConfig from "../../../../src/Models/ThemeConfig/ThemeConfig" import ThemeConfig from "../../../../src/Models/ThemeConfig/ThemeConfig"
import bookcaseLayer from "../../../../src/assets/generated/layers/public_bookcase.json" import bookcaseLayer from "../../../../public/assets/generated/layers/public_bookcase.json"
import LayerConfig from "../../../../src/Models/ThemeConfig/LayerConfig" import LayerConfig from "../../../../src/Models/ThemeConfig/LayerConfig"
import { ExtractImages } from "../../../../src/Models/ThemeConfig/Conversion/FixImages" import { ExtractImages } from "../../../../src/Models/ThemeConfig/Conversion/FixImages"
import cyclofix from "../../../../public/assets/generated/themes/cyclofix.json" import cyclofix from "../../../../public/assets/generated/themes/cyclofix.json"
@ -10,7 +10,9 @@ import { Tag } from "../../../../src/Logic/Tags/Tag"
import { DesugaringContext } from "../../../../src/Models/ThemeConfig/Conversion/Conversion" import { DesugaringContext } from "../../../../src/Models/ThemeConfig/Conversion/Conversion"
import { And } from "../../../../src/Logic/Tags/And" import { And } from "../../../../src/Logic/Tags/And"
import { describe, expect, it } from "vitest" import { describe, expect, it } from "vitest"
import { QuestionableTagRenderingConfigJson } from "../../../../src/Models/ThemeConfig/Json/QuestionableTagRenderingConfigJson" import {
QuestionableTagRenderingConfigJson
} from "../../../../src/Models/ThemeConfig/Json/QuestionableTagRenderingConfigJson"
import Constants from "../../../../src/Models/Constants" import Constants from "../../../../src/Models/Constants"
import { ConversionContext } from "../../../../src/Models/ThemeConfig/Conversion/ConversionContext" import { ConversionContext } from "../../../../src/Models/ThemeConfig/Conversion/ConversionContext"
import { MinimalTagRenderingConfigJson } from "../../../../src/Models/ThemeConfig/Json/TagRenderingConfigJson" import { MinimalTagRenderingConfigJson } from "../../../../src/Models/ThemeConfig/Json/TagRenderingConfigJson"
@ -188,7 +190,7 @@ describe("PrepareTheme", () => {
expect(rewritten.layers[1]).toEqual({ expect(rewritten.layers[1]).toEqual({
_basedOn: "layer-example", _basedOn: "layer-example",
source: { source: {
osmTags: "x=y", osmTags: { "and": ["x=y"] }
}, },
id: "layer-example", id: "layer-example",
name: null, name: null,