We would appreciate it is only project maintainers commented on this issue.
Please list the projects you maintain that you would like to be part of the beta stage and if they currently have any tests. For example if I were to comment requesting the subuser module to be added:
subuser - has tests
We will contact you if any of your projects are included in the beta stage.
Thank you.
UPDATE: Currently enabled contrib project (more to come - as we fix bugs and such).
3060 drupal
3236 devel
3256 image
3277 poormanscron
3279 privatemsg
5461 weather
8404 captcha
11873 simpletest
17345 pathauto
19304 media
23158 comment_notify
41542 porterstemmer
59060 taxonomy_filter
65961 geshifilter
106016 token
108746 admin_menu
109640 services
128621 adminrole
129418 potx
138844 autoassignrole
144819 content_access
148272 l10n_server
177400 openlayers
190124 rules
190839 xmlsitemap
204326 talk
240806 mollom
243795 linkchecker
253481 storm
287853 languageicons
313122 project_issue_file_review
333543 role_change_notify
441778 search_by_page
476696 og_statistics
477446 grammar_parser
511618 faces
520254 nd
543396 encrypt
571608 password_change
594964 examples
597564 blogapi
623208 profile2
626176 entity
689342 proxy
691808 contact
730762 simpletest_selenium
and vars, google_analytics, versioncontrol, vote_up_down, opensearch, transformers, libraries, multicron, smartcrop, edge, translation, user_delete, piwik, embeddable, skinr, rtsg, profile_permission, userpoints, userpoints_nc, userpoints_contrib, chessboard, css_injector, uuid, nodequeue, properties, date_popup_authored, references, taxonews, g2, fontyourface, myspace_sync, field_collection, mailhandler, enabled_modules, commerce, cck_signup, ubercart, restws, views, cas, current_content.
Thanks to people reporting issues we were able to fix the following issues and deploy the changes, I'll be enabling more contrib projects now. (01/28/10)
PIFR 6.x-2.2, 2010-01-27
------------------------
- Bugs:
* #695350: Provide 'last' field in pifr.retrieve() and correct query.
* Events should only be triggered when trigger modules is available.
* Remove notices when test record is saved before client record.
* #695278: Test list should be generated from root directory.
* #695278: Module 'tests' directory should be searched.
* #696044: Patches are not being applied properly to contrib projects.
* #696194: Cannot preview client test information.
* Do not add SimpleTest as dependency if it has already been added.
Comments
Comment #1
cyberswat commentedautoassignrole - has tests
Edit by Dave: confirmed testing enabled
Comment #2
dave reidXML sitemap - 100% test suite pass (and technically is already enabled for testing on drupal.org)
Admin role - 100% test suite pass
Poormanscron - 100% test suite pass
Password change confirm - 100% test suite pass
Examples - would be great to make sure they're current with development
Edit by Dave: confirmed testing enabled
Comment #3
dave reidContact - backport/experimental replacement for core contact.module
Edit by Dave: confirmed testing enabled
Comment #4
sunhttp://drupal.org/project/admin_menu - has tests
http://drupal.org/project/mollom - has extensive tests
Edit by Dave: confirmed testing enabled
Comment #8
berdirPrivatemsg - has tests (Drupal 6 and 7)
Edit by Dave: confirmed testing enabled
Comment #9
fagoContent Access has tests
Rules has tests (many more in d7)
Entity API has tests (has two different modules to test, does that work?)
Faces API has tests
Profile2 has tests
Edit by Dave: confirmed testing enabled
Comment #10
dries commentedCan we add Mollom, please? We have many tests for both D6 and D7.
Comment #11
joachim commentedhttp://drupal.org/project/image - has tests
Edit by Dave: confirmed testing enabled
Comment #12
toddy commentedhttp://drupal.org/project/weather - has tests
Edit by Dave: confirmed testing enabled
Comment #13
hass commentedhttp://drupal.org/project/linkchecker also have tests and may show many coder bugs. I have for sure found many false alarms in past and they are not fixed as I know.
Edit by Dave: confirmed testing enabled
Comment #14
gddhttp://drupal.org/project/services - has no tests but we're getting ready to start rolling a stable release and this would prove really helpful with handling bug fix patches.
Edit by Dave: confirmed testing enabled
Comment #15
zzolo commentedhttp://drupal.org/project/encrypt - has tests
http://drupal.org/project/proxy - has tests (though failing at the moment)
Edit by Dave: confirmed testing enabled
Comment #16
aaron commentedhttp://drupal.org/project/media -- still in development; has tests
Comment #17
jhodgdonNot sure how many you want here, but
http://drupal.org/project/porterstemmer has tests
http://drupal.org/project/search_by_page has tests, and several of them depend on other contrib modules
Comment #18
gábor hojtsyhttp://drupal.org/project/potx - has extensive tests
http://drupal.org/project/l10n_server - has extensive tests
Comment #19
cwgordon7 commentedtalk - has tests (DRUPAL-6--1 branch)
Comment #20
soxofaan commentedcaptcha - has tests
geshifilter - has tests
Comment #21
swentel commentedhttp://drupal.org/project/nd - has tests
Comment #22
Magnity commentedhttp://drupal.org/project/storm - has 1 basic test, with more to come soon.
Comment #23
Freso commentedhttp://drupal.org/project/languageicons - No tests (yet), but I certainly wouldn't mind being included in this beta. =)
Comment #24
joachim commentedThe issue description has been updated, and seems to imply the contrib testing bot is now live. Do existing patches that were already set to 'needs review' need prodding in some way for the bot to notice them? -- they still say 'ignored'. Eg: http://drupal.org/node/412288
Comment #25
boombatower commentedThe system does not cycle in all previously existing patches, adding a comment of any kind to the issue will trigger a test. Also making a commit will trigger the branch test. If the branch test does not pass, the patches will be postponed.
Parsing all previously existing patches is not currently supported.
Comment #26
zzolo commentedFor Proxy (listed above as being enabled), I am not able to get the testing to start for patches and it still says ignored; for example:
http://drupal.org/node/699180#comment-2548690
Also, will there be an easy way to find the branch testing, like a link on the project page?
Thanks for the great work on this. It is very awesome!
Comment #27
Magnity commentedAre there any plans to add more contrib projects to the beta stage now that the testbot is back up and running?
Thanks.
Comment #28
dave reid@zzolo You have to make a commit to the desired branch to get it 'enabled' for testing.
Comment #29
dawehnerhttp://drupal.org/project/og_statistics - has tests for every feature.
Comment #30
ilo commentedCweagans and I are going to have several sprints on blogapi module. I'm not the module mantainer, however cweagans asked me to do this request because he was not going to be able to do for the next days as commented in the issue:#686832: Split connection methods into abstracted backends
So, our interest is to have the blogapi module included in the contrib testbot.
Thanks!
Comment #31
rfayPlease add role_change_notify to the test, as it now has tests.
Comment #32
zzolo commentedopenlayers 2.x branch now has a number of tests.
Comment #33
Freso commentedPathauto's HEAD finally also has some proper tests, so it would be good to have that included in this as well. :)
Comment #34
gregglesPathauto HEAD and 6.x* branches are already included - http://qa.drupal.org/pifr/test/31923
comment_notify - has tests that pass on 6.x branch. The 7.x tests for comment notify are currently broken.
Comment #35
drummAPI module's tests will cleanly pass when #735724: Remove globalToString() echo is resolved. However, I'm guessing an automated 6.x checkout of Grammar Parser will fail due to the "Looking for 6.x support?" note at http://drupal.org/project/grammar_parser.
Comment #36
tstoecklerClock - has tests for 6 and 7.
Comment #37
steven jones commentedContext - has tests
Comment #38
steven jones commentedAh, sorry can you remove the
contextproject, we depend on a lot of contrib modules, and as simpletest gets the wrong versions a lot of our tests fail when they really shouldn't. Apologies for time wasted.Comment #39
boombatower commented@Steven Jones: please check back when you see another front page post about an update as it should provide a way to specify versions.
Comment #40
gordon commentedCan you please also add ecommerce http://drupal.org/project/ecommerce
Comment #41
Anonymous (not verified) commentedplugin_manager - has some tests. More are coming.
Comment #42
grendzy commentedsecurepages_prevent_hijack - has tests
Comment #43
janusman commentedMillennium Integration ... 6.x-2.x-dev version only. See http://drupal.org/cvs?commit=356414
Comment #44
boombatower commentedAdded last four.
Comment #45
mikeker commentedThanks to you and Kieran at the QA sprint, I now have a meager handful of tests for my module. More to come!
Better Exposed Filters - has tests
Comment #46
janusman commentedQuestion: So... by just marking an issue as "needs review" in my issue queue, that enough to get tested? Nothing looks different when I do (see http://drupal.org/node/468388#comment-2874104 )
**EDIT** nevermind, found it =) [Thanks @berdir]
Comment #47
Anonymous (not verified) commentedVariables API — has tests.
Comment #48
hass commentedPlease add Google Analytics, too.
Could someone give me a small hint, please - why the patch in http://drupal.org/node/249701#comment-2979452 is "ignored" and not tested against 6.x-2.x?
Comment #49
gábor hojtsyl10n_server which is included in the beta stage has issues with how the testing framework is set up vs. the tests being run: #802034: l10n_server testing fails due to potx not being there/not enabled
Comment #50
marvil07 commentedVersion Control API - has tests
Vote up/down - has tests, few and a little broken but including it will let me fix it quicker ;-)
Comment #51
jhodgdonRE #48: I think the project issue testing module might have a bug left over from the past where it used to ignore patches with a -D6 suffix. You might want to file an issue on the Project Issue Testing project.
Comment #52
Anonymous (not verified) commentedOpensearch feed — has tests.
Comment #53
kenorb commentedsubscribe
Comment #54
sepgil commentedRules Transformers has some few tests, but I'll add more, as the project advances.
Comment #55
tstoecklerLibraries API now comes with tests! D7 version only, for now.
Comment #56
dstolMulticron has tests.
Comment #57
grendzy commentedsmartcrop - has tests
Comment #58
sunhttp://drupal.org/project/edge comes with tests, too
Comment #59
sunhttp://drupal.org/project/translation could use the testbot, too :)
Comment #60
ilo commentedHi, http://drupal.org/project/user_delete (an attempt to backport D7) account cancellation module is including tests now.
Thanks!
Comment #61
Anonymous (not verified) commentedBook access - has tests.
Comment #62
hass commentedhttp://drupal.org/project/piwik has tests.
Comment #63
fagohttp://drupal.org/project/embeddable has tests.
Comment #64
jacinehttp://drupal.org/project/skinr has tests. :)
Comment #65
ilo commentedBoombatower, I kindly ask you to include http://drupal.org/project/rtsg in the testing framework :)
Comment #66
ilo commentedModules that would love to be integrated in the next round..
http://drupal.org/project/userpoints
http://drupal.org/project/userpoints_nc
eventually http://drupal.org/project/userpoints_contrib can be also included (due to the D7 upgrade on the way, but not sure if d6 has any testcase)
Comment #67
eric_a commentedChessboard Renderer - has tests
88 passes, 0 fails, and 0 exceptions
Comment #68
rfayPlease add css_injector.
Comment #69
ilo commenteduuid has tests now. Thanks!
Comment #70
ezra-g commentedNodequeue has tests: http://drupal.org/node/468136#comment-3816526 - Thanks!
Comment #71
berdirProperties - has tests
Thanks :)
Comment #72
zzolo commentedThis is really great! Thanks for all the hard work on this. It really adds lots of value to developing on d.o. I am curious if there is a discussion, or what the status is, on getting automated Coder reviews for the contrib space as well (as I know it is available for core)?
Comment #73
boombatower commentedWe had a halt declared on all bot improvement while Drupal 7 was attempting to stabilize for final release...now that Drupal 7 is out development when continue...stay tuned.
Comment #74
zzolo commentedThanks @boombatower for all the great work. If there's some way I can help, please let me know.
Comment #75
mark trappNot sure if you're still accepting submissions here, but Date Popup Authored has tests.
Comment #76
sunFor some time, I'm having serious trouble with the testbot in two contributed projects:
The XML-RPC output also lists that
example_info_file.librariesmodule, which does not really exist (an .info file with that name exists for testing purposes, but no module).Tests are passing successfully when being run locally.
Here, also, tests are passing successfully when being run locally.
Not sure whether this is related, but the XML-RPC output for that test shows that the testbot added a bogus additional dependency on the Skinr DRUPAL-7--1 branch (which is empty though).
I'm sorry for posting this in this issue, but I have absolutely no idea what the proper place for reporting such issues is. In particular, it's very hard to figure out whether this is an issue with SimpleTest, PIFR, PIFT, or some other component of the testing framework. In order to go out of beta with contributed module testing, we definitely need a single dedicated project issue queue to handle such issues.
Comment #77
berdirI have a very similiar issue with Userpoints. Sometimes, the testbot runs "Rules Scheduler" tests additionally to our own, producing unrelated test failures. See #945622-2: 'Reason' column: Description gets replaced by truncated operation when you edit point transactions created by other modules for an example.
Comment #78
Anonymous (not verified) commented@sun: Basing on the URL used for the test log pages (eg, http://qa.drupal.org/pifr/test/110619), I guess the project for which such issues should be reported is PIFR. AFAIK, PIFT is the module that integrates a server running PIFR with drupal.org.
I have created the issue #1023220: Tests enable extra, unrelated modules (and more).
Comment #79
fgmCan you please add
- Reference (tests on D7 only)
- Taxonews (tests on both D6 and D7)
- G2 (tests on D5 and D7, not D6)
Comment #80
gregglesbetter title.
Comment #81
gregglesAnd now with 100% less repetitiveness and redundancy.
Comment #82
sreynen commentedhttp://drupal.org/project/fontyourface - has tests
http://drupal.org/project/myspace_sync - has tests
Comment #83
fagofield-collection - has tests.
Comment #84
ilo commentedIn the time you read this comment I'll be submitted the testcase for the Mailhandler project.
Thanks in advance!
Comment #85
catchEntity cache - has a test stolen from core.
Comment #86
juliangb commentedEnabled Modules (http://drupal.org/project/enabled_modules).
Tests to be added soon.
Comment #87
rszrama commentedhttp://drupal.org/project/commerce - has hella-tests, still adding more... poor pcambra's lappy takes an hour and a half to run 'em all.
Comment #88
jhedstromhttp://drupal.org/project/cck_signup has tests (in 7.x only).
Comment #89
tr commentedhttp://drupal.org/project/ubercart - has about 1000 tests on each of the D6 and D7 branches, and more coming soon. Thanks!
Comment #90
tr commentedAre the tests being run taken directly from the repository, from the latest development snapshot, or from the latest tagged release of the contrib project?
Comment #91
dave reid@TR Tests are run based on the version specified in the individual issues. It also runs 'branch' tests when you commit code to a branch, but those results are not displayed on d.org. They're only used to determine if it can continue to run tests on issues or if it needs to postpone tests.
Comment #92
rszrama commentedIf I'm not mistaken, it's not working if you have dependencies, right? i.e. Commerce wouldn't work, so I'm guessing Ubercart won't either?
Comment #93
dave reidYes, if your module has dependencies on other projects, testing does not currently work very well. You can work around it to use testing as a 'does this patch apply or not', but not currently as a reliable testing framework.
Comment #94
rfayI do hope we'll be able to get loading dependencies to work in the next month or two (in the near term). Fixing the criticals on PIFR is my biggest community priority. It's just that there are a lot of them. And boombatower already made this work (I think) in the mainline (master), but getting that deployed is still a pain.
Comment #95
rfayBTW: I filed #1093886: Allow 'pift enable project testing' permission for project owners/enabled users to hopefully let project owners manage this themselves.
Comment #96
plachTitle has a couple of tests and we expect to have more in the near future ;)
Comment #97
klausirestws has tests.
Comment #98
berdirUser relationships has some tests, more to come (at least in D7)
Comment #99
kenorb commenteddrupal.org/project/views please
Comment #100
merlinofchaos commentedkenorb: Thank you for impacting a busy development day by having this enabled on my module without my knowledge or permission.
In case you didn't realize, PIFR will automatically mark issues 'needs work' if the patch causes tests to fail. That means if the tests aren't perfect, perfectly valid patches get thrown into the 'needs work' graveyard and nobody knows about it. Thanks to you, I'm now stuck trying to figure out what pifr just screwed up for me.
For the love of all that is holy if this framework is enabled on a module, make ABSOLUTELY SURE the primary maintainer of the module consents, because if things aren't set up right, it is destructive.
Comment #101
rszrama commentedNot to mention the test bot won't really work for Views since it has a dependency on Ctools. :-/
Comment #102
avpadernoI think that this issue is thought to be used by maintainers to request automated testing on projects they maintain.
kenorb should not have asked to enable them for Views, as he doesn't even have write access to Views repository.
Comment #103
rfay/me was already long ago adequately lashed and repentant :-)
Comment #104
kenorb commented@merlinofchaos: Sorry, if that cause any problem. I could help with test fails, I'm currently working on some of them.
I found only one issue with Views test
#1101332: Call to a member function set_display() on a non-object in tests/views_exposed_form.test on line 34
which I can solve.
Secondly enabling modules for test bot should be part of project configuration, not by posting module names in the issue. Especially when people don't know exactly how this test bot works, like me.
Comment #105
boombatower commentedIt is a project setting, but currently only a few admins have the permission to enable. There is an issue in the infra queue to enable for all, but I believe it was decided to continue waiting on #102102: Parse project .info files: present module list and dependency information. The intention was not to leave it for this long, but a lot of things got in the way of getting this fixed up.
Comment #106
bfroehle commentedThe CAS module has test routines. Please enable testing (for D7).
Comment #107
jn2 commentedI have a project used as a practice space for making and reviewing patches. Would like to enable testing so that participants can also experience what that's like.
Current content is the module. Already has test routines. Please enable D7 testing.
Comment #108
fgmCan you please enable MongoDB ? It already has some tests in some branches.
Comment #109
bfroehle commentedPlease enable testing for CAS Attributes.
Thanks!
I'm looking forward to being able to enable testing myself as a module maintainer in the future!
Comment #110
rfayCAS Attributes enabled
Comment #111
Niklas Fiekas commentedMedia Feeds - has tests
Comment #112
rfayMedia Fields enabled.
Comment #113
gregglesI'm not sure that simpletest can handle modules with dependencies which I imagine Media Feeds has.
Comment #114
Niklas Fiekas commentedThanks. Yes, it doesn't work. rfay pointed me to #102102: Parse project .info files: present module list and dependency information in IRC.
Comment #115
chx commentedMay I get Relation? We do not have dependencies. We work with everyone but we do not depend on them.
Comment #116
xjmTaxonomy access has the beginnings of tests in the 7.x branch, and they'll be backported to 6.x eventually.
Comment #117
rfayRelation and taxonomy_access are turned on.
Comment #118
dstoltaxonomy menu now has tests.
Comment #119
rfayTaxonomy Menu enabled.
Comment #120
fgmPlease add Classified Ads, which already has D6 tests which will be ported to D7.
Comment #121
rfayClassified Ads enabled.
The D7 version has lots of files[] in the .info which are not necessary (they're only needed for files that contain classes).
Comment #122
tr commentedCan you please add http://drupal.org/project/ip2country ?
This project has 200 tests that run green for D6, D7, and D8 (!!!).
Comment #123
rfayip2country enabled.
Comment #124
adooo commentedAccess by Term has tests...
Thanks...
Comment #125
rfayAccess by term enabled.
Comment #126
Niklas Fiekas commentedPlease enable feeds_namespaces. It has dependencies, but the unit tests should work regardless.
Comment #127
rfayfeeds_namespaces enabled.
Comment #128
rmiddle commentedPlease enable http://drupal.org/project/commentcloser. It has no dependencies, and the unit tests should work 100%.
Comment #129
rfaydone
Comment #130
Niklas Fiekas commentedPlease enable http://drupal.org/project/birthdays. Has tests.
Comment #131
rfayBirthdays added
Comment #132
ParisLiakos commentedIs there still a problem with modules that have dependencies, since #1241704: Deploy Project Dependency, Project and PIFT on d.o is marked as fixed?
If so are there any extra steps for modules that have dependencies before joining automated tests?
Thanks
Comment #133
boombatower commented#1093886: Allow 'pift enable project testing' permission for project owners/enabled users which needed #1299910: Remove 'Enable Automated Testing' checkbox from projects without releases
Since it has now been completed it just needs to be deployed to allow project maintainers to enable testing themselves.
Comment #134
rfay@rootatwc if you need anything enabled for testing, just say the word (until maintainers can do it themselves)
Comment #135
ParisLiakos commentedGreat!
thanks boombatower for the fast response:)
@rfray http://drupal.org/project/newsletter has some tests and i am currently writing more.
Comment #136
boombatower commentedEnabled newsletter.
Comment #137
plachEntity translation has tests.
Comment #138
rfayenabled entity translation
Comment #139
TwoMice commentedcivicrm_activity_ical - has tests
Comment #140
rfaycivicrm_activity_ical enabled
Comment #141
fgmThe mongodb tests always fail because there is no /visible/ MongoDB server to which they can connect. I wonder if it really makes sense to have them run in automated mode at all except for making sure the patches are well-formed.
Now, is there not a MongoDB server in the d.o. infra somewhere, which could be accessed by these tests ?
Comment #142
rfayNothing I know of. You might want to add a condition to your test to skip tests if it can't connect to a server.
Comment #143
rfayHooray! We no longer have to do this. Every project owner now has the capability to turn on testing. You do it by editing the project, issues tab "Enable automated testing".
See http://jthorson.doesdrupal.com/node/34 for more information.
Also please note that issues on the testbots and testing can always be filed in the Testbot queue.
Comment #145
smokrisI couldn't find documentation for this functionality in the contrib module maintainers guide, so I added http://drupal.org/node/1449736.
Comment #146
rfay@smokris++
Thanks so much.