Skip to content

Commit 223f16b

Browse files
committed
Remove unnecessary editable tests
After #653, `develop` tests already covered `editable` since it uses editable installs internally
1 parent 4a0eca1 commit 223f16b

File tree

3 files changed

+1
-128
lines changed

3 files changed

+1
-128
lines changed

tests/common/editable.rs

-89
This file was deleted.

tests/common/mod.rs

-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ use std::process::{Command, Stdio};
88
use std::{env, io, str};
99

1010
pub mod develop;
11-
pub mod editable;
1211
pub mod errors;
1312
pub mod integration;
1413
pub mod other;

tests/run.rs

+1-38
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
//! To speed up the tests, they are tests all collected in a single module
22
33
use common::{
4-
develop, editable, errors, get_python_implementation, handle_result, integration, other,
5-
test_python_path,
4+
develop, errors, get_python_implementation, handle_result, integration, other, test_python_path,
65
};
76
use indoc::indoc;
87
use maturin::Target;
@@ -124,42 +123,6 @@ fn develop_pyo3_ffi_pure() {
124123
));
125124
}
126125

127-
#[test]
128-
fn editable_pyo3_pure() {
129-
handle_result(editable::test_editable(
130-
"test-crates/pyo3-pure",
131-
None,
132-
"editable-pyo3-pure",
133-
));
134-
}
135-
136-
#[test]
137-
fn editable_pyo3_mixed() {
138-
handle_result(editable::test_editable(
139-
"test-crates/pyo3-mixed",
140-
None,
141-
"editable-pyo3-mixed",
142-
));
143-
}
144-
145-
#[test]
146-
fn editable_pyo3_mixed_py_subdir() {
147-
handle_result(editable::test_editable(
148-
"test-crates/pyo3-mixed-py-subdir",
149-
None,
150-
"editable-pyo3-mixed-py-subdir",
151-
));
152-
}
153-
154-
#[test]
155-
fn editable_pyo3_ffi_pure() {
156-
handle_result(editable::test_editable(
157-
"test-crates/pyo3-ffi-pure",
158-
None,
159-
"editable-pyo3-ffi-pure",
160-
));
161-
}
162-
163126
#[test]
164127
fn integration_pyo3_bin() {
165128
let python = test_python_path().map(PathBuf::from).unwrap_or_else(|| {

0 commit comments

Comments
 (0)