review comments fixes

.github/workflows/push.yml

@@ -36,7 +36,7 @@ jobs:
         run: make lint
 
   integration-tests:
-    runs-on: ubuntu-latest
+    runs-on: macos-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v3

contract/src/burn/api.rs

@@ -55,7 +55,7 @@ impl Contract {
 }
 
 #[cfg(not(test))]
-pub(crate) mod not_test {
+pub(crate) mod prod {
     use model::{TokensAmount, UnixTimestamp};
     use near_sdk::{
         env, ext_contract, is_promise_success, json_types::U128, near_bindgen, serde_json::json, Gas, Promise,

contract/src/claim/api.rs

@@ -123,7 +123,7 @@ impl Contract {
 }
 
 #[cfg(not(test))]
-mod not_test {
+mod prod {
     use model::{TokensAmount, UnixTimestamp};
     use near_sdk::{
         env, ext_contract, is_promise_success, json_types::U128, near_bindgen, serde_json::json, AccountId, Gas,

contract/src/record/api.rs

@@ -38,7 +38,7 @@ impl RecordApi for Contract {
 
         require!(
             existing.is_none(),
-            "Record for this timestamp: {now_seconds} already existed. It was owerwritten."
+            format!("Record for this timestamp: {now_seconds} already existed. It was owerwritten.")
         );
     }
 }

contract/src/record/tests.rs

@@ -33,3 +33,14 @@ fn record_by_oracle() {
     let bob_actual_balance = contract.get_claimable_balance_for_account(accounts.bob.clone());
     assert_eq!(bob_balance, bob_actual_balance.0);
 }
+
+
+#[test]
+#[should_panic(expected = "Record for this timestamp: 0 already existed. It was owerwritten.")]
+fn test_record() {
+    let (mut context, mut contract, accounts) = Context::init_with_oracle();
+
+    context.switch_account(&accounts.oracle);
+    contract.record_batch_for_hold(vec![(accounts.alice.clone(), 10.into())]);
+    contract.record_batch_for_hold(vec![(accounts.alice, 10.into())]);
+}

rust-toolchain.toml

@@ -1,2 +1,2 @@
 [toolchain]
-channel = "1.73"
+channel = "1.74"