[3.3.5a] Right clicking/enabling auto-attack causes pet to attack

Description:

Warlock imps set to Defensive stance should not auto attack on player right click/enable auto attack.

I wanted to find a video to back me up on this, but unfortunately no dice. Unfortunately, I didn't have time to go over all Warlock minions last night, but this could be a generalizable case.

Current behaviour:

Currently, right-clicking on a mob - even one out of melee range - enables pet attack for Imps in Defensive stance.

Expected behaviour:

Imps should not attack until the targeted mob is hit by spellcast or wand autoattack.

Steps to reproduce the problem:

1. Create any Warlock and learn Summon Imp.
2. Summon the imp and right click a mob.
3. The imp will start attacking or casting Firebolt if Firebolt cast is enabled.

Branch(es):

3.3.5a

TC rev. hash/commit:

1af6df4

TDB version: 335.63 w/updates

Operating system: Ubuntu 16.04

Start by reproducing the issue on the 3.3.5a branch using the listed Warlock and Summon Imp steps, testing Defensive stance and Firebolt enabled or disabled. Trace the pet attack behavior triggered by right-clicking or enabling auto-attack; done means the Imp waits until the target is hit by the player's spellcast or wand autoattack.