From aaf1c393b2b78669856087276ef7382ae29ff1c5 Mon Sep 17 00:00:00 2001 From: Michael Scott Asato Cuthbert Date: Wed, 7 Oct 2026 23:40:16 -1000 Subject: [PATCH] testOctaveIsImplicit: mock pitchChanged instead of filling the Note cache AI-assisted (Claude) --- music21/test/test_pitch.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/music21/test/test_pitch.py b/music21/test/test_pitch.py index e10b169f0..81d1e7fcb 100644 --- a/music21/test/test_pitch.py +++ b/music21/test/test_pitch.py @@ -101,11 +101,11 @@ def testOctaveIsImplicit(self): # the setter informs a Note client only when something changed n = note.Note('C4') - n._cache['junk'] = 1 - n.pitch.octaveIsImplicit = False - self.assertEqual(n._cache, {'junk': 1}) - n.pitch.octaveIsImplicit = True - self.assertEqual(n._cache, {}) + with mock.patch.object(n, 'pitchChanged') as pitchChanged: + n.pitch.octaveIsImplicit = False + pitchChanged.assert_not_called() + n.pitch.octaveIsImplicit = True + pitchChanged.assert_called_once() def testNameSetting(self): with self.assertRaisesRegex(ValueError,