fix(test): disable parallel testing for code that have a race condition

This is expected due to the way the test is setup. Rather than make the
test be race-condition-safe, I disable it for this test. The race
condition stems from the testing setup, not from the underlying code.

internal/notifications/sync_test.go

@@ -77,10 +77,12 @@ func TestSync(t *testing.T) {
 			expected: Notifications{n0Updated, n1},
 		},
 	}
+
+	//nolint:paralleltest // Those needs to be run sequentially
+	// The test reuses variables, so it's easier to just not run in
+	// parallel.
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
-			t.Parallel()
-
 			got := Sync(test.local, test.remote)
 
 			if len(got) != len(test.expected) {