Archive client #230
Archive client #230
Conversation
- Eksponerer Core til Archive og ArchiveTests - Legger til avhengigheter for å støtte logging
Digipost.Signature.Api.Client.Archive.Tests/Smoke/ArchiveSmokeTests.cs
Outdated
Show resolved
Hide resolved
| if(ex.InnerException is UnexpectedResponseException) | ||
| { | ||
| UnexpectedResponseException exception = (UnexpectedResponseException) ex.InnerException; | ||
| Assert.Equal("ARCHIVED_DOCUMENT_NOT_FOUND", exception.Error.Code); |
There was a problem hiding this comment.
Jeg tenkte på noe, @manuelAlonsoPerez. Kanskje det er litt vanskelig for en avsender å sjekke om InnerException er denne typen og har en error code. Kanskje vi kan gjøre en try/catch i ArchiveClient som fanger akkurat denne og lager en ny exception, f.eks DocumentNotFoundInArchiveException som har en god feilmelding?
There was a problem hiding this comment.
Høres ut som en god idé. Notér det bak øret 👂, og ta med det før final release.
|
Har lagt inn noen forslag her: #232 |
To align with the actual class name.
This is a "query" object encapsulating an ID, and not representing an actual ArchiveDocument.
|
Skrev om en test som ikke ville feilet dersom operasjonen ikke kastet (forventet) exception: 5d6f29a#diff-559953e4b3a2852a0967a834e9572865R30 Er denne good to go? Kan jeg bare merge denne og tagge som en |
|
Er litt PR-er fra dependabot. Skal vi bare squash-and-rebase de inn i denne branchen før merge? |
|
Sleng inn dependencies ja, @runeflobakk. Om det er noen knyttet til logging, sjekk gjerne at logging fungerer. Ellers er testene ganske heldekkende. |
2.x to 3.0.x
|
Fikk oppgradert Er en del PR-er som ser ut som går på internavhengigheter i prosjektet. Typ Portal avhenger av Core, og denne må oppgraderes fra 5.* til 6.*. Er dette noe man gjør manuelt? Altså, det ser ut som "current version" for biblioteket er v6, men hvor ser jeg dette, og skal interne avhengigheter oppdateres manuelt? Avhenger Portal p.t. av en tidligere versjon av Core? |
3 participants
💰 Funksjonell beskrivelse av endringen
Nytt Klient som skal handtere nedlasting av de migrerte lånekassen dokumentene
🏆 Interessante highlights
Del oppgave for Sign0010
https://trello.com/c/ukanYRAn/134-sign-010-migrering-av-l%C3%A5nekassens-signerte-avtaler