Posted by beejeebus on March 4, 2009 at 8:08pm
Jump to:
| Project: | SimpleTest |
| Version: | 7.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
while working on #389040: fix bug in UserPictureTestCase::testWithGDinvalidSize, i noticed it was hard to just get a file bigger than a given size.
talked to boombatower about this in #drupal, here's a patch that adds this.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| drupalGetTestFile.patch | 8.69 KB | Idle | Failed: 10384 passes, 43 fails, 62 exceptions | View details | Re-test |
Comments
#1
The last submitted patch failed testing.
#2
I think drupalGetTestFile() and drupalGetTestFiles() should take the same parameters, so the former can be a simple wrapper around the latter. Perhaps drupalGetTestFiles() could take an additional $count parameter.
I don't know why the existing implementation uses stat() rather than filesize(). Using filesize() seems like the obvious choice.
For now, positional parameters is probably fine, but it may be an idea to array-itize the parameters (like it was done in #255551: DX: Array-itize file_scan_directory()'s parameters).
#3
I might just replace drupalGetTestFiles().
#4
+
#5
#6