fix #110 (#111)

* fix 0 dim show

* add tests

* Update ci.yml

* Update ci.yml

* coverage

---------

Co-authored-by: Sheehan Olver <[email protected]>

Author: putianyi889

.github/workflows/ci.yml

@@ -10,22 +10,21 @@ jobs:
       fail-fast: false
       matrix:
         version:
-          - '1.6'
+          - 'lts'
           - '1'
-          - '^1.11.0-0'          
         os:
           - ubuntu-latest
           - macOS-latest
           - windows-latest
         arch:
           - x64
     steps:
-      - uses: actions/checkout@v2
-      - uses: julia-actions/setup-julia@v1
+      - uses: actions/checkout@v4
+      - uses: julia-actions/setup-julia@v2
         with:
           version: ${{ matrix.version }}
           arch: ${{ matrix.arch }}
-      - uses: actions/cache@v1
+      - uses: actions/cache@v4
         env:
           cache-name: cache-artifacts
         with:
@@ -38,6 +37,7 @@ jobs:
       - uses: julia-actions/julia-buildpkg@v1
       - uses: julia-actions/julia-runtest@v1
       - uses: julia-actions/julia-processcoverage@v1
-      - uses: codecov/codecov-action@v1
+      - uses: codecov/codecov-action@v4
         with:
+          token: ${{ secrets.CODECOV_TOKEN }}
           file: lcov.info

src/quasifill.jl

@@ -198,28 +198,31 @@ show(io::IO, F::AbstractQuasiFill) = summary(io, F)
 
 function summary(io::IO, F::QuasiOnes)
     print(io, "ones(")
-    summary(io, F.axes[1])
-    for a in tail(F.axes)
-        print(io, ", ")
-        summary(io, a)
+    if !isempty(F.axes)
+        summary(io, F.axes[1])
+        for a in tail(F.axes)
+            print(io, ", ")
+            summary(io, a)
+        end
     end
     print(io, ")")
 end
 
 function summary(io::IO, F::QuasiZeros)
     print(io, "zeros(")
-    summary(io, F.axes[1])
-    for a in tail(F.axes)
-        print(io, ", ")
-        summary(io, a)
+    if !isempty(F.axes)
+        summary(io, F.axes[1])
+        for a in tail(F.axes)
+            print(io, ", ")
+            summary(io, a)
+        end
     end
     print(io, ")")
 end
 
 function summary(io::IO, F::QuasiFill)
-    print(io, "fill($(F.value), ")
-    summary(io, F.axes[1])
-    for a in tail(F.axes)
+    print(io, "fill($(F.value)")
+    for a in F.axes
         print(io, ", ")
         summary(io, a)
     end

test/test_quasifill.jl

@@ -706,9 +706,17 @@ import QuasiArrays: AbstractQuasiFill
     end
 
     @testset "show" begin
+        @test stringmime("text/plain",ones(Inclusion([1,2,3]), Inclusion([1,2,4]))) == "ones(Inclusion([1, 2, 3]), Inclusion([1, 2, 4]))"
+        @test stringmime("text/plain",zeros(Inclusion([1,2,3]), Inclusion([1,2,4]))) == "zeros(Inclusion([1, 2, 3]), Inclusion([1, 2, 4]))"
+        @test stringmime("text/plain",fill(2,Inclusion([1,2,3]), Inclusion([1,2,4]))) == "fill(2, Inclusion([1, 2, 3]), Inclusion([1, 2, 4]))"
+
         @test stringmime("text/plain",ones(Inclusion([1,2,3]))) == "ones(Inclusion([1, 2, 3]))"
         @test stringmime("text/plain",zeros(Inclusion([1,2,3]))) == "zeros(Inclusion([1, 2, 3]))"
         @test stringmime("text/plain",fill(2,Inclusion([1,2,3]))) == "fill(2, Inclusion([1, 2, 3]))"
+
+        @test stringmime("text/plain",QuasiOnes()) == "ones()"
+        @test stringmime("text/plain",QuasiZeros()) == "zeros()"
+        @test stringmime("text/plain",QuasiFill(2)) == "fill(2)"
     end
 
     @testset "Mul" begin