Update README and add missing tests

README.md

@@ -1,7 +1,7 @@
 # Paperclip
 
 [![Build Status](https://api.travis-ci.org/wafflespeanut/paperclip.svg?branch=master)](https://travis-ci.org/wafflespeanut/paperclip)
-[![API docs](https://img.shields.io/badge/docs-latest-blue.svg)](https://paperclip.waffles.space/paperclip_openapi)
+[![API docs](https://img.shields.io/badge/docs-latest-blue.svg)](https://paperclip.waffles.space/paperclip)
 
 WIP OpenAPI code generator for type-safe compile-time checked HTTP APIs in Rust.
 
@@ -11,7 +11,7 @@ The following features are supported at the moment:
 
  - Generates API objects from schemas in an OpenAPI v2 spec.
  - Generates builder structs for the API objects and HTTP operations.
- - Fulfilled builder structs send API calls and return response futures (only `application/json` is supported as of now).
+ - Fulfilled builder structs send API calls and return response futures (only `https` and `application/json` is supported as of now).
 
 See the [projects](https://github.com/wafflespeanut/paperclip/projects) for tracking the features in queue.
 
@@ -195,4 +195,4 @@ at your option.
 
 I don't think proc macros are the right way to go for REST APIs. We need to be able to **see** the generated code somehow to identify names, fields, supported methods, etc. ([like this](https://paperclip.waffles.space/tests/test_k8s/api/)). With proc macros, you sorta have to guess.
 
-This doesn't mean you can't generate APIs in compile-time. The only difference is that you'll be using [build scripts](https://doc.rust-lang.org/cargo/reference/build-scripts.html) instead and `include!` the relevant code. That said, [we're using proc-macros](./macros) for other things.
+This doesn't mean you can't generate APIs in compile-time. The only difference is that you'll be using [build scripts](#build-script-example) instead and `include!` the relevant code. That said, [we're using proc-macros](./macros) for other things.

openapi/Cargo.toml

@@ -8,6 +8,7 @@ edition = "2018"
 failure = "0.1.5"
 failure_derive = "0.1.5"
 heck = { version = "0.3.1", optional = true }
+itertools = "0.8.0"
 lazy_static = "1.3.0"
 log = "0.4.6"
 paperclip-macros = { path = "../macros" }

openapi/src/v2/codegen/object.rs

@@ -753,7 +753,7 @@ where
             .try_for_each(|field| {
                 if let Some(ParameterIn::Query) = field.param_loc {
                     if !query.is_empty() {
-                        query.push_str(", ");
+                        query.push_str(",");
                     }
 
                     write!(query, "\n            (\"{}\", self.", &field.name)?;

openapi/src/v2/codegen/state.rs

@@ -1,6 +1,7 @@
 use super::object::ApiObject;
 use failure::Error;
 use heck::CamelCase;
+use itertools::Itertools;
 
 use std::cell::RefCell;
 use std::collections::{HashMap, HashSet};
@@ -53,7 +54,7 @@ impl EmitterState {
             mod_path.push("mod.rs");
 
             let mut contents = String::new();
-            for child in children {
+            for child in children.iter().sorted_by(|a, b| a.name.cmp(&b.name)) {
                 writeln!(
                     contents,
                     "