For first time spells these are impressive dude... I love them! But some suggestion about them all. Don't make them specific for the units in the map.
For example, you used all Illidan's spells only for illidan. Change all the events to "An Unit Starts Casting an Ability" so that they work for any units. And instead of using a variable for Firewalk just use a "Custom Value for unit" on the unit and set the value of that unit to 1 (integer custom value). And then everywhere you referr to the casting unit (aka Illidan or the Crypt Lord) just use "Casting Unit". And now, for Firewalk I have better idea.
You can still use the 0.45 timer event tralala. As condition, you should use (Number of Units Matching (Level of Firewalk for (Matching Unit) greater than 0) greater than 0). And for action you can again use Pick Up Every Unit Matching (Level of Firewalk for (Matching Unit) greater than 0) and then the two other action: Create 1 fire unit and the Expiration timer. And so, you need only Firewalk Effect spell and nothing else.

Overall 5/5 dude. You did an awsome job. I am impressed. You are going to become great.
~Daelin