Skip to content

Commit e8cc706

Browse files
mschuchardmschuchard
committed
reorganize unit tests
1 parent 860f265 commit e8cc706

6 files changed

Lines changed: 514 additions & 524 deletions

File tree

‎src/concourse.rs‎

Lines changed: 1 addition & 172 deletions
Original file line numberDiff line numberDiff line change
@@ -182,175 +182,4 @@ impl OutMetadata {
182182
}
183183

184184
#[cfg(test)]
185-
mod tests {
186-
use super::*;
187-
188-
#[test]
189-
fn test_version_new() {
190-
assert_eq!(
191-
Version::new(octocrab::models::IssueState::Open),
192-
Version {
193-
state: octocrab::models::IssueState::Open
194-
},
195-
"version could not be constructed with the correct issue state",
196-
);
197-
}
198-
#[test]
199-
fn test_version_deserialize() {
200-
let version = serde_json::from_str::<Version>("{\"state\": \"closed\"}")
201-
.expect("version could not be deserialized");
202-
assert_eq!(
203-
version,
204-
Version {
205-
state: octocrab::models::IssueState::Closed
206-
},
207-
"version did not contain the expected member values",
208-
)
209-
}
210-
211-
#[test]
212-
fn test_source_owner() {
213-
assert_eq!(
214-
Source {
215-
pat: None,
216-
owner: String::from("myorg"),
217-
repo: String::from("myrepo"),
218-
state: Some(String::from("all")),
219-
number: None,
220-
milestone: None,
221-
assignee: None,
222-
labels: None,
223-
skip_check: None,
224-
trigger: None,
225-
}
226-
.owner,
227-
String::from("myorg"),
228-
"reader for source owner did not return expected member value"
229-
)
230-
}
231-
#[test]
232-
fn test_source_deserialize() {
233-
let json_input = r#"
234-
{
235-
"owner": "mitodl",
236-
"repo": "ol-infrastructure",
237-
"number": 1,
238-
"state": "open",
239-
"skip_check": false,
240-
"trigger": "open"
241-
}"#;
242-
let source =
243-
serde_json::from_str::<Source>(json_input).expect("source could not be deserialized");
244-
assert_eq!(
245-
source,
246-
Source {
247-
pat: None,
248-
owner: String::from("mitodl"),
249-
repo: String::from("ol-infrastructure"),
250-
number: Some(1),
251-
state: Some(String::from("open")),
252-
milestone: None,
253-
assignee: None,
254-
labels: None,
255-
skip_check: Some(false),
256-
trigger: Some(octocrab::models::IssueState::Open)
257-
},
258-
"source did not contain the expected member values",
259-
)
260-
}
261-
262-
#[test]
263-
fn test_outparams_title() {
264-
assert_eq!(
265-
OutParams {
266-
title: Some(String::from("mytitle")),
267-
body: None,
268-
labels: None,
269-
assignees: None,
270-
milestone: None,
271-
state: None,
272-
}
273-
.title,
274-
Some(String::from("mytitle")),
275-
"reader for outparams title did not return expected member value"
276-
)
277-
}
278-
#[test]
279-
fn test_outparams_deserialize() {
280-
let json_input = r#"
281-
{
282-
"title": "my_issue",
283-
"body": "approve the concourse step",
284-
"assignees": ["my_user_one", "my_user_two"],
285-
"milestone": 2,
286-
"state": "closed"
287-
}"#;
288-
let out_params = serde_json::from_str::<OutParams>(json_input)
289-
.expect("outparams could not be deserialized");
290-
assert_eq!(
291-
out_params,
292-
OutParams {
293-
title: Some(String::from("my_issue")),
294-
body: Some(String::from("approve the concourse step")),
295-
labels: None,
296-
assignees: Some(vec![
297-
String::from("my_user_one"),
298-
String::from("my_user_two")
299-
]),
300-
milestone: Some(2),
301-
state: Some(String::from("closed")),
302-
},
303-
"out params did not contain the expected member values",
304-
)
305-
}
306-
307-
#[test]
308-
fn test_outmetadata_new() {
309-
assert_eq!(
310-
OutMetadata::new(
311-
5,
312-
String::from("http://does.not.exist"),
313-
String::from("some issue"),
314-
octocrab::models::IssueState::Open,
315-
vec![],
316-
vec![],
317-
None,
318-
String::from("yesterday"),
319-
String::from("today"),
320-
),
321-
OutMetadata {
322-
number: 5,
323-
url: String::from("http://does.not.exist"),
324-
title: String::from("some issue"),
325-
state: octocrab::models::IssueState::Open,
326-
labels: vec![],
327-
assignees: vec![],
328-
milestone: None,
329-
created: String::from("yesterday"),
330-
updated: String::from("today"),
331-
},
332-
"outmetadata could not be constructed with the correct values"
333-
)
334-
}
335-
336-
#[test]
337-
fn test_outmetadata_serialize() {
338-
let out_metadata = OutMetadata::new(
339-
5,
340-
String::from("http://does.not.exist"),
341-
String::from("some issue"),
342-
octocrab::models::IssueState::Open,
343-
// cannot test next three since no constructors and non-exhaustive structs
344-
vec![],
345-
vec![],
346-
None,
347-
String::from("yesterday"),
348-
String::from("today"),
349-
);
350-
assert_eq!(
351-
serde_json::to_string(&out_metadata).expect("out metadata could not be serialized"),
352-
r#"{"number":5,"url":"http://does.not.exist","title":"some issue","state":"open","created":"yesterday","updated":"today"}"#,
353-
"out metadata did not contain the correct values"
354-
)
355-
}
356-
}
185+
mod tests;

‎src/concourse/tests.rs‎

Lines changed: 170 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,170 @@
1+
use super::*;
2+
3+
#[test]
4+
fn test_version_new() {
5+
assert_eq!(
6+
Version::new(octocrab::models::IssueState::Open),
7+
Version {
8+
state: octocrab::models::IssueState::Open
9+
},
10+
"version could not be constructed with the correct issue state",
11+
);
12+
}
13+
#[test]
14+
fn test_version_deserialize() {
15+
let version = serde_json::from_str::<Version>("{\"state\": \"closed\"}")
16+
.expect("version could not be deserialized");
17+
assert_eq!(
18+
version,
19+
Version {
20+
state: octocrab::models::IssueState::Closed
21+
},
22+
"version did not contain the expected member values",
23+
)
24+
}
25+
26+
#[test]
27+
fn test_source_owner() {
28+
assert_eq!(
29+
Source {
30+
pat: None,
31+
owner: String::from("myorg"),
32+
repo: String::from("myrepo"),
33+
state: Some(String::from("all")),
34+
number: None,
35+
milestone: None,
36+
assignee: None,
37+
labels: None,
38+
skip_check: None,
39+
trigger: None,
40+
}
41+
.owner,
42+
String::from("myorg"),
43+
"reader for source owner did not return expected member value"
44+
)
45+
}
46+
#[test]
47+
fn test_source_deserialize() {
48+
let json_input = r#"
49+
{
50+
"owner": "mitodl",
51+
"repo": "ol-infrastructure",
52+
"number": 1,
53+
"state": "open",
54+
"skip_check": false,
55+
"trigger": "open"
56+
}"#;
57+
let source =
58+
serde_json::from_str::<Source>(json_input).expect("source could not be deserialized");
59+
assert_eq!(
60+
source,
61+
Source {
62+
pat: None,
63+
owner: String::from("mitodl"),
64+
repo: String::from("ol-infrastructure"),
65+
number: Some(1),
66+
state: Some(String::from("open")),
67+
milestone: None,
68+
assignee: None,
69+
labels: None,
70+
skip_check: Some(false),
71+
trigger: Some(octocrab::models::IssueState::Open)
72+
},
73+
"source did not contain the expected member values",
74+
)
75+
}
76+
77+
#[test]
78+
fn test_outparams_title() {
79+
assert_eq!(
80+
OutParams {
81+
title: Some(String::from("mytitle")),
82+
body: None,
83+
labels: None,
84+
assignees: None,
85+
milestone: None,
86+
state: None,
87+
}
88+
.title,
89+
Some(String::from("mytitle")),
90+
"reader for outparams title did not return expected member value"
91+
)
92+
}
93+
#[test]
94+
fn test_outparams_deserialize() {
95+
let json_input = r#"
96+
{
97+
"title": "my_issue",
98+
"body": "approve the concourse step",
99+
"assignees": ["my_user_one", "my_user_two"],
100+
"milestone": 2,
101+
"state": "closed"
102+
}"#;
103+
let out_params =
104+
serde_json::from_str::<OutParams>(json_input).expect("outparams could not be deserialized");
105+
assert_eq!(
106+
out_params,
107+
OutParams {
108+
title: Some(String::from("my_issue")),
109+
body: Some(String::from("approve the concourse step")),
110+
labels: None,
111+
assignees: Some(vec![
112+
String::from("my_user_one"),
113+
String::from("my_user_two")
114+
]),
115+
milestone: Some(2),
116+
state: Some(String::from("closed")),
117+
},
118+
"out params did not contain the expected member values",
119+
)
120+
}
121+
122+
#[test]
123+
fn test_outmetadata_new() {
124+
assert_eq!(
125+
OutMetadata::new(
126+
5,
127+
String::from("http://does.not.exist"),
128+
String::from("some issue"),
129+
octocrab::models::IssueState::Open,
130+
vec![],
131+
vec![],
132+
None,
133+
String::from("yesterday"),
134+
String::from("today"),
135+
),
136+
OutMetadata {
137+
number: 5,
138+
url: String::from("http://does.not.exist"),
139+
title: String::from("some issue"),
140+
state: octocrab::models::IssueState::Open,
141+
labels: vec![],
142+
assignees: vec![],
143+
milestone: None,
144+
created: String::from("yesterday"),
145+
updated: String::from("today"),
146+
},
147+
"outmetadata could not be constructed with the correct values"
148+
)
149+
}
150+
151+
#[test]
152+
fn test_outmetadata_serialize() {
153+
let out_metadata = OutMetadata::new(
154+
5,
155+
String::from("http://does.not.exist"),
156+
String::from("some issue"),
157+
octocrab::models::IssueState::Open,
158+
// cannot test next three since no constructors and non-exhaustive structs
159+
vec![],
160+
vec![],
161+
None,
162+
String::from("yesterday"),
163+
String::from("today"),
164+
);
165+
assert_eq!(
166+
serde_json::to_string(&out_metadata).expect("out metadata could not be serialized"),
167+
r#"{"number":5,"url":"http://does.not.exist","title":"some issue","state":"open","created":"yesterday","updated":"today"}"#,
168+
"out metadata did not contain the correct values"
169+
)
170+
}

0 commit comments

Comments
 (0)