Skip comparison tests with no consensus

Simplify `compare/compare_test.go` by skipping tests with no consensus,
which were the source of the previous unknowns in the test. Based on
advice from cburgmer/json-path-comparison#153#issuecomment-3374075044.

While at it, upgrade `golangci-lint` to v2.5.0 and tweak its
configuration to scan `compare/compare_test.go` and fix the long lines
it complains about. Also have the `test-compare` target run the tests
verbosely, and run the CI tests on Go 1.25.

Author: theory

.github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: Test and Lint
+name: ✅ Test and Lint
 on:
   push:
     branches-ignore: [wip/**]
@@ -7,7 +7,7 @@ jobs:
     strategy:
       matrix:
         os: [[🐧, Ubuntu], [🍎, macOS], [🪟, Windows]]
-        go: ["1.24", "1.23"]
+        go: ["1.25", "1.24", "1.23"]
     name: ${{ matrix.os[0] }} Test Go ${{ matrix.go }} on ${{ matrix.os[1] }}
     runs-on: ${{ matrix.os[1] }}-latest
     steps:

.golangci.yaml

@@ -1,4 +1,7 @@
 version: "2"
+run:
+  build-tags:
+    - compare
 linters:
   default: all
   disable:

CHANGELOG.md

@@ -7,6 +7,13 @@ All notable changes to this project will be documented in this file. It uses the
   [Semantic Versioning]: https://semver.org/spec/v2.0.0.html
     "Semantic Versioning 2.0.0"
 
+## [v0.10.2] — Unreleased
+
+### 📔 Notes
+
+*   Upgraded to `golangci-lint` v2.5.0.
+*   Removed unused constant from `parser/lex.go`
+
 ## [v0.10.1] — 2025-09-16
 
 ### 🐞 Bug Fixes

Makefile

@@ -13,7 +13,7 @@ compare/regression_suite.yaml:
 	curl -so "$@" https://raw.githubusercontent.com/cburgmer/json-path-comparison/refs/heads/master/regression_suite/regression_suite.yaml
 
 test-compare: compare/regression_suite.yaml
-	GOTOOLCHAIN=local ${GO} test -tags=compare ./compare -count=1
+	GOTOOLCHAIN=local ${GO} test -v -tags=compare ./compare -count=1
 
 test-all: compare/regression_suite.yaml
 	GOTOOLCHAIN=local ${GO} test -tags=compare ./... -count=1
@@ -47,7 +47,7 @@ brew-lint-depends:
 
 .PHONY: debian-lint-depends # Install linting tools on Debian
 debian-lint-depends:
-	curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sudo sh -s -- -b /usr/bin v2.4.0
+	curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sudo sh -s -- -b /usr/bin v2.5.0
 
 .PHONY: install-generators # Install Go code generators
 install-generators:

compare/compare_test.go

@@ -55,42 +55,38 @@ func TestConsensus(t *testing.T) {
 	t.Parallel()
 
 	skip := map[string]string{
-		"array_slice_with_step_and_leading_zeros": "leading zeros disallowed integers; see RFC 9535 sections 2.3.3.1, 2.3.4.1",
-		"dot_notation_with_number_on_object":      "leading digits disallowed in shorthand names; see RFC 9535 section 2.5.1.1",
-		"dot_notation_with_dash":                  "dash disallowed in shorthand hames; see RFC 9535 section 2.5.1.1",
+		"array_slice_with_step_and_leading_zeros": "RFC 9535 § 2.3.3.1, 2.3.4.1: leading zeros disallowed in integers",
+		"dot_notation_with_number_on_object":      "RFC 9535 § 2.5.1.1: leading digits disallowed in shorthand names",
+		"dot_notation_with_dash":                  "RFC 9535 § 2.5.1.1: dash disallowed in shorthand hames",
 	}
 
 	for _, q := range queries(t) {
 		t.Run(q.ID, func(t *testing.T) {
 			t.Parallel()
 
+			// Skip tests with no consensus.
+			// https://github.com/cburgmer/json-path-comparison/pull/153#issuecomment-3374075044
+			if q.Consensus == nil {
+				t.Skip("No consensus")
+			}
+
+			// Skip tests where the consensus NOT_SUPPORTED.
 			if q.Consensus == "NOT_SUPPORTED" {
 				t.Skip(q.Consensus)
 			}
 
+			// Skip tests that violate RFC 9535.
 			if r, ok := skip[q.ID]; ok {
 				t.Skip(r)
 			}
 
 			path, err := jsonpath.Parse(q.Selector)
-			// XXX Why is consensus empty?
-			if q.Consensus != nil {
-				require.NoError(t, err)
-			}
-			if err != nil {
-				// XXX Why is there an error? TODOs?
-				assert.Nil(t, path)
-				return
-			}
+			require.NoError(t, err)
 			result := []any(path.Select(q.Document))
 
-			switch {
-			case q.Ordered:
+			if q.Ordered {
 				assert.Equal(t, q.Consensus, result)
-			case q.Consensus == nil:
-				// XXX What to do here?
-				// assert.Empty(t, result)
-			default:
+			} else {
 				assert.ElementsMatch(t, q.Consensus, result)
 			}
 		})

parser/lex.go

@@ -48,8 +48,7 @@ const (
 	hex     = 16
 
 	// Token literals.
-	nullByte  = 0x00
-	backslash = '\\'
+	nullByte = 0x00
 
 	// blanks selects blank space characters.
 	blanks = uint64(1<<'\t' | 1<<'\n' | 1<<'\r' | 1<<' ')