Skip to content

Commit a0853d3

Browse files
committed
refactor; move tgests to own file everywhere
1 parent 24bf773 commit a0853d3

File tree

11 files changed

+135
-155
lines changed

11 files changed

+135
-155
lines changed

macros_impl/src/zwang_router/path.rs

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -127,14 +127,4 @@ fn construct_typed_path(parts: &[Part], cur_idx: usize) -> proc_macro2::TokenStr
127127
}
128128

129129
#[cfg(test)]
130-
mod tests {
131-
use super::*;
132-
use crate::zwang_url;
133-
134-
#[test]
135-
fn basic_url_construction() {
136-
let expr: syn::LitStr = parse_quote! { "/owner_name=hi/repo_name=hello/issues/new" };
137-
let url = zwang_url(expr);
138-
println!("{}", url);
139-
}
140-
}
130+
mod tests;
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
use super::*;
2+
use crate::zwang_url;
3+
4+
#[test]
5+
fn basic_url_construction() {
6+
let expr: syn::LitStr = parse_quote! { "/owner_name=hi/repo_name=hello/issues/new" };
7+
let url = zwang_url(expr);
8+
println!("{}", url);
9+
}

macros_impl/src/zwang_router/routes/main_model.rs

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -266,16 +266,4 @@ impl TryFrom<parsing::Part> for Parts {
266266
}
267267

268268
#[cfg(test)]
269-
mod tests {
270-
use crate::zwang_router::TEST_STR;
271-
272-
use super::*;
273-
274-
#[test]
275-
fn test_converting_to_main_model_parts() -> Result<()> {
276-
let parsed: parsing::Part = parse_str(TEST_STR).expect("Unable to parse routes");
277-
let main_model_parts = Parts::try_from(parsed)?;
278-
println!("{:#?}", main_model_parts);
279-
Ok(())
280-
}
281-
}
269+
mod tests;
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
use crate::zwang_router::TEST_STR;
2+
3+
use super::*;
4+
5+
#[test]
6+
fn test_converting_to_main_model_parts() -> Result<()> {
7+
let parsed: parsing::Part = parse_str(TEST_STR).expect("Unable to parse routes");
8+
let main_model_parts = Parts::try_from(parsed)?;
9+
println!("{:#?}", main_model_parts);
10+
Ok(())
11+
}

macros_impl/src/zwang_router/routes/parsing.rs

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -153,15 +153,4 @@ impl Parse for Part {
153153
}
154154

155155
#[cfg(test)]
156-
mod tests {
157-
use crate::zwang_router::TEST_STR;
158-
159-
use super::*;
160-
use syn::parse_str;
161-
162-
#[test]
163-
fn test_parse_routes_with_fallback() {
164-
let parsed: Part = parse_str(TEST_STR).expect("Unable to parse routes");
165-
println!("{parsed:#?}");
166-
}
167-
}
156+
mod tests;
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
use crate::zwang_router::TEST_STR;
2+
3+
use super::*;
4+
use syn::parse_str;
5+
6+
#[test]
7+
fn test_parse_routes_with_fallback() {
8+
let parsed: Part = parse_str(TEST_STR).expect("Unable to parse routes");
9+
println!("{parsed:#?}");
10+
}

macros_impl/src/zwang_router/routes/write_output.rs

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -783,23 +783,4 @@ fn write_param_struct_construction_at_sub_part(sub_part: &main_model::Part) -> T
783783
}
784784

785785
#[cfg(test)]
786-
mod tests {
787-
use main_model::Parts;
788-
789-
use crate::zwang_router::{parsing, TEST_STR};
790-
791-
use super::*;
792-
793-
#[test]
794-
fn test_writing_output() -> Result<()> {
795-
let parsed: parsing::Part = parse_str(TEST_STR).expect("Unable to parse routes");
796-
let main_model_parts = Parts::try_from(parsed)?;
797-
let output = write_output(main_model_parts).unwrap().to_string();
798-
// println!("\n\n{output}\n\n");
799-
println!(
800-
"\n\n{}\n\n",
801-
prettyplease::unparse(&syn::parse_file(&output).unwrap())
802-
);
803-
Ok(())
804-
}
805-
}
786+
mod tests;
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
use main_model::Parts;
2+
3+
use crate::zwang_router::{parsing, TEST_STR};
4+
5+
use super::*;
6+
7+
#[test]
8+
fn test_writing_output() -> Result<()> {
9+
let parsed: parsing::Part = parse_str(TEST_STR).expect("Unable to parse routes");
10+
let main_model_parts = Parts::try_from(parsed)?;
11+
let output = write_output(main_model_parts).unwrap().to_string();
12+
// println!("\n\n{output}\n\n");
13+
println!(
14+
"\n\n{}\n\n",
15+
prettyplease::unparse(&syn::parse_file(&output).unwrap())
16+
);
17+
Ok(())
18+
}

shared/src/sync_engine/websocket_updates/transport.rs

Lines changed: 1 addition & 83 deletions
Original file line numberDiff line numberDiff line change
@@ -122,86 +122,4 @@ where
122122
}
123123

124124
#[cfg(test)]
125-
pub mod tests {
126-
use std::{
127-
pin::Pin,
128-
task::{Context, Poll},
129-
};
130-
131-
use futures::{channel::mpsc, Sink, Stream};
132-
133-
use crate::endpoints::defns::api::websocket_updates::{ClientMsg, ServerMsg};
134-
135-
use super::TransportTrait;
136-
137-
pub struct MockTransportHandler {
138-
send: mpsc::Sender<ServerMsg>,
139-
recv: mpsc::Receiver<ClientMsg>,
140-
}
141-
142-
pub struct MockTransport {
143-
recv: mpsc::Receiver<ServerMsg>,
144-
send: mpsc::Sender<ClientMsg>,
145-
}
146-
147-
impl TransportTrait for MockTransport {
148-
type TransportError = mpsc::SendError;
149-
}
150-
151-
impl MockTransport {
152-
pub fn new() -> (Self, MockTransportHandler) {
153-
let (server_msg_sender, server_msg_receiver) = mpsc::channel(100);
154-
let (client_msg_sender, client_msg_receiver) = mpsc::channel(100);
155-
156-
(
157-
Self {
158-
recv: server_msg_receiver,
159-
send: client_msg_sender,
160-
},
161-
MockTransportHandler {
162-
send: server_msg_sender,
163-
recv: client_msg_receiver,
164-
},
165-
)
166-
}
167-
}
168-
169-
impl Stream for MockTransport {
170-
type Item = Result<ServerMsg, mpsc::SendError>;
171-
172-
fn poll_next(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Option<Self::Item>> {
173-
Pin::new(&mut self.recv)
174-
.poll_next(cx)
175-
.map(|opt| opt.map(Ok))
176-
}
177-
}
178-
179-
impl Sink<ClientMsg> for MockTransport {
180-
type Error = mpsc::SendError;
181-
182-
fn poll_ready(
183-
mut self: Pin<&mut Self>,
184-
cx: &mut Context<'_>,
185-
) -> Poll<Result<(), Self::Error>> {
186-
Pin::new(&mut self.send).poll_ready(cx)
187-
}
188-
189-
fn start_send(mut self: Pin<&mut Self>, item: ClientMsg) -> Result<(), Self::Error> {
190-
Pin::new(&mut self.send).start_send(item)
191-
}
192-
193-
fn poll_flush(
194-
mut self: Pin<&mut Self>,
195-
cx: &mut Context<'_>,
196-
) -> Poll<Result<(), Self::Error>> {
197-
Pin::new(&mut self.send).poll_flush(cx)
198-
}
199-
200-
fn poll_close(
201-
mut self: Pin<&mut Self>,
202-
cx: &mut Context<'_>,
203-
) -> Poll<Result<(), Self::Error>> {
204-
Pin::new(&mut self.send).poll_close(cx)
205-
}
206-
}
207-
}
125+
pub mod tests;
Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
use std::{
2+
pin::Pin,
3+
task::{Context, Poll},
4+
};
5+
6+
use futures::{channel::mpsc, Sink, Stream};
7+
8+
use crate::endpoints::defns::api::websocket_updates::{ClientMsg, ServerMsg};
9+
10+
use super::TransportTrait;
11+
12+
pub struct MockTransportHandler {
13+
send: mpsc::Sender<ServerMsg>,
14+
recv: mpsc::Receiver<ClientMsg>,
15+
}
16+
17+
pub struct MockTransport {
18+
recv: mpsc::Receiver<ServerMsg>,
19+
send: mpsc::Sender<ClientMsg>,
20+
}
21+
22+
impl TransportTrait for MockTransport {
23+
type TransportError = mpsc::SendError;
24+
}
25+
26+
impl MockTransport {
27+
pub fn new() -> (Self, MockTransportHandler) {
28+
let (server_msg_sender, server_msg_receiver) = mpsc::channel(100);
29+
let (client_msg_sender, client_msg_receiver) = mpsc::channel(100);
30+
31+
(
32+
Self {
33+
recv: server_msg_receiver,
34+
send: client_msg_sender,
35+
},
36+
MockTransportHandler {
37+
send: server_msg_sender,
38+
recv: client_msg_receiver,
39+
},
40+
)
41+
}
42+
}
43+
44+
impl Stream for MockTransport {
45+
type Item = Result<ServerMsg, mpsc::SendError>;
46+
47+
fn poll_next(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Option<Self::Item>> {
48+
Pin::new(&mut self.recv)
49+
.poll_next(cx)
50+
.map(|opt| opt.map(Ok))
51+
}
52+
}
53+
54+
impl Sink<ClientMsg> for MockTransport {
55+
type Error = mpsc::SendError;
56+
57+
fn poll_ready(
58+
mut self: Pin<&mut Self>,
59+
cx: &mut Context<'_>,
60+
) -> Poll<Result<(), Self::Error>> {
61+
Pin::new(&mut self.send).poll_ready(cx)
62+
}
63+
64+
fn start_send(mut self: Pin<&mut Self>, item: ClientMsg) -> Result<(), Self::Error> {
65+
Pin::new(&mut self.send).start_send(item)
66+
}
67+
68+
fn poll_flush(
69+
mut self: Pin<&mut Self>,
70+
cx: &mut Context<'_>,
71+
) -> Poll<Result<(), Self::Error>> {
72+
Pin::new(&mut self.send).poll_flush(cx)
73+
}
74+
75+
fn poll_close(
76+
mut self: Pin<&mut Self>,
77+
cx: &mut Context<'_>,
78+
) -> Poll<Result<(), Self::Error>> {
79+
Pin::new(&mut self.send).poll_close(cx)
80+
}
81+
}

0 commit comments

Comments
 (0)