Use integegers for timestamps of sub-directories in directory listings
This commit is contained in:
@@ -125,7 +125,7 @@ module RemoteStorage
|
|||||||
|
|
||||||
sub_directories(user, directory).each do |entry|
|
sub_directories(user, directory).each do |entry|
|
||||||
directory_name = CGI.unescape(entry["name"]).split("/").last
|
directory_name = CGI.unescape(entry["name"]).split("/").last
|
||||||
listing.merge!({ "#{directory_name}/" => entry["timestamp"] })
|
listing.merge!({ "#{directory_name}/" => entry["timestamp"].to_i })
|
||||||
end
|
end
|
||||||
|
|
||||||
directory_entries(user, directory).each do |entry|
|
directory_entries(user, directory).each do |entry|
|
||||||
|
|||||||
@@ -30,6 +30,7 @@ describe "Directories" do
|
|||||||
content = JSON.parse(last_response.body)
|
content = JSON.parse(last_response.body)
|
||||||
content.must_include "http://5apps.com"
|
content.must_include "http://5apps.com"
|
||||||
content.must_include "foo"
|
content.must_include "foo"
|
||||||
|
content["foo"].must_be_kind_of Integer
|
||||||
content["foo"].to_s.must_match /\d+/
|
content["foo"].to_s.must_match /\d+/
|
||||||
content["foo"].to_s.length.must_be :>=, 10
|
content["foo"].to_s.length.must_be :>=, 10
|
||||||
end
|
end
|
||||||
@@ -64,6 +65,7 @@ describe "Directories" do
|
|||||||
content.must_include "foo"
|
content.must_include "foo"
|
||||||
content.must_include "http://5apps.com"
|
content.must_include "http://5apps.com"
|
||||||
content.must_include "home/"
|
content.must_include "home/"
|
||||||
|
content["home/"].must_be_kind_of Integer
|
||||||
content["home/"].to_s.must_match /\d+/
|
content["home/"].to_s.must_match /\d+/
|
||||||
content["home/"].to_s.length.must_be :>=, 10
|
content["home/"].to_s.length.must_be :>=, 10
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user