no wonder why the examples gha test was failing. cargo pgx test wasn'…

…t updated to the new PgConfig stuff

cargo-pgx/src/commands/test.rs

@@ -1,10 +1,12 @@
 // Copyright 2020 ZomboDB, LLC <[email protected]>. All rights reserved. Use of this source code is
 // governed by the MIT license that can be found in the LICENSE file.
 
+use pgx_utils::pg_config::PgConfig;
 use pgx_utils::{exit_with_error, get_target_dir, handle_result};
 use std::process::{Command, Stdio};
 
-pub fn test_extension(major_version: u16, is_release: bool) {
+pub fn test_extension(pg_config: &PgConfig, is_release: bool) -> Result<(), std::io::Error> {
+    let major_version = pg_config.major_version()?;
     let target_dir = get_target_dir();
 
     let mut command = Command::new("cargo");
@@ -32,4 +34,6 @@ pub fn test_extension(major_version: u16, is_release: bool) {
     if !status.success() {
         exit_with_error!("cargo pgx test failed with status = {:?}", status.code())
     }
+
+    Ok(())
 }

cargo-pgx/src/main.rs

@@ -171,10 +171,11 @@ fn do_it() -> std::result::Result<(), std::io::Error> {
                 connect_psql(Pgx::from_config()?.get(pgver)?, &dbname)
             }
             ("test", Some(test)) => {
-                let pgver = test.value_of("pg_version").unwrap_or("all");
                 let is_release = test.is_present("release");
-                for major_version in make_pg_major_version(pgver) {
-                    test_extension(*major_version, is_release);
+                let pgver = test.value_of("pg_version").unwrap_or("all");
+                let pgx = Pgx::from_config()?;
+                for pg_config in pgx.iter(PgConfigSelector::new(pgver)) {
+                    test_extension(pg_config?, is_release)?
                 }
                 Ok(())
             }
@@ -202,13 +203,3 @@ fn validate_extension_name(extname: &str) {
         }
     }
 }
-
-fn make_pg_major_version(version_string: &str) -> &'static [u16] {
-    match version_string {
-        "all" => &[10, 11, 12],
-        "pg10" => &[10],
-        "pg11" => &[11],
-        "pg12" => &[12],
-        _ => exit_with_error!("unrecognized Postgres version: {}", version_string),
-    }
-}