Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4353 closed defect (fixed)

typo in test_Attack.js

Reported by: mimo Owned by: elexis
Priority: Should Have Milestone: Alpha 22
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

line 155 has "for (let allowCapturing of allowCapturing)"

Attachments (1)

0001-Fix-a-test-failure.patch​ (1.1 KB ) - added by leper 7 years ago.
Taken from the SM45 work.

Download all attachments as: .zip

Change History (4)

by leper, 7 years ago

Attachment: 0001-Fix-a-test-failure.patch​ added

Taken from the SM45 work.

comment:1 by leper, 7 years ago

Keywords: patch review added

This caused a test failure with SM45, no issue with committing this first.

comment:2 by elexis, 7 years ago

Owner: set to elexis
Resolution: → fixed
Status: new → closed

In 18959:

Fix a test failure on SM45. Patch by leper, fixes #4353.

comment:3 by elexis, 7 years ago

Keywords: review removed

Thanks for the patch leper! I see that SM45 finally complains about this, but how did mimo find out?

Note: See TracTickets for help on using tickets.