Created a test to make sure license can change between versions.

Author: baileyn

src/tests/all.rs

@@ -237,6 +237,11 @@ impl<'a> VersionBuilder<'a> {
         }
     }
 
+    fn license(mut self, license: Option<&'a str>) -> Self {
+        self.license = license;
+        self
+    }
+
     fn build(self, connection: &PgConnection, crate_id: i32) -> CargoResult<Version> {
         let license = match self.license {
             Some(license) => Some(license.to_owned()),

src/tests/version.rs

@@ -29,8 +29,10 @@ fn index() {
         let conn = app.diesel_database.get().unwrap();
         let u = ::new_user("foo").create_or_update(&conn).unwrap();
         ::CrateBuilder::new("foo_vers_index", u.id)
-            .version(::VersionBuilder::new("2.0.0"))
-            .version(::VersionBuilder::new("2.0.1"))
+            .version(::VersionBuilder::new("2.0.0")
+                        .license(Some("MIT")))
+            .version(::VersionBuilder::new("2.0.1")
+                        .license(Some("MIT/Apache-2.0")))
             .expect_build(&conn);
         let ids = versions::table
             .select(versions::id)
@@ -42,6 +44,14 @@ fn index() {
     let mut response = ok_resp!(middle.call(&mut req));
     let json: VersionList = ::json(&mut response);
     assert_eq!(json.versions.len(), 2);
+
+    for v in &json.versions {
+        match v.num.as_ref() {
+            "2.0.0" => assert_eq!(v.license, Some(String::from("MIT"))),
+            "2.0.1" => assert_eq!(v.license, Some(String::from("MIT/Apache-2.0"))),
+            _ => panic!("unexpected version")
+        }
+    }
 }
 
 #[test]