As title.
| Comment | File | Size | Author |
|---|---|---|---|
| #21 | simplytest_demo-2045301-21.patch | 18.47 KB | lolandese |
| #16 | enable_test-2045301-16.patch | 681 bytes | lolandese |
| #13 | added_cron-2045301-13.patch | 709 bytes | lolandese |
| #11 | enable_test-2045301-11.patch | 541 bytes | lolandese |
| #9 | removed_double_crons-2045301-9.patch | 2.67 KB | lolandese |
Comments
Comment #1
lolandese commentedPatch provided that adds a submodule 'Uptime test' that:
test_dependencies[] = uptime_widget_testto the .info to be recognized by simplytest.me (should be enabled automatically)Comment #2
lolandese commentedCommitted. Tested by using http://simplytest.me/project/uptime_widget/7.x-1.x
Simplytest.me uses git, thus our test module is in the module list (under 'Development') even before the commit has been pacckaged by d.o.
Our test module is not enabled by default, thus leaving status "needs work".
Comment #3
lolandese commentedProjects specified with test_dependency will only be automatically downloaded (if they are external), but not enabled .
We still have to make use of a patch to add the real dependency to the main .info file.
Simplytest.me's maintainer indicated this behavior might change in the future.
Added mini patch to add uptime_widget_test as a hard dependency, so it will get enabled.
Comment #4
lolandese commentedIn this particular case we also need cron to run. A patch to the .install file to both enable the test module and run cron is the easiest.
Comment #5
lolandese commentedhttp://simplytest.me/project/uptime_widget/7.x-1.x?patch[]=https://drupal.org/files/enable_test_and_run_cron-2045301-4.patch
is working as expected.
Comment #6
lolandese commentedTo adapt the patch after last commit http://drupalcode.org/project/uptime_widget.git/commitdiff/d798146 related to #2047017: Render the widget identical to the sister module PageRank Widget
Comment #7
lolandese commentedTest on:
http://simplytest.me/project/uptime_widget/7.x-1.x?add[]=pagerank_widget&patch[]=https://drupal.org/files/enable_test_and_run_cron-2045301-5.patch
Comment #8
lolandese commentedOne first test simplytest.me hanged on 67% of installation. Another test this morning went without problems.
Needs to remove activation link to simpletest.me on the sample node. If we see that, we are already in the demo.
Comment #9
lolandese commentedAttached patch removes some double cron runs. That might be reason for longer or hanging install of the simplytest.me. Also from the latest patch in #7 we can remove cron run.
Furthermore we improved the content of the created node.
Comment #10
lolandese commentedCommitted: http://drupalcode.org/project/uptime_widget.git/commitdiff/b9415f2
Comment #11
lolandese commentedAnd a new patch, not to be committed but only for deploy on simplytest.me.
Comment #12
lolandese commentedSo, finally, the new demo site should be under:
http://simplytest.me/project/uptime_widget/7.x-1.x?add[]=pagerank_widget&patch[]=https://drupal.org/files/enable_test-2045301-11.patch
Comment #13
lolandese commentedWell, we forgot that the cron was needed also to update the uptime ratio.
New patch.
Comment #14
lolandese commentedCommitted: https://drupal.org/commitlog/commit/48176/aef24a0a2ea1ca55c50b7545091052...
Comment #15
lolandese commentedNow it works as expected.
The new demo site is still under:
http://simplytest.me/project/uptime_widget/7.x-1.x?add[]=pagerank_widget&patch[]=https://drupal.org/files/enable_test-2045301-11.patch
Comment #16
lolandese commentedYet another patch. :)
Test with http://simplytest.me/project/uptime_widget/7.x-1.x?add[]=pagerank_widget&patch[]=https://drupal.org/files/enable_test-2045301-16.patch
Comment #18
lolandese commentedAdded
simplytest_dependencies[] = uptime_widget_testto the .info file. Committed.After a great enhancement on Simplytest.me this now results in the demo sub-module being enabled without having to use a patch to do that.
http://simplytest.me/project/uptime_widget/7.x-1.x?add[]=pagerank_widget
Comment #19
lolandese commentedRemoved PageRank Widget from the demo that has its own demo now.
Comment #20
lolandese commentedComment #21
lolandese commentedComment #23
lolandese commented