Skip to content

Commit

Permalink
Ignore certain tests when run on GitHub CI.
Browse files Browse the repository at this point in the history
  • Loading branch information
vikpe committed Mar 2, 2025
1 parent 5fad85b commit 610f579
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 6 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: Test

on: [push, pull_request]
on: [ push, pull_request ]

env:
CARGO_TERM_COLOR: always
Expand All @@ -16,7 +16,7 @@ jobs:
- uses: Swatinem/rust-cache@v2

- name: Build
run: cargo build
run: cargo build --features ci

- name: Test
run: cargo test
run: cargo test --features ci
3 changes: 3 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,6 @@ tokio = { version = "1.37.0", features = ["macros", "net", "rt-multi-thread", "s

[dev-dependencies]
pretty_assertions = "1.4.0"

[features]
ci = []
8 changes: 5 additions & 3 deletions src/command.rs
Original file line number Diff line number Diff line change
Expand Up @@ -107,24 +107,26 @@ fn sorted_and_unique(server_addresses: &[ServerAddress]) -> Vec<ServerAddress> {
#[cfg(test)]
mod tests {
use super::*;
// use pretty_assertions::assert_eq;
use pretty_assertions::assert_eq;

#[tokio::test]
#[cfg_attr(feature = "ci", ignore)]
async fn test_server_addresses() -> Result<()> {
let master = "master.quakeservers.net:27000";
let timeout = Duration::from_secs(10);
let timeout = Duration::from_secs(2);
let result = server_addresses(master, timeout).await?;
assert!(!result.is_empty());
Ok(())
}

#[tokio::test]
#[cfg_attr(feature = "ci", ignore)]
async fn test_server_addresses_from_many() -> Result<()> {
let masters = [
"master.quakeservers.net:27000",
"master.quakeworld.nu:27000",
];
let timeout = Duration::from_secs(10);
let timeout = Duration::from_secs(2);
let result = server_addresses_from_many(&masters, timeout).await;
assert!(result.len() > 500);
Ok(())
Expand Down

0 comments on commit 610f579

Please sign in to comment.