Add -only-skipped and -run-skipped test flags (#14)

Author: kyleconroy

CLAUDE.md

@@ -20,7 +20,22 @@ This file lists all skipped tests ordered by query file size (smallest first). S
 6. Add JSON marshaling functions in `parser/parser.go`
 7. Enable the test by setting `{"skip": false}` in its `metadata.json`
 8. Run `go test ./parser/...` to verify
-9. **Update `skipped_tests_by_size.txt`** after enabling tests
+9. **Check if other skipped tests now pass** (see below)
+10. **Update `skipped_tests_by_size.txt`** after enabling tests
+
+## Checking for Newly Passing Skipped Tests
+
+After implementing parser changes, run:
+
+```bash
+go test ./parser/... -only-skipped -v 2>&1 | grep "PASS:"
+```
+
+This shows any skipped tests that now pass. Enable those tests by setting `{"skip": false}` in their `metadata.json`.
+
+Available test flags:
+- `-only-skipped` - Run only skipped tests (find newly passing tests)
+- `-run-skipped` - Run skipped tests along with normal tests
 
 ## Updating skipped_tests_by_size.txt
 

parser/parser_test.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
+	"flag"
 	"os"
 	"path/filepath"
 	"testing"
@@ -13,6 +14,12 @@ type testMetadata struct {
 	Skip bool `json:"skip"`
 }
 
+// Test flags for running skipped tests
+// Usage: go test ./parser/... -run-skipped     # run all tests including skipped
+// Usage: go test ./parser/... -only-skipped    # run only skipped tests (find newly passing tests)
+var runSkippedTests = flag.Bool("run-skipped", false, "run skipped tests along with normal tests")
+var onlySkippedTests = flag.Bool("only-skipped", false, "run only skipped tests (useful to find tests that now pass)")
+
 func TestParse(t *testing.T) {
 	entries, err := os.ReadDir("testdata")
 	if err != nil {
@@ -40,9 +47,12 @@ func TestParse(t *testing.T) {
 				t.Fatalf("failed to parse metadata.json: %v", err)
 			}
 
-			if metadata.Skip {
+			if metadata.Skip && !*runSkippedTests && !*onlySkippedTests {
 				t.Skip("skipped via metadata.json")
 			}
+			if !metadata.Skip && *onlySkippedTests {
+				t.Skip("not a skipped test")
+			}
 
 			// Read the test SQL file
 			sqlPath := filepath.Join(testDir, "query.sql")

parser/testdata/AlterTableSwitchStatementTests/metadata.json

@@ -1 +1 @@
-{"skip": true}
+{"skip": false}

skipped_tests_by_size.txt

@@ -30,7 +30,6 @@
 203 Baselines160_CreateExternalFileFormatStatementTests160
 208 BaselinesCommon_AlterProcedureStatementTests
 208 RowsetsInSelectTests100
-210 AlterCreateCredentialStatementTests
 213 Baselines150_AlterDatabaseScopedConfigClearProcCacheTests150
 213 DeclareTableVariableTests150
 217 Baselines90_ExecuteAsStatementTests
@@ -44,15 +43,13 @@
 225 AlterServerConfigurationExternalAuthenticationTests150
 225 DeclareTableStatementTests
 226 Baselines90_RowsetsInSelectTests90
-231 Baselines90_AlterTableSwitchStatementTests
 231 Baselines90_ExecuteStatementTests90
 233 AlterResourceGovernorStatementTests
 233 Baselines100_AlterResourceGovernorStatementTests
 235 AlterFulltextCatalogStatementTests
 235 UseFederationTests110
 236 AlterTableStatementTests140_Azure
 236 Baselines140_AlterTableStatementTests140_Azure
-236 Baselines90_AlterCreateCredentialStatementTests
 239 Baselines100_EventNotificationStatementTests100
 239 Baselines120_AlterDatabaseOptionsTests120
 239 Baselines160_FuzzyStringMatchingTests160
@@ -161,7 +158,6 @@
 379 AlterCreateServiceStatementTests
 382 Baselines90_AlterCreateServiceStatementTests
 382 TableParametersTests
-383 AlterTableSwitchStatementTests
 384 BitManipulationFunctionTests160
 385 Baselines100_SymmetricKeyStatementTests100
 386 Baselines100_CTEStatementTests100