Skip to content

Commit 7ce9a0e

Browse files
committed
tests: Move NEXT_GH_ID to tests/util/github module
1 parent ee88277 commit 7ce9a0e

File tree

3 files changed

+12
-7
lines changed

3 files changed

+12
-7
lines changed

src/tests/mod.rs

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ use crate::{
77
EncodableOwner, EncodableVersion, GoodCrate,
88
},
99
};
10-
use std::sync::atomic::{AtomicUsize, Ordering};
1110

11+
use crate::tests::util::github::next_gh_id;
1212
use diesel::prelude::*;
1313

1414
mod account_lock;
@@ -86,11 +86,9 @@ pub struct OkBool {
8686
ok: bool,
8787
}
8888

89-
static NEXT_GH_ID: AtomicUsize = AtomicUsize::new(0);
90-
9189
fn new_user(login: &str) -> NewUser<'_> {
9290
NewUser {
93-
gh_id: NEXT_GH_ID.fetch_add(1, Ordering::SeqCst) as i32,
91+
gh_id: next_gh_id(),
9492
gh_login: login,
9593
name: None,
9694
gh_avatar: None,
@@ -100,8 +98,8 @@ fn new_user(login: &str) -> NewUser<'_> {
10098

10199
fn new_team(login: &str) -> NewTeam<'_> {
102100
NewTeam {
103-
org_id: NEXT_GH_ID.fetch_add(1, Ordering::SeqCst) as i32,
104-
github_id: NEXT_GH_ID.fetch_add(1, Ordering::SeqCst) as i32,
101+
org_id: next_gh_id(),
102+
github_id: next_gh_id(),
105103
login,
106104
name: None,
107105
avatar: None,

src/tests/util.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ use std::net::SocketAddr;
4141
use tower::ServiceExt;
4242

4343
mod chaosproxy;
44-
mod github;
44+
pub mod github;
4545
pub mod insta;
4646
pub mod matchers;
4747
mod mock_request;

src/tests/util/github.rs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,13 @@ use crates_io_github::{
55
GitHubTeam, GitHubTeamMembership, GithubUser,
66
};
77
use oauth2::AccessToken;
8+
use std::sync::atomic::{AtomicUsize, Ordering};
9+
10+
static NEXT_GH_ID: AtomicUsize = AtomicUsize::new(0);
11+
12+
pub fn next_gh_id() -> i32 {
13+
NEXT_GH_ID.fetch_add(1, Ordering::SeqCst) as i32
14+
}
815

916
pub(crate) const MOCK_GITHUB_DATA: MockData = MockData {
1017
orgs: &[MockOrg {

0 commit comments

Comments
 (0)