From 8f04d2d1879083a0a158801d0aa7fb389d7346e1 Mon Sep 17 00:00:00 2001 From: AdrianBalunan Date: Tue, 14 Apr 2026 17:17:51 +0200 Subject: [PATCH] Fix: SettingsTest works --- .../ntnu/systemutvikling/team6/models/user/Settings.java | 8 ++++---- .../systemutvikling/team6/models/user/SettingsTest.java | 9 ++++----- 2 files changed, 8 insertions(+), 9 deletions(-) diff --git a/helpmehelpapplication/src/main/java/ntnu/systemutvikling/team6/models/user/Settings.java b/helpmehelpapplication/src/main/java/ntnu/systemutvikling/team6/models/user/Settings.java index d814283..086ddb9 100644 --- a/helpmehelpapplication/src/main/java/ntnu/systemutvikling/team6/models/user/Settings.java +++ b/helpmehelpapplication/src/main/java/ntnu/systemutvikling/team6/models/user/Settings.java @@ -8,13 +8,13 @@ * @author Robin Strand Prestmo */ public class Settings { - private boolean lightMode; - private Language language; private boolean anonymous; + private Language language; + private boolean lightMode; /** Sets standard settings. LightMode enabled, language set to English, Anonymous disabled */ public Settings() { - this(true, Language.ENGLISH, false); + this(false, Language.ENGLISH, true); } /** @@ -24,7 +24,7 @@ public Settings() { * @param language choose language * @param anonymous choose if user is anonymous */ - public Settings(boolean lightMode, Language language, boolean anonymous) { + public Settings(boolean anonymous, Language language, boolean lightMode) { if (language == null) { throw new IllegalArgumentException("Language cannot be null"); } diff --git a/helpmehelpapplication/src/test/java/ntnu/systemutvikling/team6/models/user/SettingsTest.java b/helpmehelpapplication/src/test/java/ntnu/systemutvikling/team6/models/user/SettingsTest.java index 1cc8701..1b2191f 100644 --- a/helpmehelpapplication/src/test/java/ntnu/systemutvikling/team6/models/user/SettingsTest.java +++ b/helpmehelpapplication/src/test/java/ntnu/systemutvikling/team6/models/user/SettingsTest.java @@ -17,7 +17,7 @@ void testDefaultConstructorSetsStandardValues() { @Test void testCustomConstructorSetsValuesCorrectly() { - Settings settings = new Settings(false, Language.ENGLISH, true); + Settings settings = new Settings(true, Language.ENGLISH, false); assertFalse(settings.isLightMode()); assertEquals(Language.ENGLISH, settings.getLanguage()); @@ -33,16 +33,15 @@ void testConstructorThrowsExceptionWhenLanguageIsNull() { void testToggleLightMode() { Settings settings = new Settings(true, Language.ENGLISH, false); - settings.toggleLightMode(); - assertFalse(settings.isLightMode()); - settings.toggleLightMode(); assertTrue(settings.isLightMode()); + settings.toggleLightMode(); + assertFalse(settings.isLightMode()); } @Test void testToggleAnonymousMode() { - Settings settings = new Settings(true, Language.ENGLISH, false); + Settings settings = new Settings(false, Language.ENGLISH, false); settings.toggleAnonymousMode(); assertTrue(settings.isAnonymous());