Fix tests

Since we moved the adding of tracks and clips out of the constructors, we have to call the functions to add them ourselves within AutomationTrackTest

tests/src/tracks/AutomationTrackTest.cpp

@@ -94,23 +94,31 @@ private slots:
 		FloatModel model;
 
 		auto song = Engine::getSong();
+
 		AutomationTrack track(song);
+		song->addTrack(&track);
 
 		AutomationClip c1(&track);
+		track.addClip(&c1);
+
 		c1.setProgressionType(AutomationClip::ProgressionType::Linear);
 		c1.putValue(0, 0.0, false);
 		c1.putValue(10, 1.0, false);
 		c1.movePosition(0);
 		c1.addObject(&model);
 
 		AutomationClip c2(&track);
+		track.addClip(&c2);
+
 		c2.setProgressionType(AutomationClip::ProgressionType::Linear);
 		c2.putValue(0, 0.0, false);
 		c2.putValue(100, 1.0, false);
 		c2.movePosition(100);
 		c2.addObject(&model);
 
 		AutomationClip c3(&track);
+		track.addClip(&c3);
+
 		c3.addObject(&model);
 		//XXX: Why is this even necessary?
 		c3.clear();
@@ -130,9 +138,13 @@ private slots:
 		FloatModel model;
 
 		auto song = Engine::getSong();
+
 		AutomationTrack track(song);
+		song->addTrack(&track);
 
 		AutomationClip c(&track);
+		track.addClip(&c);
+
 		c.setProgressionType(AutomationClip::ProgressionType::Linear);
 		c.addObject(&model);
 
@@ -157,8 +169,11 @@ private slots:
 		auto song = Engine::getSong();
 
 		InstrumentTrack instrumentTrack(song);
+		song->addTrack(&instrumentTrack);
 
 		MidiClip midiClip(&instrumentTrack);
+		instrumentTrack.addClip(&midiClip);
+
 		midiClip.changeLength(TimePos(4, 0));
 		Note* note = midiClip.addNote(Note(TimePos(4, 0)), false);
 		note->createDetuning();
@@ -181,8 +196,13 @@ private slots:
 
 		auto song = Engine::getSong();
 		auto patternStore = Engine::patternStore();
+
 		PatternTrack patternTrack(song);
+		song->addTrack(&patternTrack);
+
 		AutomationTrack automationTrack(patternStore);
+		patternStore->addTrack(&automationTrack);
+
 		automationTrack.createClipsForPattern(patternStore->numOfPatterns() - 1);
 
 		QVERIFY(automationTrack.numOfClips());
@@ -202,11 +222,14 @@ private slots:
 		QCOMPARE(patternStore->automatedValuesAt(50, patternTrack.patternIndex())[&model], 1.0f);
 
 		PatternTrack patternTrack2(song);
+		song->addTrack(&patternTrack2);
 
 		QCOMPARE(patternStore->automatedValuesAt(5, patternTrack.patternIndex())[&model], 0.5f);
 		QVERIFY(! patternStore->automatedValuesAt(5, patternTrack2.patternIndex()).size());
 
 		PatternClip clip(&patternTrack);
+		patternTrack.addClip(&clip);
+
 		clip.changeLength(TimePos::ticksPerBar() * 2);
 		clip.movePosition(0);
 
@@ -221,13 +244,18 @@ private slots:
 
 		// Global automation should not have priority, see https://github.com/LMMS/lmms/issues/4268
 		// Tests regression caused by 75077f6200a5aee3a5821aae48a3b8466ed8714a
-		auto song = Engine::getSong();
 
+		auto song = Engine::getSong();
 		auto globalTrack = song->globalAutomationTrack();
+
 		AutomationClip globalClip(globalTrack);
+		globalTrack->addClip(&globalClip);
 
 		AutomationTrack localTrack(song);
+		song->addTrack(&localTrack);
+
 		AutomationClip localClip(&localTrack);
+		localTrack.addClip(&localClip);
 
 		FloatModel model;
 		globalClip.setProgressionType(AutomationClip::ProgressionType::Discrete);