Merge pull request #48 from sintefore/tf/eachindex

Add eachindex method for AbstractSparseArray and update tests

Author: trulsf

src/sparsearray.jl

@@ -63,6 +63,8 @@ function Base.show(io::IO, ::MIME"text/plain", sa::AbstractSparseArray)
 end
 Base.show(io::IO, sa::AbstractSparseArray) = show(io, _data(sa))
 
+Base.eachindex(sa::AbstractSparseArray) = keys(_data(sa))
+
 function Base.firstindex(sa::AbstractSparseArray, d)
     return minimum(x -> x[d], _data(sa).indices)
 end
@@ -89,7 +91,7 @@ end
 """
     SparseArray{T,N, K <: NTuple{N,Any} }
 Implementation of an AbstractSparseArray where data is stored
-in a dictionary. 
+in a dictionary.
 """
 struct SparseArray{T,N,K<:NTuple{N,Any}} <: AbstractSparseArray{T,N}
     data::Dictionary{K,T}

test/runtests.jl

@@ -102,6 +102,11 @@ end
     @test typeof(SparseArray{Int,3,NTuple{3,String}}()) ==
           SparseArray{Int,3,Tuple{String,String,String}}
     @test length(SparseArray{Int,3,Tuple{String,String,String}}()) == 0
+
+    # indexing
+    indices = eachindex(car_cost)
+    @test length(indices) == 5
+    @test car_cost[first(indices)] == 200
 end
 
 @testset "Repurposed from SparseVarArray" begin