Skip to content

Commit

Permalink
FIX: Sonarfiks (#480)
Browse files Browse the repository at this point in the history
* FIX: Sonarfiks - public tester
  • Loading branch information
pekern authored Mar 13, 2024
1 parent 787a25d commit a557b6b
Show file tree
Hide file tree
Showing 105 changed files with 705 additions and 705 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {


@Test
public void skalIkkeFåAvklaringsbehovArbeidstakerMedBG() {
void skalIkkeFåAvklaringsbehovArbeidstakerMedBG() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.SELVSTENDIG_NÆRINGSDRIVENDE, BigDecimal.TEN);
Expand All @@ -46,7 +46,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovArbeidstakerUtenGradering() {
void skalIkkeFåAvklaringsbehovArbeidstakerUtenGradering() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.ARBEIDSTAKER, BigDecimal.ZERO);
Expand All @@ -60,7 +60,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalFåAvklaringsbehovArbeidstakerNårGraderingOgHarIkkeBG() {
void skalFåAvklaringsbehovArbeidstakerNårGraderingOgHarIkkeBG() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand All @@ -84,7 +84,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalFåAvklaringsbehovSelvstendigNårGraderingOgHarIkkeBG() {
void skalFåAvklaringsbehovSelvstendigNårGraderingOgHarIkkeBG() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand All @@ -106,7 +106,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovSelvstendigNårGraderingOgHarBG() {
void skalIkkeFåAvklaringsbehovSelvstendigNårGraderingOgHarBG() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.SELVSTENDIG_NÆRINGSDRIVENDE, BigDecimal.TEN);
Expand All @@ -120,7 +120,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovSelvstendigNårIkkeGradering() {
void skalIkkeFåAvklaringsbehovSelvstendigNårIkkeGradering() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.SELVSTENDIG_NÆRINGSDRIVENDE, BigDecimal.ZERO);
Expand All @@ -134,7 +134,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalFåAvklaringsbehovFrilanserNårGraderingOgHarIkkeBG() {
void skalFåAvklaringsbehovFrilanserNårGraderingOgHarIkkeBG() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand All @@ -156,7 +156,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovFrilanserNårGraderingOgHarBG() {
void skalIkkeFåAvklaringsbehovFrilanserNårGraderingOgHarBG() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.FRILANSER, BigDecimal.TEN);
Expand All @@ -170,7 +170,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovFrilanserNårIkkeGradering() {
void skalIkkeFåAvklaringsbehovFrilanserNårIkkeGradering() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.FRILANSER, BigDecimal.ZERO);
Expand All @@ -184,7 +184,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovFrilanserNårGraderingUtenforPeriodeUtenBeregningsgrunnlag() {
void skalIkkeFåAvklaringsbehovFrilanserNårGraderingUtenforPeriodeUtenBeregningsgrunnlag() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, SKJÆRINGSTIDSPUNKT.plusMonths(3));
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.FRILANSER, BigDecimal.ZERO);
Expand All @@ -198,7 +198,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovSNNårGraderingUtenforPeriodeUtenBeregningsgrunnlag() {
void skalIkkeFåAvklaringsbehovSNNårGraderingUtenforPeriodeUtenBeregningsgrunnlag() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT.plusMonths(2), null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.FRILANSER, BigDecimal.ZERO);
Expand All @@ -212,7 +212,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovArbeidstakerNårGraderingUtenforPeriodeUtenBeregningsgrunnlag2() {
void skalIkkeFåAvklaringsbehovArbeidstakerNårGraderingUtenforPeriodeUtenBeregningsgrunnlag2() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, SKJÆRINGSTIDSPUNKT.plusMonths(3));
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.ARBEIDSTAKER, BigDecimal.ZERO);
Expand All @@ -226,7 +226,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeFåAvklaringsbehovNårAAP() {
void skalIkkeFåAvklaringsbehovNårAAP() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.ARBEIDSAVKLARINGSPENGER, BigDecimal.ZERO);
Expand All @@ -239,7 +239,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_ikke_finne_andel_når_det_er_sn_med_gradering_med_inntekt_på_grunnlag() {
void skal_ikke_finne_andel_når_det_er_sn_med_gradering_med_inntekt_på_grunnlag() {
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.SELVSTENDIG_NÆRINGSDRIVENDE, BigDecimal.TEN);

Expand All @@ -254,7 +254,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_ikke_finne_andel_når_det_er_gradering_men_ikke_fastsatt_redusert_pr_år() {
void skal_ikke_finne_andel_når_det_er_gradering_men_ikke_fastsatt_redusert_pr_år() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.ARBEIDSTAKER, null);
Expand All @@ -270,7 +270,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_ikke_finne_andel_når_det_er_gradering_men_fastsatt_grunnlag_over_0_redusert_pr_år() {
void skal_ikke_finne_andel_når_det_er_gradering_men_fastsatt_grunnlag_over_0_redusert_pr_år() {
// Arrange
var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(SKJÆRINGSTIDSPUNKT, null);
lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.ARBEIDSTAKER, BigDecimal.TEN);
Expand All @@ -286,7 +286,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_finne_andel_når_det_er_gradering() {
void skal_finne_andel_når_det_er_gradering() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand All @@ -311,7 +311,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_finne_andel_når_det_er_sn_med_gradering_uten_inntekt_på_grunnlag() {
void skal_finne_andel_når_det_er_sn_med_gradering_uten_inntekt_på_grunnlag() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand All @@ -336,7 +336,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_finne_riktig_andel_når_det_er_flere_med_gradering_men_kun_en_mangler_inntekt() {
void skal_finne_riktig_andel_når_det_er_flere_med_gradering_men_kun_en_mangler_inntekt() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand Down Expand Up @@ -365,7 +365,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skal_gi_false_når_to_andeler_i_graderingsperiode_men_ikke_0_på_andel_som_skal_graderes() {
void skal_gi_false_når_to_andeler_i_graderingsperiode_men_ikke_0_på_andel_som_skal_graderes() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand All @@ -391,7 +391,7 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
}

@Test
public void skalIkkeSlåUtNårDetManglerBGMenGraderingErUtenforArbeidsperiode() {
void skalIkkeSlåUtNårDetManglerBGMenGraderingErUtenforArbeidsperiode() {
// Arrange
LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
LocalDate graderingFom = skjæringstidspunkt;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ private BeregningsgrunnlagPeriodeDto lagPeriode(BeregningsgrunnlagDto bg) {
}

@Test
public void skal_matche_på_andelsnr() {
void skal_matche_på_andelsnr() {
// Arrange
var arbId = InternArbeidsforholdRefDto.nullRef();
Long andelsnr = 1L;
Expand All @@ -55,7 +55,7 @@ private BeregningsgrunnlagPeriodeDto lagPeriode(BeregningsgrunnlagDto bg) {
}

@Test
public void skal_matche_på_arbid_om_andelsnr_input_er_null() {
void skal_matche_på_arbid_om_andelsnr_input_er_null() {
// Arrange
var arbId = InternArbeidsforholdRefDto.nyRef();
Long andelsnr = 1L;
Expand All @@ -78,7 +78,7 @@ private BeregningsgrunnlagPeriodeDto lagPeriode(BeregningsgrunnlagDto bg) {
}

@Test
public void skal_kaste_exception_om_andel_ikkje_eksisterer_i_grunnlag() {
void skal_kaste_exception_om_andel_ikkje_eksisterer_i_grunnlag() {
// Arrange
var arbId = InternArbeidsforholdRefDto.nyRef();
Long andelsnr = 1L;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
public class OpptjeningsaktiviteterPerYtelseTest {

@Test
public void aap_relevant_for_foreldrepenger() {
void aap_relevant_for_foreldrepenger() {
// Act
var opptjeningsaktiviteterPerYtelse = new OpptjeningsaktiviteterPerYtelse(FagsakYtelseType.FORELDREPENGER);
boolean relevant = opptjeningsaktiviteterPerYtelse.erRelevantAktivitet(OpptjeningAktivitetType.AAP, null);
Expand All @@ -21,7 +21,7 @@ public void aap_relevant_for_foreldrepenger() {
}

@Test
public void dp_relevant_for_foreldrepenger() {
void dp_relevant_for_foreldrepenger() {
// Act
var opptjeningsaktiviteterPerYtelse = new OpptjeningsaktiviteterPerYtelse(FagsakYtelseType.FORELDREPENGER);
boolean relevant = opptjeningsaktiviteterPerYtelse.erRelevantAktivitet(OpptjeningAktivitetType.DAGPENGER, null);
Expand All @@ -31,7 +31,7 @@ public void dp_relevant_for_foreldrepenger() {
}

@Test
public void aap_ikke_relevant_for_svp() {
void aap_ikke_relevant_for_svp() {
// Act
var opptjeningsaktiviteterPerYtelse = new OpptjeningsaktiviteterPerYtelse(FagsakYtelseType.SVANGERSKAPSPENGER);
boolean relevant = opptjeningsaktiviteterPerYtelse.erRelevantAktivitet(OpptjeningAktivitetType.AAP, null);
Expand All @@ -41,7 +41,7 @@ public void aap_ikke_relevant_for_svp() {
}

@Test
public void dp_ikke_relevant_for_svp() {
void dp_ikke_relevant_for_svp() {
// Act
var opptjeningsaktiviteterPerYtelse = new OpptjeningsaktiviteterPerYtelse(FagsakYtelseType.SVANGERSKAPSPENGER);
boolean relevant = opptjeningsaktiviteterPerYtelse.erRelevantAktivitet(OpptjeningAktivitetType.DAGPENGER, null);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ public class MapBeregningAktiviteterFraRegelTilVLTest {
private static final String ORGNR = "900050001";

@Test
public void mapFrilanserOgArbeidstakerAktiviteter() {
void mapFrilanserOgArbeidstakerAktiviteter() {
// Arrange
AktivitetStatusModell regelmodell = new AktivitetStatusModell();
LocalDate idag = LocalDate.now();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public class MapBeregningsgrunnlagFraRegelTilVLTest {
private KoblingReferanse koblingReferanse = new KoblingReferanseMock();

@Test
public void testMappingBGForArbeidstaker() {
void testMappingBGForArbeidstaker() {
final BeregningsgrunnlagDto vlBG = buildVLBGForATOgFL();

no.nav.folketrygdloven.beregningsgrunnlag.regelmodell.resultat.Beregningsgrunnlag resultatGrunnlag = buildRegelBGForAT();
Expand All @@ -80,7 +80,7 @@ public void testMappingBGForArbeidstaker() {
}

@Test
public void skal_mappe_beregningsgrunnlag_når_arbeidsgiver_er_privatperson() {
void skal_mappe_beregningsgrunnlag_når_arbeidsgiver_er_privatperson() {
// Arrange
LocalDate skjæringstidspunkt = LocalDate.of(2018, 1, 1);
LocalDate førsteUttaksdag = skjæringstidspunkt.plusWeeks(2);
Expand Down Expand Up @@ -110,7 +110,7 @@ public void testMappingBGForArbeidstaker() {
}

@Test
public void skal_mappe_beregningsgrunnlag_når_arbeidsgiver_er_virksomhet() {
void skal_mappe_beregningsgrunnlag_når_arbeidsgiver_er_virksomhet() {
// Arrange
LocalDate skjæringstidspunkt = LocalDate.of(2018, 1, 1);
LocalDate førsteUttaksdag = skjæringstidspunkt.plusWeeks(2);
Expand Down Expand Up @@ -139,7 +139,7 @@ public void testMappingBGForArbeidstaker() {
}

@Test
public void skal_mappe_beregningsgrunnlag_for_næring() {
void skal_mappe_beregningsgrunnlag_for_næring() {
// Arrange
LocalDate skjæringstidspunkt = LocalDate.of(2018, 1, 1);
LocalDate førsteUttaksdag = skjæringstidspunkt.plusWeeks(2);
Expand All @@ -161,7 +161,7 @@ public void testMappingBGForArbeidstaker() {


@Test
public void skalSetteRiktigSammenligningsgrunnlagPrStatusNårBeregningsgrunnlagInneholderSammenlingningsgrunnlagForFL() {
void skalSetteRiktigSammenligningsgrunnlagPrStatusNårBeregningsgrunnlagInneholderSammenlingningsgrunnlagForFL() {
final BeregningsgrunnlagDto vlBG = buildVLBGForATOgFL();

no.nav.folketrygdloven.beregningsgrunnlag.regelmodell.resultat.Beregningsgrunnlag resultatGrunnlag = buildRegelBGForFL();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ public class MapFastsettBeregningsgrunnlagPerioderFraRegelTilVLTest {


@Test
public void skalMappeBeregningsgrunnlagUtenSplittNaturalytelse() {
void skalMappeBeregningsgrunnlagUtenSplittNaturalytelse() {
// Arrange
BeregningsgrunnlagDto vlBeregningsgrunnlag = lagBeregningsgrunnlag();
byggAktivitetStatus(vlBeregningsgrunnlag);
Expand All @@ -64,7 +64,7 @@ public void skalMappeBeregningsgrunnlagUtenSplittNaturalytelse() {
}

@Test
public void skalMappeBeregningsgrunnlagUtenSplittRefusjonOgGradering() {
void skalMappeBeregningsgrunnlagUtenSplittRefusjonOgGradering() {
// Arrange
BeregningsgrunnlagDto vlBeregningsgrunnlag = lagBeregningsgrunnlag();
byggAktivitetStatus(vlBeregningsgrunnlag);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class MapFraFordelingsmodellTest {
private BeregningsgrunnlagDto.Builder bg = BeregningsgrunnlagDto.builder().medSkjæringstidspunkt(STP);

@Test
public void skal_teste_mapping_av_enkel_andel() {
void skal_teste_mapping_av_enkel_andel() {
// Arrange
var ref = UUID.randomUUID();
lagBGPeriode(STP, STP.plusMonths(1),
Expand All @@ -48,7 +48,7 @@ public void skal_teste_mapping_av_enkel_andel() {
}

@Test
public void skal_teste_mapping_av_to_andeler_blir_tre() {
void skal_teste_mapping_av_to_andeler_blir_tre() {
// Arrange
var ref = UUID.randomUUID();
lagBGPeriode(STP, STP.plusMonths(1),
Expand Down Expand Up @@ -77,7 +77,7 @@ public void skal_teste_mapping_av_to_andeler_blir_tre() {
}

@Test
public void skal_teste_mapping_av_tre_andeler_blir_fire_med_ny_andel_frilans() {
void skal_teste_mapping_av_tre_andeler_blir_fire_med_ny_andel_frilans() {
// Arrange
lagBGPeriode(STP, STP.plusMonths(1),
lagBGAndel(1L, AktivitetStatus.BRUKERS_ANDEL,
Expand Down Expand Up @@ -111,7 +111,7 @@ public void skal_teste_mapping_av_tre_andeler_blir_fire_med_ny_andel_frilans() {
}

@Test
public void skal_teste_mapping_av_tre_andeler_blir_fem() {
void skal_teste_mapping_av_tre_andeler_blir_fem() {
// Arrange
var ref = UUID.randomUUID();
var ref2 = UUID.randomUUID();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ public class KopierBeregningsgrunnlagUtilTest {
private static final Periode periode = Periode.of(LocalDate.now().minusMonths(3), LocalDate.now().minusMonths(1));

@Test
public void skalKopiereBeregninggrunnlagPeriode() {
void skalKopiereBeregninggrunnlagPeriode() {
// Arrange
BeregningsgrunnlagPeriode orginal = lagPeriode();
BeregningsgrunnlagPeriode kopi = BeregningsgrunnlagPeriode.builder()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
public class AktivitetKodeverkMappingTest {

@Test
public void skal_verifisere_at_beregningsreglene_kjenner_alle_opptjeningsaktiviteter_i_kodeverk() {
void skal_verifisere_at_beregningsreglene_kjenner_alle_opptjeningsaktiviteter_i_kodeverk() {
for (OpptjeningAktivitetType kode : EnumSet.allOf(OpptjeningAktivitetType.class)) {
//TODO(OJR) skal fjerne UTDANNINGSPERMISJON fra kodeverk
if (!OpptjeningAktivitetType.UDEFINERT.equals(kode) && !kode.getKode().equals("UTDANNINGSPERMISJON")) {
Expand Down
Loading

0 comments on commit a557b6b

Please sign in to comment.