pam/integration-tests: Do not use utf-8 ellipses in broker messages

These are not supported by some SSH clients, so better to be reliable and support characters that are visible the same ways in all the known cases
ubuntu · Oct 10, 2024 · fc8e735 · fc8e735
1 parent b458ad9
commit fc8e735
Show file tree

Hide file tree

Showing 41 changed files with 1,302 additions and 1,302 deletions.
diff --git a/examplebroker/broker.go b/examplebroker/broker.go
@@ -340,7 +340,7 @@ func getSupportedModes(sessionInfo sessionInfo, supportedUILayouts []map[string]
 				if layout["button"] == "optional" {
 					allModes["totp_with_button"] = map[string]string{
 						"selection_label": "Authentication code",
-						"phone":           "+33…",
+						"phone":           "+33...",
 						"wantedCode":      "temporary pass",
 						"ui": mapToJSON(map[string]string{
 							"type":   "form",
@@ -352,7 +352,7 @@ func getSupportedModes(sessionInfo sessionInfo, supportedUILayouts []map[string]
 				} else {
 					allModes["totp"] = map[string]string{
 						"selection_label": "Authentication code",
-						"phone":           "+33…",
+						"phone":           "+33...",
 						"wantedCode":      "temporary pass",
 						"ui": mapToJSON(map[string]string{
 							"type":  "form",
@@ -363,21 +363,21 @@ func getSupportedModes(sessionInfo sessionInfo, supportedUILayouts []map[string]
 				}
 
 				allModes["phoneack1"] = map[string]string{
-					"selection_label": "Use your phone +33…",
-					"phone":           "+33…",
+					"selection_label": "Use your phone +33...",
+					"phone":           "+33...",
 					"ui": mapToJSON(map[string]string{
 						"type":  "form",
-						"label": "Unlock your phone +33… or accept request on web interface:",
+						"label": "Unlock your phone +33... or accept request on web interface:",
 						"wait":  "true",
 					}),
 				}
 
 				allModes["phoneack2"] = map[string]string{
-					"selection_label": "Use your phone +1…",
-					"phone":           "+1…",
+					"selection_label": "Use your phone +1...",
+					"phone":           "+1...",
 					"ui": mapToJSON(map[string]string{
 						"type":  "form",
-						"label": "Unlock your phone +1… or accept request on web interface",
+						"label": "Unlock your phone +1... or accept request on web interface",
 						"wait":  "true",
 					}),
 				}

diff --git a/pam/integration-tests/gdm_test.go b/pam/integration-tests/gdm_test.go
@@ -83,7 +83,7 @@ var testFidoDeviceUILayout = authd.UILayout{
 
 var testPhoneAckUILayout = authd.UILayout{
 	Type:    "form",
-	Label:   ptrValue("Unlock your phone +33… or accept request on web interface:"),
+	Label:   ptrValue("Unlock your phone +33... or accept request on web interface:"),
 	Content: ptrValue(""),
 	Wait:    ptrValue("true"),
 	Button:  ptrValue(""),

diff --git a/...tegration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_switching_auth_mode b/...tegration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_switching_auth_mode
@@ -136,8 +136,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 
@@ -202,8 +202,8 @@ Click on the link received at [email protected] or enter th
   1. Password authentication
 > 2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 
@@ -268,8 +268,8 @@ Plug your fido device and press with your thumb
   1. Password authentication
   2. Send URL to [email protected]
 > 3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...ion-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_switching_to_local_broker b/...ion-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_switching_to_local_broker
@@ -136,8 +136,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...on-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_form_mode_with_button b/...on-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_form_mode_with_button
@@ -136,8 +136,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/pam/integration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_mfa b/pam/integration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_mfa
@@ -136,8 +136,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 
@@ -233,7 +233,7 @@ Plug your fido device and press with your thumb
   Select your authentication method
 
 > 1. Use your fido device foo
-  2. Use your phone +33…
+  2. Use your phone +33...
   3. Authentication code
 
 
@@ -296,7 +296,7 @@ Plug your fido device and press with your thumb
 
 ────────────────────────────────────────────────────────────────────────────────
 > ./pam_authd login socket=${AUTHD_TESTS_CLI_AUTHENTICATE_TESTS_SOCK}
-Unlock your phone +33… or accept request on web interface:
+Unlock your phone +33... or accept request on web interface:
 
 
 
@@ -331,7 +331,7 @@ Unlock your phone +33… or accept request on web interface:
 > ./pam_authd login socket=${AUTHD_TESTS_CLI_AUTHENTICATE_TESTS_SOCK}
   Select your authentication method
 
-> 1. Use your phone +33…
+> 1. Use your phone +33...
   2. Authentication code
 
 
@@ -362,7 +362,7 @@ Unlock your phone +33… or accept request on web interface:
 
 ────────────────────────────────────────────────────────────────────────────────
 > ./pam_authd login socket=${AUTHD_TESTS_CLI_AUTHENTICATE_TESTS_SOCK}
-Unlock your phone +33… or accept request on web interface:
+Unlock your phone +33... or accept request on web interface:
 
 
 

diff --git a/pam/integration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code b/pam/integration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code
@@ -70,8 +70,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...stdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_after_many_regenerations b/...stdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_after_many_regenerations
@@ -176,8 +176,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...gration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_in_a_tty b/...gration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_in_a_tty
@@ -90,8 +90,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_in_a_tty_session b/...tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_in_a_tty_session
@@ -90,8 +90,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...ration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_in_screen b/...ration-tests/testdata/TestCLIAuthenticate/golden/authenticate_user_with_qr_code_in_screen
@@ -90,8 +90,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...ntegration-tests/testdata/TestCLIAuthenticate/golden/prevent_user_from_switching_username b/...ntegration-tests/testdata/TestCLIAuthenticate/golden/prevent_user_from_switching_username
@@ -103,8 +103,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/...ration-tests/testdata/TestCLIAuthenticate/golden/remember_last_successful_broker_and_mode b/...ration-tests/testdata/TestCLIAuthenticate/golden/remember_last_successful_broker_and_mode
@@ -103,8 +103,8 @@ Username: user-integration-remember-mode
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 

diff --git a/pam/integration-tests/testdata/TestCLIChangeAuthTok/golden/change_passwd_after_mfa_auth b/pam/integration-tests/testdata/TestCLIChangeAuthTok/golden/change_passwd_after_mfa_auth
@@ -136,8 +136,8 @@ Gimme your password
 > 1. Password authentication
   2. Send URL to [email protected]
   3. Use your fido device foo
-  4. Use your phone +33…
-  5. Use your phone +1…
+  4. Use your phone +33...
+  5. Use your phone +1...
   6. Use a QR code
   7. Authentication code
 
@@ -233,7 +233,7 @@ Plug your fido device and press with your thumb
   Select your authentication method
 
 > 1. Use your fido device foo
-  2. Use your phone +33…
+  2. Use your phone +33...
   3. Authentication code
 
 
@@ -296,7 +296,7 @@ Plug your fido device and press with your thumb
 
 ────────────────────────────────────────────────────────────────────────────────
 > ./pam_authd passwd socket=${AUTHD_TESTS_CLI_AUTHTOK_TESTS_SOCK}
-Unlock your phone +33… or accept request on web interface:
+Unlock your phone +33... or accept request on web interface:
 
 
 
@@ -331,7 +331,7 @@ Unlock your phone +33… or accept request on web interface:
 > ./pam_authd passwd socket=${AUTHD_TESTS_CLI_AUTHTOK_TESTS_SOCK}
   Select your authentication method
 
-> 1. Use your phone +33…
+> 1. Use your phone +33...
   2. Authentication code
 
 
@@ -362,7 +362,7 @@ Unlock your phone +33… or accept request on web interface:
 
 ────────────────────────────────────────────────────────────────────────────────
 > ./pam_authd passwd socket=${AUTHD_TESTS_CLI_AUTHTOK_TESTS_SOCK}
-Unlock your phone +33… or accept request on web interface:
+Unlock your phone +33... or accept request on web interface: