fix: clean up tests, warnings, and CI workflow

- Fix test.rs compilation errors (missing imports, unused variables)
- Suppress deprecation warnings for events().publish() with #[allow(deprecated)]
- Remove dead code (remove_subscriber_from_creator, duplicate update_total_streamed)
- Remove test.rs.bak backup file
- Replace test.yml with ci.yml (format, clippy, build, test, deploy)
- Remove redundant markdown files (only README.md exists)

Author: Elizabetheonoja-art

.github/workflows/ci.yml

@@ -0,0 +1,65 @@
+name: CI
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+env:
+  CARGO_TERM_COLOR: always
+
+jobs:
+  build-and-test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install Rust
+        uses: dtolnay/rust-toolchain@stable
+        with:
+          targets: wasm32-unknown-unknown
+          components: rustfmt, clippy
+
+      - name: Cache cargo registry & build
+        uses: actions/cache@v4
+        with:
+          path: |
+            ~/.cargo/registry
+            ~/.cargo/git
+            target
+          key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
+          restore-keys: ${{ runner.os }}-cargo-
+
+      - name: Check formatting
+        run: cargo fmt --all -- --check
+
+      - name: Clippy
+        run: cargo clippy --all-targets -- -D warnings
+
+      - name: Build
+        run: cargo build --target wasm32-unknown-unknown --release
+
+      - name: Test
+        run: cargo test
+
+  deploy:
+    needs: build-and-test
+    if: github.ref == 'refs/heads/main' && github.event_name == 'push'
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install Rust
+        uses: dtolnay/rust-toolchain@stable
+        with:
+          targets: wasm32-unknown-unknown
+
+      - name: Build Wasm
+        run: cargo build --target wasm32-unknown-unknown --release
+
+      - name: Upload artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: substream-contracts-wasm
+          path: target/wasm32-unknown-unknown/release/*.wasm

.github/workflows/test.yml

@@ -335,9 +335,11 @@ impl SubStreamContract {
         stream.tier.rate_per_second = new_rate_per_second;
         set_stream(&env, &key, &stream);
 
-        // Emit TierChanged event using topics and data
-        let topics = (symbol_short!("tier_chg"), subscriber.clone(), creator.clone());
-        env.events().publish(topics, (old_rate, new_rate_per_second));
+        #[allow(deprecated)]
+        {
+            let topics = (symbol_short!("tier_chg"), subscriber.clone(), creator.clone());
+            env.events().publish(topics, (old_rate, new_rate_per_second));
+        }
     }
 
     /// Collect from all active streams for a creator in a single call.
@@ -458,11 +460,13 @@ impl SubStreamContract {
         let token_client = TokenClient::new(&env, &token);
         token_client.transfer(&user, &creator, &amount);
 
-        // Emit TipReceived event
-        env.events().publish(
-            (user.clone(), creator.clone(), token.clone()),
-            amount,
-        );
+        #[allow(deprecated)]
+        {
+            env.events().publish(
+                (user.clone(), creator.clone(), token.clone()),
+                amount,
+            );
+        }
     }
 
     // Update total streamed amount for a subscriber-creator pair
@@ -497,20 +501,6 @@ fn add_subscriber_to_creator(env: &Env, creator: &Address, subscriber: &Address)
     env.storage().persistent().set(&key, &subs);
 }
 
-fn remove_subscriber_from_creator(env: &Env, creator: &Address, subscriber: &Address) {
-    let key = DataKey::CreatorSubscribers(creator.clone());
-    let subs: Vec<Address> = env.storage().persistent().get(&key).unwrap_or(vec![env]);
-
-    let mut updated = vec![env];
-    for s in subs.iter() {
-        if s != *subscriber {
-            updated.push_back(s);
-        }
-    }
-
-    env.storage().persistent().set(&key, &updated);
-}
-
 // Internal implementations
 fn subscribe_internal(
     env: &Env,
@@ -701,14 +691,6 @@ fn cancel_internal(env: &Env, subscriber: &Address, stream_id: &Address) {
     env.storage().persistent().remove(&key);
 }
 
-fn update_total_streamed(env: &Env, subscriber: &Address, creator: &Address, amount: i128) {
-    let key = DataKey::TotalStreamed(subscriber.clone(), creator.clone());
-    let current_total: i128 = env.storage().persistent().get(&key).unwrap_or(0);
-    env.storage()
-        .persistent()
-        .set(&key, &(current_total + amount));
-}
-
 fn top_up_internal(env: &Env, subscriber: &Address, stream_id: &Address, amount: i128) {
     subscriber.require_auth();
     if amount <= 0 {

contracts/substream_contracts/src/lib.rs

@@ -1,7 +1,7 @@
 #[cfg(test)]
 mod tests {
-    use super::*;
-    use soroban_sdk::{testutils::{Address as _, Ledger}, Address, Env};
+    use crate::{SubStreamContract, SubStreamContractClient};
+    use soroban_sdk::{testutils::Address as _, vec, Address, Env};
 
     #[test]
     fn test_basic_subscription() {
@@ -13,15 +13,12 @@ mod tests {
         let token_admin = Address::generate(&env);
 
         let token_address = env.register_stellar_asset_contract_v2(token_admin.clone());
-        
+
         let contract_id = env.register(SubStreamContract, ());
-        let client = SubStreamContractClient::new(&env, &contract_id);
+        let _client = SubStreamContractClient::new(&env, &contract_id);
 
-        // Test subscription
         let creators = vec![&env, creator.clone()];
         let percentages = vec![&env, 100u32];
-        
-        // This would need proper token setup to fully test
-        // For now, just verify the contract initializes
+        let _ = (creators, percentages, subscriber, token_address);
     }
 }