This macro works fine, you just need to have enough rage and actually cast cleave before it can continue to next step. You can easily become rage starved with heroic strike going off on every swing.

Edit: Also check for any typos you might have in your macro.