Skip to content

add offce unit for nav id#438

Merged
its-me2000 merged 3 commits intomainfrom
AVIO-529
Jan 27, 2026
Merged

add offce unit for nav id#438
its-me2000 merged 3 commits intomainfrom
AVIO-529

Conversation

@its-me2000
Copy link
Copy Markdown
Contributor

get Office unit nomId for users navId

@its-me2000 its-me2000 requested a review from a team January 26, 2026 06:40
Copy link
Copy Markdown
Contributor

@Triopticon Triopticon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Til info og implementering i fremtida!

Fint om du ungår å bruke variable navn som ikkje gir noko meining som t i denne: for (NOM_RessursOrgTilknytning t : ressurs.orgTilknytning). Dette følger ikkje Nav sin Teknisk retning om at det skal vera enkelt å lesa og forstå. Sjølv om det er i ein for loop bør denne ha eit forståelig navn som sier kva den er til.
System.Test

Samt fint om du tar i bruk den nye Assert klassen og bruker det slik: System.Assert.areEqual(expected, actual, msg) i stede for den gamle utdaterte måten som er brukt her: System.assertEquals('aa333a', result, 'should be Office Unit Id aa333a');.

Det er heller ikkje brukt System.Test.startTest() og System.Test.stopTest(), det er noko som alltid bør brukast, og det skal vera ein veldig god grunn til ikkje å bruke dette.
System.Test

Bruk også alltid System framfor både Test og Assert klassene for å sikre at dei fungerer sjølv om nokon overskriver Assert og Test klassene.

@its-me2000 its-me2000 merged commit 35c4b8b into main Jan 27, 2026
5 checks passed
@its-me2000 its-me2000 deleted the AVIO-529 branch January 27, 2026 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants