bug: fix a flakey test

[GrantBirki]

resolves: #22

[Copilot]

Pull Request Overview

This PR addresses a flakey test issue by removing t.Parallel() from the function level in TestBranchMatchesCriteria and moving it to the subtests, along with making local copies of the test variables and preserving global state with a deferred restoration.

• Removed t.Parallel() from the function scope to avoid global variable races.
• Introduced a local copy of the test variable in the loop.
• Added logic to save and restore global variables during each subtest.

Comments suppressed due to low confidence (1)

internal/cmd/match_criteria_test.go:266

• [nitpick] Consider renaming the local variable (e.g., to 'testCase') for improved clarity and to better differentiate it from the loop variable.

test := test // Create a local copy of the test variable to use in the closure