fix development_mode_not_allowed test

MystenLabs · Sep 5, 2024 · 04ed464 · 04ed464
1 parent ae27f76
commit 04ed464
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/crates/sui-move-build/src/lib.rs b/crates/sui-move-build/src/lib.rs
@@ -647,7 +647,9 @@ impl PackageHooks for SuiPackageHooks {
         &self,
         manifest: &SourceManifest,
     ) -> anyhow::Result<PackageIdentifier> {
-        if !cfg!(debug_assertions) && manifest.package.edition == Some(Edition::DEVELOPMENT) {
+        if (!cfg!(debug_assertions) || cfg!(test))
+            && manifest.package.edition == Some(Edition::DEVELOPMENT)
+        {
             return Err(Edition::DEVELOPMENT.unknown_edition_error());
         }
         Ok(manifest.package.name)

diff --git a/crates/sui-move-build/src/unit_tests/build_tests.rs b/crates/sui-move-build/src/unit_tests/build_tests.rs
@@ -3,6 +3,8 @@
 
 use std::path::Path;
 
+use move_compiler::editions::Edition;
+
 use crate::BuildConfig;
 
 #[test]
@@ -37,5 +39,10 @@ fn development_mode_not_allowed() {
         .join("unit_tests")
         .join("data")
         .join("no_development_mode");
-    assert!(BuildConfig::new_for_testing().build(&path).is_err());
+    let err = BuildConfig::new_for_testing()
+        .build(&path)
+        .expect_err("Should have failed due to unsupported edition");
+    assert!(err
+        .to_string()
+        .contains(&Edition::DEVELOPMENT.unknown_edition_error().to_string()));
 }