Fix more tests

Author: mtfishman

src/abstractblocksparsearray/arraylayouts.jl

@@ -64,8 +64,8 @@ function ArrayLayouts.sub_materialize(layout::BlockLayout{<:SparseLayout}, a, ax
     a_dest = BlockSparseArray{eltype(a), length(axes), blocktype_a}(undef, axes)
     for I in CartesianIndices(blocks(a_dest))
         b = Block(Tuple(I))
-        block_a = @view a[b]
-        if !iszero(block_a)
+        if isstored(a, b)
+            block_a = @view a[b]
             block_dest = similar(a_dest[b])
             copyto!(block_dest, block_a)
             a_dest[b] = block_dest

test/test_abstract_blocktype.jl

@@ -88,22 +88,12 @@ arrayts = (Array, JLArray)
     for f in (left_orth, left_polar, qr_compact, qr_full)
         u, c = f(a)
         @test u * c ≈ a
-        if arrayt ≡ Array
-            @test isisometric(u; side = :left)
-        else
-            # TODO: Fix comparison with UniformScaling on GPU.
-            @test isisometric(u; side = :left)
-        end
+        @test isisometric(u; side = :left)
     end
     for f in (right_orth, right_polar, lq_compact, lq_full)
         c, u = f(a)
         @test c * u ≈ a
-        if arrayt ≡ Array
-            @test isisometric(u; side = :right)
-        else
-            # TODO: Fix comparison with UniformScaling on GPU.
-            @test isisometric(u; side = :right)
-        end
+        @test isisometric(u; side = :right)
     end
     for f in (svd_compact, svd_full, svd_trunc)
         u, s, v = f(a)