changes from review

Author: joshua-spacetime

sdks/rust/tests/test-client/src/test_handlers.rs

@@ -88,8 +88,6 @@ pub async fn dispatch(test: &str, db_name: &str) {
         // "resubscribe" => exec_resubscribe(),
         //
         "reauth" => exec_reauth(db_name).await,
-        "reauth-part-1" => exec_reauth_part_1(db_name).await,
-        "reauth-part-2" => exec_reauth_part_2(db_name).await,
 
         "should-fail" => exec_should_fail(db_name).await,
 

sdks/rust/tests/test-counter/src/lib.rs

@@ -12,27 +12,36 @@ static SERVER_URL: OnceLock<String> = OnceLock::new();
 
 pub fn set_server_url(url: String) {
     if SERVER_URL.set(url).is_err() {
-        panic!("{TEST_SERVER_URL_ENV_VAR} was set more than once");
+        panic!("SDK test server URL was initialized more than once");
     }
 }
 
 pub fn server_url() -> &'static str {
-    SERVER_URL.get_or_init(server_url_from_env).as_str()
-}
-
-fn server_url_from_env() -> String {
     #[cfg(not(target_arch = "wasm32"))]
     {
-        std::env::var(TEST_SERVER_URL_ENV_VAR)
-            .unwrap_or_else(|_| panic!("{TEST_SERVER_URL_ENV_VAR} must be set by the SDK test harness"))
+        SERVER_URL.get_or_init(read_server_url_from_env).as_str()
     }
 
     #[cfg(target_arch = "wasm32")]
     {
-        panic!("{TEST_SERVER_URL_ENV_VAR} must be passed to the wasm SDK test harness")
+        SERVER_URL
+            .get()
+            .unwrap_or_else(|| {
+                panic!(
+                    "server URL was not initialized; \
+                     the wasm SDK test harness must read {TEST_SERVER_URL_ENV_VAR} and pass it to run()"
+                )
+            })
+            .as_str()
     }
 }
 
+#[cfg(not(target_arch = "wasm32"))]
+fn read_server_url_from_env() -> String {
+    std::env::var(TEST_SERVER_URL_ENV_VAR)
+        .unwrap_or_else(|_| panic!("{TEST_SERVER_URL_ENV_VAR} must be set by the SDK test harness"))
+}
+
 #[derive(Default)]
 struct TestCounterInner {
     /// Maps test names to their outcomes