Download & Extend

assertField() is broken

Project:SimpleTest
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

After the rash of commits tonight, assertField() fails because:

function assertField($id, $message = '') {
$this->assertFieldByXPath($this->_constructFieldXpath('name', $name) .'|'. $this->_constructFieldXpath('id', $id), '', $message);
}

$name isn't passed into the function.

Patch should be credited to chx, who told me how to fix it.

Comments

#1

Status:active» needs review
AttachmentSizeStatusTest resultOperations
simpletest-fix-assertfield-237407-1.patch1.07 KBIgnored: Check issue status.NoneNone

#2

Status:needs review» fixed

I committed something that fixed this -- replaced both $name and $id with $field.

#3

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

nobody click here