Opened 3 years ago

Closed 3 years ago

Last modified 3 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 3 years ago.
Taken from the SM45 work.

Download all attachments as: .zip

Change History (4)

Changed 3 years ago by leper

Taken from the SM45 work.

comment:1 Changed 3 years ago by leper

Keywords: patch review added

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

comment:2 Changed 3 years ago by elexis

Owner: set to elexis
Resolution: fixed
Status: newclosed

In 18959:

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

comment:3 Changed 3 years ago by elexis

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.