/script TargetUnit("focus1");
/script UseAction(1);
/script OnClick_QuestListButton(1,1);
/script AcceptQuest();
/run ChoiceOption(1)
/script OnClick_QuestListButton(3,1);
/script CompleteQuest();
Make sure you put the Demo Battle NPC guy on focus.
First, very nice macro.
Second, very sad. The first two lines do accomplish one action: clicking the npc. The rest of it only does one action: clicking the appropriate dialog button. So, this macro does two actions with one input that under normal game play require two inputs.
Personally, I have a quest accept/turnin macro that only simulates clicking the accpet/complete button. I click the npc to open the dialog, then the macro. For npcs that have multiple quest, I still have to click the npc, then the appropriate quest, then run the macro.