Skip to content

Commit b9a7b37

Browse files
committed
Remove unmaintained surf dependency
`surf` is unmaintained. Use `reqwest` in examples and tests. Closes #158.
1 parent 428142c commit b9a7b37

File tree

13 files changed

+2379
-145
lines changed

13 files changed

+2379
-145
lines changed

Cargo.lock

Lines changed: 2130 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ base64 = "0.22"
3737
url = "2.2"
3838

3939
[dev-dependencies]
40-
async-std = { version = "1.13.0", features = ["attributes"] }
41-
surf = "2.3.2"
42-
reqwest = "0.12.7"
40+
async-std = { version = "1.13.0", features = ["attributes", "tokio1"] }
41+
reqwest = { version = "0.12.7", features = ["json"] }
4342
tokio = { version = "1.5.0", features = ["macros", "rt-multi-thread"] }
4443
actix-rt = "2.2.0"
44+
serde = { version = "1", features = ["derive"] }

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,14 +86,14 @@ async fn main() {
8686
.await;
8787

8888
// If we probe the MockServer using any HTTP client it behaves as expected.
89-
let status = surf::get(format!("{}/hello", &mock_server.uri()))
89+
let status = reqwest::get(format!("{}/hello", &mock_server.uri()))
9090
.await
9191
.unwrap()
9292
.status();
9393
assert_eq!(status.as_u16(), 200);
9494

9595
// If the request doesn't match any `Mock` mounted on our `MockServer` a 404 is returned.
96-
let status = surf::get(format!("{}/missing", &mock_server.uri()))
96+
let status = reqwest::get(format!("{}/missing", &mock_server.uri()))
9797
.await
9898
.unwrap()
9999
.status();

src/lib.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,14 @@
3838
//! .await;
3939
//!
4040
//! // If we probe the MockServer using any HTTP client it behaves as expected.
41-
//! let status = surf::get(format!("{}/hello", &mock_server.uri()))
41+
//! let status = reqwest::get(format!("{}/hello", &mock_server.uri()))
4242
//! .await
4343
//! .unwrap()
4444
//! .status();
4545
//! assert_eq!(status, 200);
4646
//!
4747
//! // If the request doesn't match any `Mock` mounted on our `MockServer` a 404 is returned.
48-
//! let status = surf::get(format!("{}/missing", &mock_server.uri()))
48+
//! let status = reqwest::get(format!("{}/missing", &mock_server.uri()))
4949
//! .await
5050
//! .unwrap()
5151
//! .status();

src/matchers.rs

Lines changed: 52 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ where
5151
/// mock_server.register(mock).await;
5252
///
5353
/// // Act
54-
/// let status = surf::get(&mock_server.uri())
54+
/// let status = reqwest::get(&mock_server.uri())
5555
/// .await
5656
/// .unwrap()
5757
/// .status();
@@ -114,7 +114,7 @@ impl Match for MethodExactMatcher {
114114
/// mock_server.register(mock).await;
115115
///
116116
/// // Act
117-
/// let status = surf::get(&mock_server.uri())
117+
/// let status = reqwest::get(&mock_server.uri())
118118
/// .await
119119
/// .unwrap()
120120
/// .status();
@@ -155,7 +155,7 @@ impl Match for AnyMatcher {
155155
/// mock_server.register(mock).await;
156156
///
157157
/// // Act
158-
/// let status = surf::get(format!("{}/hello", &mock_server.uri()))
158+
/// let status = reqwest::get(format!("{}/hello", &mock_server.uri()))
159159
/// .await
160160
/// .unwrap()
161161
/// .status();
@@ -184,7 +184,7 @@ impl Match for AnyMatcher {
184184
/// mock_server.register(mock).await;
185185
///
186186
/// // Act
187-
/// let status = surf::get(format!("{}/hello?a_parameter=some_value", &mock_server.uri()))
187+
/// let status = reqwest::get(format!("{}/hello?a_parameter=some_value", &mock_server.uri()))
188188
/// .await
189189
/// .unwrap()
190190
/// .status();
@@ -251,7 +251,7 @@ impl Match for PathExactMatcher {
251251
/// mock_server.register(mock).await;
252252
///
253253
/// // Act
254-
/// let status = surf::get(format!("{}/hello/123", &mock_server.uri()))
254+
/// let status = reqwest::get(format!("{}/hello/123", &mock_server.uri()))
255255
/// .await
256256
/// .unwrap()
257257
/// .status();
@@ -277,7 +277,7 @@ impl Match for PathExactMatcher {
277277
/// mock_server.register(mock).await;
278278
///
279279
/// // Act
280-
/// let status = surf::get(format!("{}/users/da2854ea-b70f-46e7-babc-2846eff4d33c/posts", &mock_server.uri()))
280+
/// let status = reqwest::get(format!("{}/users/da2854ea-b70f-46e7-babc-2846eff4d33c/posts", &mock_server.uri()))
281281
/// .await
282282
/// .unwrap()
283283
/// .status();
@@ -330,9 +330,12 @@ impl Match for PathRegexMatcher {
330330
/// .await;
331331
///
332332
/// // Act
333-
/// let status = surf::get(&mock_server.uri())
333+
/// let client = reqwest::Client::new();
334+
/// let status = client
335+
/// .get(&mock_server.uri())
334336
/// .header("custom", "header")
335337
/// .header("cache-control", "no-cache, no-store")
338+
/// .send()
336339
/// .await
337340
/// .unwrap()
338341
/// .status();
@@ -424,8 +427,11 @@ impl Match for HeaderExactMatcher {
424427
/// .await;
425428
///
426429
/// // Act
427-
/// let status = surf::get(&mock_server.uri())
430+
/// let client = reqwest::Client::new();
431+
/// let status = client
432+
/// .get(&mock_server.uri())
428433
/// .header("custom", "header")
434+
/// .send()
429435
/// .await
430436
/// .unwrap()
431437
/// .status();
@@ -483,8 +489,10 @@ impl Match for HeaderExistsMatcher {
483489
/// .await;
484490
///
485491
/// // Act
486-
/// let status = surf::get(&mock_server.uri())
492+
/// let client = reqwest::Client::new();
493+
/// let status = client.get(&mock_server.uri())
487494
/// .header("custom", "headers are fun to match on with a regex")
495+
/// .send()
488496
/// .await
489497
/// .unwrap()
490498
/// .status();
@@ -551,8 +559,10 @@ impl Match for HeaderRegexMatcher {
551559
/// .await;
552560
///
553561
/// // Act
554-
/// let status = surf::post(&mock_server.uri())
562+
/// let client = reqwest::Client::new();
563+
/// let status = client.post(&mock_server.uri())
555564
/// .body("hello world!")
565+
/// .send()
556566
/// .await
557567
/// .unwrap()
558568
/// .status();
@@ -582,8 +592,10 @@ impl Match for HeaderRegexMatcher {
582592
/// .await;
583593
///
584594
/// // Act
585-
/// let status = surf::post(&mock_server.uri())
586-
/// .body(expected_body)
595+
/// let client = reqwest::Client::new();
596+
/// let status = client.post(&mock_server.uri())
597+
/// .json(&expected_body)
598+
/// .send()
587599
/// .await
588600
/// .unwrap()
589601
/// .status();
@@ -689,8 +701,10 @@ impl Match for BodyExactMatcher {
689701
/// .await;
690702
///
691703
/// // Act
692-
/// let status = surf::post(&mock_server.uri())
704+
/// let client = reqwest::Client::new();
705+
/// let status = client.post(&mock_server.uri())
693706
/// .body("this is a hello world example!")
707+
/// .send()
694708
/// .await
695709
/// .unwrap()
696710
/// .status();
@@ -768,8 +782,10 @@ impl Match for BodyContainsMatcher {
768782
/// "hello": "world!",
769783
/// "foo": "bar"
770784
/// });
771-
/// let status = surf::post(&mock_server.uri())
772-
/// .body(body)
785+
/// let client = reqwest::Client::new();
786+
/// let status = client.post(&mock_server.uri())
787+
/// .json(&body)
788+
/// .send()
773789
/// .await
774790
/// .unwrap()
775791
/// .status();
@@ -832,7 +848,7 @@ impl Match for BodyPartialJsonMatcher {
832848
/// .await;
833849
///
834850
/// // Act
835-
/// let status = surf::get(format!("{}?hello=world", &mock_server.uri()))
851+
/// let status = reqwest::get(format!("{}?hello=world", &mock_server.uri()))
836852
/// .await
837853
/// .unwrap()
838854
/// .status();
@@ -890,7 +906,7 @@ impl Match for QueryParamExactMatcher {
890906
/// .await;
891907
///
892908
/// // Act
893-
/// let status = surf::get(format!("{}?hello=some_world", &mock_server.uri()))
909+
/// let status = reqwest::get(format!("{}?hello=some_world", &mock_server.uri()))
894910
/// .await
895911
/// .unwrap()
896912
/// .status();
@@ -948,7 +964,7 @@ impl Match for QueryParamContainsMatcher {
948964
/// .await;
949965
///
950966
/// // Act
951-
/// let ok_status = surf::get(mock_server.uri().to_string())
967+
/// let ok_status = reqwest::get(mock_server.uri().to_string())
952968
/// .await
953969
/// .unwrap()
954970
/// .status();
@@ -957,7 +973,7 @@ impl Match for QueryParamContainsMatcher {
957973
/// assert_eq!(ok_status, 200);
958974
///
959975
/// // Act
960-
/// let err_status = surf::get(format!("{}?unexpected=foo", mock_server.uri()))
976+
/// let err_status = reqwest::get(format!("{}?unexpected=foo", mock_server.uri()))
961977
/// .await.
962978
/// unwrap().status();
963979
///
@@ -1020,9 +1036,11 @@ impl Match for QueryParamIsMissingMatcher {
10201036
/// json!({"hello": "world!"}),
10211037
/// json!({"hello": "everyone!"}),
10221038
/// ];
1023-
/// for case in success_cases.into_iter() {
1024-
/// let status = surf::post(&mock_server.uri())
1025-
/// .body(case)
1039+
/// let client = reqwest::Client::new();
1040+
/// for case in &success_cases {
1041+
/// let status = client.post(&mock_server.uri())
1042+
/// .json(case)
1043+
/// .send()
10261044
/// .await
10271045
/// .unwrap()
10281046
/// .status();
@@ -1035,8 +1053,9 @@ impl Match for QueryParamIsMissingMatcher {
10351053
/// // because it does not have the `hello` field.
10361054
/// // It won't match.
10371055
/// let failure_case = json!({"world": "hello!"});
1038-
/// let status = surf::post(&mock_server.uri())
1039-
/// .body(failure_case)
1056+
/// let status = client.post(&mock_server.uri())
1057+
/// .json(&failure_case)
1058+
/// .send()
10401059
/// .await
10411060
/// .unwrap()
10421061
/// .status();
@@ -1061,7 +1080,6 @@ where
10611080
/// use wiremock::{MockServer, Mock, ResponseTemplate};
10621081
/// use wiremock::matchers::basic_auth;
10631082
/// use serde::{Deserialize, Serialize};
1064-
/// use std::convert::TryInto;
10651083
///
10661084
/// #[async_std::main]
10671085
/// async fn main() {
@@ -1074,13 +1092,13 @@ where
10741092
/// .mount(&mock_server)
10751093
/// .await;
10761094
///
1077-
/// let client: surf::Client = surf::Config::new()
1078-
/// .set_base_url(surf::Url::parse(&mock_server.uri()).unwrap())
1079-
/// .add_header("Authorization", "Basic dXNlcm5hbWU6cGFzc3dvcmQ=").unwrap()
1080-
/// .try_into().unwrap();
1095+
/// let client = reqwest::Client::new();
10811096
///
10821097
/// // Act
1083-
/// let status = client.get("/")
1098+
/// let status = client
1099+
/// .get(&mock_server.uri())
1100+
/// .header("Authorization", "Basic dXNlcm5hbWU6cGFzc3dvcmQ=")
1101+
/// .send()
10841102
/// .await
10851103
/// .unwrap()
10861104
/// .status();
@@ -1145,9 +1163,12 @@ impl Match for BasicAuthMatcher {
11451163
/// .mount(&mock_server)
11461164
/// .await;
11471165
///
1166+
/// let client = reqwest::Client::new();
1167+
///
11481168
/// // Act
1149-
/// let status = surf::get(&mock_server.uri())
1169+
/// let status = client.get(&mock_server.uri())
11501170
/// .header("Authorization", "Bearer token")
1171+
/// .send()
11511172
/// .await
11521173
/// .unwrap()
11531174
/// .status();

0 commit comments

Comments
 (0)