Fix: SettingsTest works

cathrkri · Apr 14, 2026 · 8f04d2d · 8f04d2d
1 parent 42218b7
commit 8f04d2d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 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
@@ -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
@@ -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());