Skip to content

Commit 1617a9f

Browse files
committed
Fix tests
1 parent ee0b7fc commit 1617a9f

File tree

5 files changed

+14
-11
lines changed

5 files changed

+14
-11
lines changed

src/bin/update-downloads.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ mod test {
158158
let tx = conn.transaction().unwrap();
159159
let user = user(&tx);
160160
let krate = Crate::find_or_insert(&tx, "foo", user.id,
161-
&None, &None, &None).unwrap();
161+
&None, &None, &None, &None).unwrap();
162162
let version = Version::insert(&tx, krate.id,
163163
&semver::Version::parse("1.0.0").unwrap(),
164164
&HashMap::new(), []).unwrap();
@@ -184,7 +184,7 @@ mod test {
184184
let tx = conn.transaction().unwrap();
185185
let user = user(&tx);
186186
let krate = Crate::find_or_insert(&tx, "foo", user.id, &None,
187-
&None, &None).unwrap();
187+
&None, &None, &None).unwrap();
188188
let version = Version::insert(&tx, krate.id,
189189
&semver::Version::parse("1.0.0").unwrap(),
190190
&HashMap::new(), []).unwrap();

src/krate.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,13 +94,13 @@ impl Crate {
9494
let stmt = try!(conn.prepare("UPDATE crates
9595
SET documentation = $1,
9696
homepage = $2,
97-
description = $3
97+
description = $3,
9898
readme = $4
9999
WHERE name = $5
100100
RETURNING *"));
101101
let mut rows = try!(stmt.query(&[&documentation, &homepage,
102-
&description, &name as &ToSql,
103-
&readme]));
102+
&description, &readme,
103+
&name as &ToSql]));
104104
match rows.next() {
105105
Some(row) => return Ok(Model::from_row(&row)),
106106
None => {}
@@ -111,7 +111,7 @@ impl Crate {
111111
description, homepage, documentation,
112112
readme)
113113
VALUES ($1, $2, $3, $3, 0, '0.0.0',
114-
$4, $5, $6, &7)
114+
$4, $5, $6, $7)
115115
RETURNING *"));
116116
let now = ::now();
117117
let mut rows = try!(stmt.query(&[&name as &ToSql, &user_id, &now,

src/tests/all.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,7 @@ fn krate(name: &str) -> Crate {
177177
documentation: None,
178178
homepage: None,
179179
description: None,
180+
readme: None,
180181
}
181182
}
182183

@@ -197,7 +198,8 @@ fn mock_crate(req: &mut Request, krate: Crate) -> Crate {
197198
let krate = Crate::find_or_insert(req.tx().unwrap(), krate.name.as_slice(),
198199
user.id, &krate.description,
199200
&krate.homepage,
200-
&krate.documentation).unwrap();
201+
&krate.documentation,
202+
&krate.readme).unwrap();
201203
Version::insert(req.tx().unwrap(), krate.id,
202204
&semver::Version::parse("1.0.0").unwrap(),
203205
&HashMap::new(), []).unwrap();

src/tests/krate.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,7 @@ fn new_req_full(api_token: &str, krate: Crate, version: &str,
159159
description: krate.description,
160160
homepage: krate.homepage,
161161
documentation: krate.documentation,
162+
readme: krate.readme,
162163
};
163164
let json = json::encode(&json);
164165
let mut body = MemWriter::new();

src/tests/middleware.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,11 @@ impl Middleware for MockDependency {
3535
let MockDependency(ref a, ref b) = *self;
3636
let user = req.extensions().find::<User>().unwrap();
3737
let crate_a = Crate::find_or_insert(req.tx().unwrap(),
38-
a.name.as_slice(),
39-
user.id, &None, &None, &None).unwrap();
38+
a.name.as_slice(), user.id, &None,
39+
&None, &None, &None).unwrap();
4040
let crate_b = Crate::find_or_insert(req.tx().unwrap(),
41-
b.name.as_slice(),
42-
user.id, &None, &None, &None).unwrap();
41+
b.name.as_slice(), user.id, &None,
42+
&None, &None, &None).unwrap();
4343
let va = Version::insert(req.tx().unwrap(), crate_a.id,
4444
&semver::Version::parse("1.0.0").unwrap(),
4545
&HashMap::new(), []).unwrap();

0 commit comments

Comments
 (0)