Skip to content

Commit

Permalink
Introduce method to compare only organisation part of Organisasjonsnu…
Browse files Browse the repository at this point in the history
…mmer

The changes introduced in the two previous commits change the behaviour
of the Organisasjonsnummer equals method. Previously, this method could
be used to compare only the organisation number string itself between two
instances of Organisasjonsnummer. The method introduced in this commit,
erSammeOrganisasjonsnummerUavhengigAvLandkode, is intended to replace
the described use of the equals method for uses where ignoring country
code is desirable.
  • Loading branch information
hermanwh committed Jan 9, 2024
1 parent cda56e4 commit fef09b7
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,10 @@ public boolean er(String organisasjonsnummerString) {
.isPresent();
}

public boolean erSammeOrganisasjonsnummerUavhengigAvLandkode(Organisasjonsnummer organisasjonsnummer) {
return this.organisasjonsnummer.equals(organisasjonsnummer.organisasjonsnummer);
}

public boolean erEnAv(Organisasjonsnummer... kandidater) {
return erEnAv(asList(kandidater));
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,23 @@ public void evaluates_other_strings_as_not_same() {
assertFalse(organisasjonsnummer.er("991825827"));
}

@Test
public void evaluates_other_organisasjonsnummer_as_same() {
Organisasjonsnummer withoutCountryCode = Organisasjonsnummer.of("984661185");
Organisasjonsnummer withNewCountryCode = Organisasjonsnummer.of(COUNTRY_CODE_ORGANIZATION_NUMBER_NORWAY_NEW + ":984661185");
Organisasjonsnummer withOldCountryCode = Organisasjonsnummer.of(COUNTRY_CODE_ORGANIZATION_NUMBER_NORWAY_OLD + ":984661185");
Organisasjonsnummer notEqual = Organisasjonsnummer.of("123456789");
assertTrue(withoutCountryCode.erSammeOrganisasjonsnummerUavhengigAvLandkode(withNewCountryCode));
assertTrue(withoutCountryCode.erSammeOrganisasjonsnummerUavhengigAvLandkode(withOldCountryCode));
assertTrue(withNewCountryCode.erSammeOrganisasjonsnummerUavhengigAvLandkode(withoutCountryCode));
assertTrue(withNewCountryCode.erSammeOrganisasjonsnummerUavhengigAvLandkode(withOldCountryCode));
assertTrue(withOldCountryCode.erSammeOrganisasjonsnummerUavhengigAvLandkode(withoutCountryCode));
assertTrue(withOldCountryCode.erSammeOrganisasjonsnummerUavhengigAvLandkode(withNewCountryCode));
assertFalse(notEqual.erSammeOrganisasjonsnummerUavhengigAvLandkode(withoutCountryCode));
assertFalse(notEqual.erSammeOrganisasjonsnummerUavhengigAvLandkode(withNewCountryCode));
assertFalse(notEqual.erSammeOrganisasjonsnummerUavhengigAvLandkode(withOldCountryCode));
}

@Test
public void correct_equals_and_hashcode() {
EqualsVerifier.forClass(Organisasjonsnummer.class).verify();
Expand Down

0 comments on commit fef09b7

Please sign in to comment.