Merge pull request #21 from arran4/fix-upper-indicator-delimiter-8071639884133601336

Fix UpperIndicator delimiter behavior

Author: arran4

edge_cases_test.go

@@ -86,13 +86,32 @@ func TestEdgeCases(t *testing.T) {
 		}
 
 		// Case B: Indicator != Delimiter
-		// With proposed fix, this should use the indicator as delimiter
+		// This should use the indicator as delimiter
 		res = ToFormattedCase(input, OptionDelimiter("-"), OptionUpperIndicator("="))
 		if res != "hello=world" {
 			t.Errorf("UpperIndicator Override: got %q, want %q", res, "hello=world")
 		}
 	})
 
+	// 5.1 UpperIndicator with MixCaseSupport (Consistency Check)
+	t.Run("UpperIndicator MixCase Consistency", func(t *testing.T) {
+		input := []Word{ExactCaseWord("helloWorld"), SingleCaseWord("foo")}
+
+		// Case A: Override behavior
+		// Expectation: hello=World=foo (UpperIndicator "=" overrides Delimiter "-")
+		res := ToFormattedCase(input, OptionDelimiter("-"), OptionUpperIndicator("="), OptionMixCaseSupport())
+		if res != "hello=World=foo" {
+			t.Errorf("UpperIndicator MixCase Override: got %q, want %q", res, "hello=World=foo")
+		}
+
+		// Case B: Double Delimiter behavior
+		// Expectation: hello--World--foo (UpperIndicator "-" matches Delimiter "-", so double delimiter)
+		res = ToFormattedCase(input, OptionDelimiter("-"), OptionUpperIndicator("-"), OptionMixCaseSupport())
+		if res != "hello--World--foo" {
+			t.Errorf("UpperIndicator MixCase Double: got %q, want %q", res, "hello--World--foo")
+		}
+	})
+
 	// 6. Empty and Nil Input
 	t.Run("Empty and Nil Input", func(t *testing.T) {
 		res := ToFormattedCase(nil)

types.go

@@ -249,6 +249,14 @@ func WordsToFormattedCase(words []Word, opts ...any) (string, error) {
 		}
 	}
 
+	if cfg.upperIndicator != "" {
+		if cfg.upperIndicator == cfg.delimiter {
+			cfg.delimiter = cfg.delimiter + cfg.delimiter
+		} else {
+			cfg.delimiter = cfg.upperIndicator
+		}
+	}
+
 	switch cfg.caseMode {
 	case CMScreaming:
 		cfg.screaming = true
@@ -324,15 +332,7 @@ func WordsToFormattedCase(words []Word, opts ...any) (string, error) {
 		result = append(result, w)
 	}
 
-	delimiter := cfg.delimiter
-	if cfg.upperIndicator != "" {
-		if cfg.upperIndicator == cfg.delimiter {
-			delimiter = cfg.delimiter + cfg.delimiter
-		} else {
-			delimiter = cfg.upperIndicator
-		}
-	}
-	final := strings.Join(result, delimiter)
+	final := strings.Join(result, cfg.delimiter)
 
 	if cfg.firstUpper {
 		final = UpperCaseFirst(final)