Last modified: 2011-09-12 14:18:21 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T23598, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 21598 - Configure which paths to run tests for
Configure which paths to run tests for
Status: RESOLVED WONTFIX
Product: MediaWiki extensions
Classification: Unclassified
CodeReview (Other open bugs)
unspecified
All All
: Lowest enhancement (vote)
: ---
Assigned To: Aaron Schulz
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-11-22 22:46 UTC by Sam Reed (reedy)
Modified: 2011-09-12 14:18 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2009-11-22 22:46:18 UTC
Seems a bit pointless running the unit tests against phase3 when the only change has been say the RELEASE-NOTES or similar...
Comment 1 Bryan Tong Minh 2010-01-13 22:46:19 UTC
But it doesn't hurt. Creating a bunch of special cases does not seem worth to me the slight speed increase.
Comment 2 Chad H. 2010-10-04 14:34:58 UTC
Retitling. I think being able to configure "run tests when /trunk/phase3 or /trunk/extensions changes" or any other set of paths would be nice. It's true, running parser tests when somebody updated their USERINFO or something in /trunk/tools

However, I agree with Bryan that we shouldn't make a billion special cases, so I think this should be very general. Don't need to say "unless only X Y or Z changed, then skip it"

In any case, I removed our crappy test system in r69385 (which had been broken forever). Marking this LATER, for revival one day once someone puts a new test system into Code Review.
Comment 3 Chad H. 2010-10-04 14:35:24 UTC
(In reply to comment #2)
> Retitling. I think being able to configure "run tests when /trunk/phase3 or
> /trunk/extensions changes" or any other set of paths would be nice. It's true,
> running parser tests when somebody updated their USERINFO or something in
> /trunk/tools
> 

...would be pointless.
Comment 4 Chad H. 2011-09-12 14:18:21 UTC
We're not integrating test running with CR again -> WONTFIX.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links