revert

Author: DonJayamanne

crates/pet/tests/ci_test.rs

@@ -344,43 +344,43 @@ fn compare_environments(env: PythonEnvironment, environment: PythonEnvironment,
     let mut env = env.clone();
     let mut environment = environment.clone();
 
-    // assert_eq!(
-    //     env.category,
-    //     environment.clone().category,
-    //     "Category mismatch when using {} for {:?} and {:?}",
-    //     method,
-    //     environment,
-    //     env
-    // );
-
-    if env.category != environment.clone().category {
-        error!(
-            "Category mismatch when using {} for {:?} and {:?}",
-            method, environment, env
-        );
-    }
+    assert_eq!(
+        env.category,
+        environment.clone().category,
+        "Category mismatch when using {} for {:?} and {:?}",
+        method,
+        environment,
+        env
+    );
+
+    // if env.category != environment.clone().category {
+    //     error!(
+    //         "Category mismatch when using {} for {:?} and {:?}",
+    //         method, environment, env
+    //     );
+    // }
 
     if let (Some(version), Some(expected_version)) =
         (environment.clone().version, env.clone().version)
     {
-        // assert!(
-        //     does_version_match(&version, &expected_version),
-        //     "Version mismatch when using {} for (expected {:?} to start with {:?}) for env = {:?} and environment = {:?}",
-        //     method,
-        //     expected_version,
-        //     version,
-        //     env.clone(),
-        //     environment.clone()
-        // );
-        if !does_version_match(&version, &expected_version) {
-            error!("Version mismatch when using {} for (expected {:?} to start with {:?}) for env = {:?} and environment = {:?}",
+        assert!(
+            does_version_match(&version, &expected_version),
+            "Version mismatch when using {} for (expected {:?} to start with {:?}) for env = {:?} and environment = {:?}",
             method,
             expected_version,
             version,
             env.clone(),
             environment.clone()
-            );
-        }
+        );
+        // if !does_version_match(&version, &expected_version) {
+        //     error!("Version mismatch when using {} for (expected {:?} to start with {:?}) for env = {:?} and environment = {:?}",
+        //     method,
+        //     expected_version,
+        //     version,
+        //     env.clone(),
+        //     environment.clone()
+        //     );
+        // }
     }
     // We have compared the versions, now ensure they are treated as the same
     // So that we can compare the objects easily
@@ -467,18 +467,18 @@ fn compare_environments(env: PythonEnvironment, environment: PythonEnvironment,
     }
     env.prefix = environment.clone().prefix;
 
-    // assert_eq!(
-    //     env, environment,
-    //     "Environment mismatch when using {} for {:?}",
-    //     method, environment
-    // );
+    assert_eq!(
+        env, environment,
+        "Environment mismatch when using {} for {:?}",
+        method, environment
+    );
 
-    if env != environment {
-        error!(
-            "Environment mismatch when using {} for {:?} and {:?}",
-            method, environment, env
-        );
-    }
+    // if env != environment {
+    //     error!(
+    //         "Environment mismatch when using {} for {:?} and {:?}",
+    //         method, environment, env
+    //     );
+    // }
 }
 
 fn verify_we_can_get_same_env_info_using_resolve_with_exe(