Compare commits

..

No commits in common. "cabb4daded8df2762c07c5d7fbef05a8572cf7be" and "ac3e9f14cd5824f10750fd5a705beed3ef387663" have entirely different histories.

2 changed files with 3 additions and 28 deletions

9
app.rb
View File

@ -50,14 +50,9 @@ class App < Roda
end end
url = r.params['url'] url = r.params['url']
if url.nil? if url.nil? or url.empty?
response.status = 400 response.status = 400
return {message: "missing url parameter"}.to_json return {message: "Please enter a valid URL"}
end
if url.empty?
response.status = 400
return {message: "invalid url parameter"}.to_json
end end
if links.filter(:url => url).first.nil? if links.filter(:url => url).first.nil?

View File

@ -25,7 +25,7 @@ describe "Submit API request to create new link" do
after :each do after :each do
@links.delete @links.delete
end end
it "should return link data in json format when a valid url is submitted" do it "should return link data in json format when a valid url is entered" do
data = { data = {
url: 'http://google.com' url: 'http://google.com'
} }
@ -36,25 +36,5 @@ describe "Submit API request to create new link" do
expect(response_json['code']).not_to eq(nil) expect(response_json['code']).not_to eq(nil)
expect(response_json['link']).to include(response_json['code']) expect(response_json['link']).to include(response_json['code'])
end end
it "should return with a 400 status and 'invalid url parameter' message when an empty url is submitted" do
data = {
url: ''
}
post('/links', data.to_json, "CONTENT_TYPE" => "application/json")
expect(last_response.status).to eq(400)
response_json = JSON.parse(last_response.body)
expect(response_json['message']).to eq('invalid url parameter')
end
it "should return with a 400 status and 'missing url parameter' message when an empty url is submitted" do
data = {
}
post('/links', data.to_json, "CONTENT_TYPE" => "application/json")
expect(last_response.status).to eq(400)
response_json = JSON.parse(last_response.body)
expect(response_json['message']).to eq('missing url parameter')
end
end end