Update test expectations

software-mansion · Jan 8, 2025 · 56a8ca0 · 56a8ca0
1 parent 3fb9467
commit 56a8ca0
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 36 deletions.
diff --git a/.github/workflows/_check-release.yml b/.github/workflows/_check-release.yml
@@ -41,10 +41,10 @@ jobs:
       - uses: dtolnay/rust-toolchain@stable
       - run: cargo test --profile=ci --no-fail-fast
 
-  snforge-init:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - uses: dtolnay/rust-toolchain@stable
-      - uses: foundry-rs/setup-snfoundry@v3
-      - run: cargo test --profile=ci --package scarb --test snforge_init new_simple -- --ignored
+#  snforge-init:
+#    runs-on: ubuntu-latest
+#    steps:
+#      - uses: actions/checkout@v4
+#      - uses: dtolnay/rust-toolchain@stable
+#      - uses: foundry-rs/setup-snfoundry@v3
+#      - run: cargo test --profile=ci --package scarb --test snforge_init new_simple -- --ignored
diff --git a/extensions/scarb-cairo-run/tests/examples.rs b/extensions/scarb-cairo-run/tests/examples.rs
@@ -97,7 +97,7 @@ fn can_limit_gas() {
                 Finished `dev` profile target(s) in [..]
                  Running hello_world
             Run completed successfully, returning [987]
-            Remaining gas: 59760
+            Remaining gas: 57270
         "#});
 }
 

diff --git a/extensions/scarb-cairo-test/tests/build.rs b/extensions/scarb-cairo-test/tests/build.rs
@@ -237,12 +237,12 @@ fn integration_tests() {
             [..]Finished `dev` profile target(s) in [..]
             testing hello ...
             running 2 tests
-            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 40740)
-            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 40740)
+            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 43130)
+            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 43130)
             test result: ok. 2 passed; 0 failed; 0 ignored; 0 filtered out;
 
             running 1 test
-            test hello::tests::it_works ... ok (gas usage est.: 40740)
+            test hello::tests::it_works ... ok (gas usage est.: 43130)
             test result: ok. 1 passed; 0 failed; 0 ignored; 0 filtered out;
 
         "#});
@@ -336,7 +336,7 @@ fn can_choose_test_kind_to_run() {
             [..]Finished `dev` profile target(s) in [..]
             testing hello ...
             running 1 test
-            test hello::tests::it_works ... ok (gas usage est.: 40740)
+            test hello::tests::it_works ... ok (gas usage est.: 43130)
             test result: ok. 1 passed; 0 failed; 0 ignored; 0 filtered out;
 
         "#});
@@ -352,8 +352,8 @@ fn can_choose_test_kind_to_run() {
             [..]Finished `dev` profile target(s) in [..]
             testing hello ...
             running 2 tests
-            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 40740)
-            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 40740)
+            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 43130)
+            test hello_integrationtest::[..]::tests::it_works ... ok (gas usage est.: 43130)
             test result: ok. 2 passed; 0 failed; 0 ignored; 0 filtered out;
             
         "#});

diff --git a/scarb/tests/build_starknet_contract_allowed_libfuncs.rs b/scarb/tests/build_starknet_contract_allowed_libfuncs.rs
@@ -26,6 +26,7 @@ const TESTING_LIST: &str = indoc! {r#"
 "#};
 
 #[test]
+#[ignore = "redeposit_gas is audited now"]
 fn default_behaviour() {
     let t = assert_fs::TempDir::new().unwrap();
     ProjectBuilder::start()
@@ -59,6 +60,7 @@ fn default_behaviour() {
 }
 
 #[test]
+#[ignore = "redeposit_gas is audited now"]
 fn check_true() {
     let t = assert_fs::TempDir::new().unwrap();
     ProjectBuilder::start()

diff --git a/scarb/tests/fmt.rs b/scarb/tests/fmt.rs
@@ -225,47 +225,42 @@ fn format_with_import_sorting() {
             Diff in file [..]lib.cairo:
              --- original
             +++ modified
-            @@ -1,21 +1,18 @@
-            -use openzeppelin::introspection::interface;
+            @@ -1,21 +1,21 @@
+            +use openzeppelin::introspection::first;
+             use openzeppelin::introspection::interface;
             -use openzeppelin::introspection::first;
-            +use openzeppelin::introspection::{first, interface};
-
+            
              #[starknet::contract]
              mod SRC5 {
-            -    use openzeppelin::introspection::interface;
+                 use openzeppelin::introspection::interface;
             -    use openzeppelin::introspection::{interface, AB};
             +    use openzeppelin::introspection::{AB, interface};
-
+            
                  #[storage]
                  struct Storage {
             -        supported_interfaces: LegacyMap<felt252, bool>
             +        supported_interfaces: LegacyMap<felt252, bool>,
                  }
-            -
+            
                  use openzeppelin::introspection::first;
-
+            
                  mod A {}
-            +    mod F;
-                 mod G;
-            -    mod F;
-
+            -    mod G;
+                 mod F;
+            +    mod G;
+            
                  #[abi(embed_v0)]
                  impl SRC5Impl of interface::ISRC5<ContractState> {
-            @@ -23,12 +20,10 @@
-                         true
-                     }
-                 }
-            -
-                 use A;
+            @@ -28,7 +28,7 @@
                  use starknet::ArrayTrait;
-
+            
                  mod Inner {
-            -        use C;
+            +        use B;
+                     use C;
             -        use B;
-            +        use {B, C};
                  }
              }
-
+            
         "});
 }