Fix tests on windows

tests/common.rs

@@ -22,3 +22,13 @@ pub fn check_installed(package: &Path, python: &PathBuf) -> Result<(), ()> {
 
     Ok(())
 }
+
+pub fn install_cffi(python: &PathBuf) {
+    let output = Command::new(&python)
+        .args(&["-m", "pip", "install", "cffi"])
+        .output()
+        .unwrap();
+    if !output.status.success() {
+        panic!(output.status);
+    }
+}

tests/test_develop.rs

@@ -1,4 +1,5 @@
 use crate::common::check_installed;
+use crate::common::install_cffi;
 use pyo3_pack::{develop, Target};
 use std::fs;
 use std::path::Path;
@@ -8,7 +9,7 @@ use std::process::Stdio;
 mod common;
 
 #[test]
-#[cfg(not(feature = "skip-pyo3-test"))]
+#[cfg(not(target_os = "windows"))] // TODO: Virtualenv support is broken on windows
 fn test_develop_get_fourtytwo() {
     test_develop(Path::new("get-fourtytwo"), None);
 }
@@ -23,7 +24,7 @@ fn test_develop_hello_world() {
     test_develop(Path::new("hello-world"), Some("bin".to_string()));
 }
 
-/// Creates a virtualenv and activates it, checks that the package isn't installed, uses
+/// Creates a virtual env and activates it, checks that the package isn't installed, uses
 /// "pyo3-pack develop" to install it and checks it is working
 fn test_develop(package: &Path, bindings: Option<String>) {
     let venv_dir = package.join("venv_develop");
@@ -48,13 +49,7 @@ fn test_develop(package: &Path, bindings: Option<String>) {
     // Ensure the test doesn't wrongly pass
     check_installed(&package, &python).unwrap_err();
 
-    let output = Command::new(&python)
-        .args(&["-m", "pip", "install", "cffi"])
-        .output()
-        .unwrap();
-    if !output.status.success() {
-        panic!(output.status);
-    }
+    install_cffi(&python);
 
     let manifest_file = package.join("Cargo.toml");
     develop(

tests/test_integration.rs

@@ -1,4 +1,5 @@
 use crate::common::check_installed;
+use crate::common::install_cffi;
 use pyo3_pack::{BuildOptions, Target};
 use std::path::Path;
 use std::process::{Command, Stdio};
@@ -64,6 +65,7 @@ fn test_integration(package: &Path, bindings: Option<String>) {
 
     let options = BuildOptions::from_iter_safe(cli).unwrap();
 
+    install_cffi(&target.get_python());
     let wheels = options
         .into_build_context(false, false)
         .unwrap()