Skip to content

Commit

Permalink
fix: return correct not found for disconnected client and fix tests
Browse files Browse the repository at this point in the history
Tests weren't passing because autopush had a dependency change for FCM.
This updates the travis test to pull down the current dependencies for
autopush for integration tests.

Closes #89
  • Loading branch information
bbangert committed Jan 3, 2019
1 parent 8866986 commit b4e918a
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 78 deletions.
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,5 @@ autopush_rs/target
autopush_rs/_native*
target
*.rs.bk
requirements.txt
test-requirements.txt
2 changes: 2 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ dist: trusty

install:
- make ddb
- curl https://raw.githubusercontent.com/mozilla-services/autopush-rs/master/requirements.txt > requirements.txt
- curl https://raw.githubusercontent.com/mozilla-services/autopush-rs/master/test_requirements.txt > test_requirements.txt
- pip install --upgrade pip
- pip install -r requirements.txt
- pip install -r test-requirements.txt
Expand Down
62 changes: 0 additions & 62 deletions requirements.txt

This file was deleted.

2 changes: 1 addition & 1 deletion src/http.rs
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ impl Service for Push {
Ok(hyper::Response::new().with_status(StatusCode::Ok))
} else {
Ok(hyper::Response::new()
.with_status(StatusCode::BadGateway)
.with_status(StatusCode::NotFound)
.with_body("Client not available."))
}
} else {
Expand Down
15 changes: 0 additions & 15 deletions test-requirements.txt

This file was deleted.

0 comments on commit b4e918a

Please sign in to comment.