-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TB-58: Energi, oppvarming, vannforsyning, avløp og noen bruksareal fra matrikkelen #90
Conversation
Dependency ReviewThe following issues were found:
License Issuessettings.gradle.kts
Allowed Licenses: MIT, Apache-2.0, Apache-2.0 AND BSD-3-Clause OpenSSF ScorecardScorecard details
Scanned Manifest Files.github/workflows/dependency-review-pr.yaml
settings.gradle.kts
|
Hvordan skal vi forholde oss til feilede Dependency Reviews som det her? 🤔 |
src/main/kotlin/no/kartverket/matrikkel/bygning/models/Bygning.kt
Outdated
Show resolved
Hide resolved
matrikkel-api/src/main/kotlin/no/kartverket/matrikkel/bygning/matrikkelapi/XmlDate.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/no/kartverket/matrikkel/bygning/matrikkel/adapters/MatrikkelBygningClient.kt
Show resolved
Hide resolved
src/main/kotlin/no/kartverket/matrikkel/bygning/matrikkel/adapters/MatrikkelBygningClient.kt
Show resolved
Hide resolved
src/main/kotlin/no/kartverket/matrikkel/bygning/matrikkel/adapters/MatrikkelBygningClient.kt
Show resolved
Hide resolved
|
||
class MatrikkelBygningClientTest { | ||
@Test | ||
fun bygningUnummerertBruksenhetUtenEtasje() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Litt smak og behag, men liker ofte selv en type given_when_then
stil på testnavnene. Man kan også benytte følgende syntaks for å kunne skrive mellomnavn i testnavnet.
fun `gitt en bygning med unummerert bruksenhet ...`()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jeg sliter med disse rare fnuttene av en eller annen grunn. Jeg føler meg også egentlig ikke helt ferdig med testing-biten, men hvis jeg skal vente på det, så blir det bare mer merge conflicts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tenker kanskje testene gjerne kan omdøpes @kvstrant, jeg er også fan av å beskrive test casen litt mer i funksjonsnavnet til testen
src/main/kotlin/no/kartverket/matrikkel/bygning/matrikkel/adapters/KodeMappers.kt
Show resolved
Hide resolved
matrikkel-api/src/main/kotlin/no/kartverket/matrikkel/bygning/matrikkelapi/Ids.kt
Show resolved
Hide resolved
...rc/testFixtures/kotlin/no/kartverket/matrikkel/bygning/matrikkelapi/builders/DatoBuilders.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gjerne juster på testnavnene, men ikke nøye nok til at jeg ikke bare godkjenner nå :)
No description provided.