fix Informative error message at macro eval time for extra arguments #48 (#49)

* fix Informative error message at macro eval time for extra arguments #48

* try fix ci

* bump julia compat

Author: jw3126

.github/workflows/CI.yml

@@ -10,8 +10,7 @@ jobs:
       fail-fast: false
       matrix:
         version:
-          - '1.0'
-          - '1.1'
+          - '1.10' # lts
           - '1'
           - 'nightly'
         os:
@@ -27,15 +26,5 @@ jobs:
           version: ${{ matrix.version }}
           arch: ${{ matrix.arch }}
           show-versioninfo: true
-      - uses: actions/cache@v1
-        env:
-          cache-name: cache-artifacts
-        with:
-          path: ~/.julia/artifacts
-          key: ${{ runner.os }}-test-${{ env.cache-name }}-${{ hashFiles('**/Project.toml') }}
-          restore-keys: |
-            ${{ runner.os }}-test-${{ env.cache-name }}-
-            ${{ runner.os }}-test-
-            ${{ runner.os }}-
       - uses: julia-actions/julia-buildpkg@latest
       - uses: julia-actions/julia-runtest@latest

Project.toml

@@ -1,10 +1,10 @@
 name = "ArgCheck"
 uuid = "dce04be8-c92d-5529-be00-80e4d2c0e197"
 license = "MIT"
-version = "2.4.0"
+version = "2.4.1"
 
 [compat]
-julia = "1"
+julia = "1.10"
 
 [extras]
 BenchmarkTools = "6e4b80f9-dd63-53aa-95a3-0cdb28fa8baf"

src/checks.jl

@@ -80,7 +80,7 @@ end
 See also [`@check`](@ref).
 """
 macro argcheck(ex, options...)
-    check(ex, ArgCheckFlavor(), options...)
+    checkmacro(ex, ArgCheckFlavor(), options...)
 end
 
 """
@@ -97,10 +97,13 @@ Usage is as follows:
 See also [`@argcheck`](@ref).
 """
 macro check(ex, options...)
-    check(ex, CheckFlavor(), options...)
+    checkmacro(ex, CheckFlavor(), options...)
 end
 
-function check(ex, checkflavor, options...)
+function checkmacro(ex, checkflavor, options...)
+    if length(options) > 1
+        error("Too many arguments for @check/@argcheck macro")
+    end
     codeflavor = if isexpr(ex, :comparison)
         ComparisonFlavor()
     elseif isexpr(ex, :call)

test/checks.jl

@@ -308,4 +308,10 @@ end
     @test Meta.isexpr(ArgCheck.LABEL_END_CHECK, :meta)
 end
 
+@testset "Informative error message at macro eval time for extra arguments #48" begin
+    # https://github.com/jw3126/ArgCheck.jl/issues/48
+    @test_throws "Too many arguments" @macroexpand( @argcheck 1 == 1 1.8 "That's good")
+    @test_throws "Too many arguments" @macroexpand( @argcheck 1 == 2 1.8 "That's good")
+end
+
 end#module