Avoid running some test 2 times when unneeded (#9124)

* avoid running some test 2 times when unneeded

* Update frame/support/test/Cargo.toml

substrate/.gitlab-ci.yml

@@ -319,7 +319,7 @@ test-linux-stable:                 &test-linux
   script:
     # this job runs all tests in former runtime-benchmarks, frame-staking and wasmtime tests
     - time cargo test --workspace --locked --release --verbose --features runtime-benchmarks --manifest-path bin/node/cli/Cargo.toml
-    - time cargo test -p frame-support-test --features=conditional-storage --manifest-path frame/support/test/Cargo.toml # does not reuse cache 1 min 44 sec
+    - time cargo test -p frame-support-test --features=conditional-storage --manifest-path frame/support/test/Cargo.toml --test pallet # does not reuse cache 1 min 44 sec
     - SUBSTRATE_TEST_TIMEOUT=1 time cargo test -p substrate-test-utils --release --verbose --locked -- --ignored timeout
     - sccache -s
 

substrate/frame/support/test/Cargo.toml

@@ -40,4 +40,6 @@ std = [
 	"sp-state-machine",
 ]
 try-runtime = ["frame-support/try-runtime"]
+# WARNING: CI only execute pallet test with this feature,
+# if the feature intended to be used outside, CI and this message need to be updated.
 conditional-storage = []

substrate/frame/support/test/tests/pallet_ui.rs

@@ -16,7 +16,6 @@
 // limitations under the License.
 
 #[rustversion::attr(not(stable), ignore)]
-#[cfg(not(feature = "conditional-storage"))]
 #[test]
 fn pallet_ui() {
 	// As trybuild is using `cargo check`, we don't need the real WASM binaries.