Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 CreditAttribution: lolandese commentedPatch provided that adds a submodule 'Uptime test' that:
test_dependencies[] = uptime_widget_test
to the .info to be recognized by simplytest.me (should be enabled automatically)Comment #2
lolandese CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: lolandese commentedCommitted: http://drupalcode.org/project/uptime_widget.git/commitdiff/b9415f2
Comment #11
lolandese CreditAttribution: lolandese commentedAnd a new patch, not to be committed but only for deploy on simplytest.me.
Comment #12
lolandese CreditAttribution: 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 CreditAttribution: lolandese commentedWell, we forgot that the cron was needed also to update the uptime ratio.
New patch.
Comment #14
lolandese CreditAttribution: lolandese commentedCommitted: https://drupal.org/commitlog/commit/48176/aef24a0a2ea1ca55c50b7545091052...
Comment #15
lolandese CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: lolandese commentedAdded
simplytest_dependencies[] = uptime_widget_test
to 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 CreditAttribution: lolandese commentedRemoved PageRank Widget from the demo that has its own demo now.
Comment #20
lolandese CreditAttribution: lolandese commentedComment #21
lolandese CreditAttribution: lolandese commentedComment #23
lolandese CreditAttribution: lolandese commented