update tests

src/extractor.rs

@@ -312,8 +312,10 @@ impl<T: fmt::Display> fmt::Display for Form<T> {
 ///     let app = App::new().resource(
 ///         "/index.html",
 ///         |r| {
-///             r.method(http::Method::GET).with(index).limit(4096);
+///             r.method(http::Method::GET)
-///         }, // <- change form extractor configuration
+///                 // register form handler and change form extractor configuration
+///                 .with_config(index, |cfg| {cfg.limit(4096);})
+///         },
 ///     );
 /// }
 /// ```

src/json.rs

@@ -171,12 +171,13 @@ where
 /// fn main() {
 ///     let app = App::new().resource("/index.html", |r| {
 ///         r.method(http::Method::POST)
-///               .with(index)
+///               .with_config(index, |cfg| {
-///               .limit(4096)   // <- change json extractor configuration
+///                   cfg.limit(4096)   // <- change json extractor configuration
 ///                      .error_handler(|err, req| {  // <- create custom error response
 ///                          error::InternalError::from_response(
 ///                              err, HttpResponse::Conflict().finish()).into()
 ///                          });
+///               })
 ///     });
 /// }
 /// ```
@@ -326,7 +327,7 @@ mod tests {
     use http::header;
 
     use handler::Handler;
-    use with::{ExtractorConfig, With};
+    use with::With;
 
     impl PartialEq for JsonPayloadError {
         fn eq(&self, other: &JsonPayloadError) -> bool {
@@ -409,7 +410,7 @@ mod tests {
 
     #[test]
     fn test_with_json() {
-        let mut cfg = ExtractorConfig::<_, Json<MyObject>>::default();
+        let mut cfg = JsonConfig::default();
         cfg.limit(4096);
         let mut handler = With::new(|data: Json<MyObject>| data, cfg);
 

src/route.rs

@@ -257,7 +257,7 @@ impl<S: 'static> Route<S> {
     /// # extern crate actix_web;
     /// # extern crate futures;
     /// #[macro_use] extern crate serde_derive;
-    /// use actix_web::{http, App, Error, Path};
+    /// use actix_web::{http, App, Error, Form};
     /// use futures::Future;
     ///
     /// #[derive(Deserialize)]
@@ -318,7 +318,7 @@ impl<S: 'static> InnerHandler<S> {
 
     #[inline]
     pub fn handle(&self, req: HttpRequest<S>) -> AsyncResult<HttpResponse> {
-        // reason: handler is unique per thread, handler get called from async code only
+        // reason: handler is unique per thread, handler get called from sync code only
         let h = unsafe { &mut *self.0.as_ref().get() };
         h.handle(req)
     }

tests/test_handlers.rs

@@ -152,14 +152,17 @@ fn test_form_extractor() {
 fn test_form_extractor2() {
     let mut srv = test::TestServer::new(|app| {
         app.resource("/{username}/index.html", |r| {
-            r.route()
+            r.route().with_config(
-                .with(|form: Form<FormData>| format!("{}", form.username))
+                |form: Form<FormData>| format!("{}", form.username),
-                .error_handler(|err, _| {
+                |cfg| {
+                    cfg.error_handler(|err, _| {
                         error::InternalError::from_response(
                             err,
                             HttpResponse::Conflict().finish(),
                         ).into()
                     });
+                },
+            );
         });
     });