(minor) adaptations to tests to accomodate indexing

Author: lkdvos

README.md

@@ -89,7 +89,7 @@ using Dictionaries: IndexError
 @test isstored(a, 1, 2)
 @test setstoredindex!(copy(a), 21, 1, 2) == [0 21; 0 0]
 @test_throws IndexError setstoredindex!(copy(a), 21, 2, 1)
-@test setunstoredindex!(copy(a), 21, 1, 2) == [0 21; 0 0]
+@test_throws IndexError setunstoredindex!(copy(a), 21, 1, 2) == [0 21; 0 0]
 @test storedlength(a) == 1
 @test issetequal(storedpairs(a), [CartesianIndex(1, 2) => 12])
 @test issetequal(storedvalues(a), [12])

examples/README.jl

@@ -86,7 +86,7 @@ using Dictionaries: IndexError
 @test isstored(a, 1, 2)
 @test setstoredindex!(copy(a), 21, 1, 2) == [0 21; 0 0]
 @test_throws IndexError setstoredindex!(copy(a), 21, 2, 1)
-@test setunstoredindex!(copy(a), 21, 1, 2) == [0 21; 0 0]
+@test_throws IndexError setunstoredindex!(copy(a), 21, 1, 2) == [0 21; 0 0]
 @test storedlength(a) == 1
 @test issetequal(storedpairs(a), [CartesianIndex(1, 2) => 12])
 @test issetequal(storedvalues(a), [12])

test/test_basics.jl

@@ -36,7 +36,7 @@ arrayts = (Array, JLArray)
   # probably we can have a trait for that. It could be based
   # on the `ArrayLayouts.MemoryLayout`.
   @allowscalar @test storedvalues(a) == a
-  @allowscalar @test storedpairs(a) == collect(pairs(vec(a)))
+  @allowscalar @test storedpairs(a) == pairs(a)
   @allowscalar for I in eachindex(a)
     @test getstoredindex(a, I) == a[I]
     @test iszero(getunstoredindex(a, I))

test/test_oneelementarray.jl

@@ -35,8 +35,8 @@ elts = (Float32, Float64, Complex{Float32}, Complex{Float64})
     @test a[2, 2] === zero(Bool)
     @test storedlength(a) == 1
     @test collect(eachstoredindex(a)) == [CartesianIndex(1, 2)]
-    @test storedpairs(a) == [CartesianIndex(1, 2) => 1]
-    @test storedvalues(a) == [1]
+    @test collect(storedpairs(a)) == [CartesianIndex(1, 2) => 1]
+    @test collect(storedvalues(a)) == [1]
   end
 
   for a in (OneElementArray(1, 2), OneElementVector(1, 2))
@@ -47,8 +47,8 @@ elts = (Float32, Float64, Complex{Float32}, Complex{Float64})
     @test a[2] === zero(Bool)
     @test storedlength(a) == 1
     @test collect(eachstoredindex(a)) == [CartesianIndex(1)]
-    @test storedpairs(a) == [CartesianIndex(1) => 1]
-    @test storedvalues(a) == [1]
+    @test collect(storedpairs(a)) == [CartesianIndex(1) => 1]
+    @test collect(storedvalues(a)) == [1]
   end
 
   a = OneElementArray()