Add in-place repatching unit test and Heavyweight Electron pipeline validation

Author: Armaldio

.github/workflows/ci.yml

@@ -80,3 +80,28 @@ jobs:
         # Perform the definitive check
         cmp final_rust.exe final_cs.exe
         echo "SUCCESS: Bit-for-bit compatibility achieved!"
+
+    - name: Heavyweight Real-World Validation (Electron Browser Shell)
+      working-directory: verification_env
+      run: |
+        echo "--> Downloading standalone Electron x64 binary..."
+        curl -L https://github.com/electron/electron/releases/download/v28.2.0/electron-v28.2.0-win32-x64.zip -o electron_bin.zip
+        unzip electron_bin.zip electron.exe
+        
+        echo "--> Executing Heavyweight Binary Parity Test (electron.exe)"
+        
+        # Rust side
+        cp electron.exe target_electron.exe
+        ../target/release/gpupatch target_electron.exe rust_electron.exe
+        
+        # C# Side (Restoring target path so names match)
+        cp electron.exe target_electron.exe
+        dotnet exec ${{ env.TOOL_DLL }} target_electron.exe --enable
+        mv target_electron.exe cs_electron.exe
+        
+        echo "--> Electron Validation Finalization:"
+        sha256sum rust_electron.exe
+        sha256sum cs_electron.exe
+        
+        cmp rust_electron.exe cs_electron.exe
+        echo "EPIC SUCCESS: Deep compatibility verified on heavyweight Chromium shell!"

src/main.rs

@@ -553,4 +553,30 @@ mod tests {
 
         assert_eq!(p1, p2, "Deterministic patching failed");
     }
+
+    #[test]
+    fn test_inplace_repatching() {
+        let mock = create_mock_pe();
+        
+        // 1. First pass: appends the section
+        let patched_v1 = patch_pe(&mock, false, "app.exe").unwrap();
+        let pe_ptr = read_u32(&patched_v1, 0x3C) as usize;
+        let coff = pe_ptr + 4;
+        assert_eq!(read_u16(&patched_v1, coff + 2), 2, "Should have generated section");
+
+        // 2. Second pass: updates it to disabled. It must NOT add another section!
+        let patched_v2 = patch_pe(&patched_v1, true, "app.exe").unwrap();
+        let pe_ptr2 = read_u32(&patched_v2, 0x3C) as usize;
+        assert_eq!(read_u16(&patched_v2, pe_ptr2 + 6), 2, "Should have kept 2 sections (in-place edit)");
+
+        // 3. Verify it's actually disabled (value is 0)
+        // In our mock, target_val sits at the beginning of new section's raw data.
+        // The new section's raw data is appended at 1024.
+        assert_eq!(read_u32(&patched_v2, 1024), 0, "Symbol value should have been toggled to 0");
+        
+        // 4. Third pass: toggle back to enabled.
+        let patched_v3 = patch_pe(&patched_v2, false, "app.exe").unwrap();
+        assert_eq!(read_u32(&patched_v3, 1024), 1, "Symbol value should have been toggled back to 1");
+        assert_eq!(read_u16(&patched_v3, pe_ptr2 + 6), 2, "Section count remains 2");
+    }
 }