Test updates for the change in behavior

- Makes two additional tests futures since we had to back out the quick fix
- Updates the comments on three tests for the new state of affairs
- Modifies the future for ternaryPlainRef to reflect its new behavior
- Remove execopts for ternaryPlainRef now that it fails more broadly instead of
  just with memory leaks

----
Signed-off-by: Lydia Duncan <[email protected]>

test/types/string/ternaryPlain.future

@@ -0,0 +1,2 @@
+bug: strings in ternary expressions will sometimes segfault
+#25032

test/types/string/ternaryPlainRef.future

@@ -1,4 +1,2 @@
-bug: const ref strings in ternary expressions sometimes leak memory
-#25417
-
-When this future is resolved, please remove the .execopts file
+bug: strings in ternary expressions will sometimes segfault
+#25032

test/types/string/ternaryReturn.chpl

@@ -1,8 +1,8 @@
-// This test is to lock in that a bug was resolved involving returning strings
-// from nested ternary returns
+// This test is to lock in a bug involving returning strings from nested ternary
+// returns
 
 // This is the version that originally failed.  Please do not modify the return
-// statement, to ensure the test continues to lock in the fix
+// statement, to ensure the test continues to lock in any future fix
 proc positive(z : string) {
   param EMPTY = '';
   return if z == '~' then ' ' else (if z == '+' then z else EMPTY);

test/types/string/ternaryReturn.future

@@ -0,0 +1,2 @@
+bug: strings in ternary expressions will sometimes segfault
+#25032

test/types/string/ternaryReturn2.chpl

@@ -1,8 +1,8 @@
-// This test is to lock in that a bug was resolved involving returning strings
+// This test is to lock in behavior around a bug involving returning strings
 // from nested ternary returns
 
 // This is not the version that failed, it was added to ensure that the cases
-// that worked before were not broken by the solution
+// that worked before are not broken by the solution
 proc positive(z : string) {
   param EMPTY = '';
   const temp = if z == '+' then z else EMPTY;

test/types/string/ternaryReturn3.chpl

@@ -1,4 +1,4 @@
-// This test is to lock in that a bug was resolved involving returning strings
+// This test is to lock in behavior around a bug involving returning strings
 // from nested ternary returns
 
 // This is not the version that failed, it was added to ensure that the cases