add drupalGetTestFile method to drupal_web_test_case
justinrandell - March 4, 2009 - 20:08
| Project: | SimpleTest |
| Version: | 7.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | justinrandell |
| Status: | needs work |
Jump to:
Description
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 |
|---|---|
| drupalGetTestFile.patch | 8.69 KB |
#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