Gregory Szorc <gregory.szorc@gmail.com> [Sun, 20 Apr 2014 17:10:49 -0700] rev 21491
run-tests: replace Test.ignore() with raise IgnoreTest
Gregory Szorc <gregory.szorc@gmail.com> [Sun, 20 Apr 2014 17:09:52 -0700] rev 21490
run-tests: replace Test.skip() with raise SkipTest
All the result functions now raise an Exception. We no longer need these
functions and we can inline the raise.
Gregory Szorc <gregory.szorc@gmail.com> [Sun, 20 Apr 2014 17:06:57 -0700] rev 21489
run-tests: remove some legacy code around result handling
unittest handles all results now. We no longer need to manually keep
track of results.
Gregory Szorc <gregory.szorc@gmail.com> [Sun, 20 Apr 2014 17:04:42 -0700] rev 21488
run-tests: merge MercurialTest into Test
Now that we execute all tests via unittest, the MercurialTest wrapper is
not necessary. This patches moves the logic from MercurialTest into Test
and makes Test a child of unittest.TestCase.
Steven Brown <StevenGBrown@gmail.com> [Sat, 17 May 2014 17:11:06 +0800] rev 21487
check-code: check for consistent usage of the websub filter in hgweb templates
The check-code tool now expects the "desc" keyword to be followed by the
"websub" filter, with the following exceptions:
a) It has no filters at all, e.g. a changeset description in the raw style
templates or the repository description in the summary page.
b) It is followed by the "firstline" filter, e.g. the first line of the
changeset description is displayed as a summary or title.
Steven Brown <StevenGBrown@gmail.com> [Sat, 17 May 2014 17:10:23 +0800] rev 21486
hgweb: apply websub filter to the changeset description in rss and atom feeds
For example, this is useful for linking from the feed reader to a bug tracker.
This follows the existing pattern used within the hgweb templates. With the
exception of the raw style, all usages of the changeset "desc" keyword are now
followed by either the "firstline" filter or the "websub" filter. When "websub"
is used, it always follows the "escape" filter.
Steven Brown <StevenGBrown@gmail.com> [Sat, 17 May 2014 17:09:46 +0800] rev 21485
tests: write the hgrc file for test-hgweb-commands.t with cat instead of echo
To improve the readability.
Steven Brown <StevenGBrown@gmail.com> [Sat, 17 May 2014 17:09:10 +0800] rev 21484
tests: tests for the rss feeds
This matches the existing tests for the atom feeds.
Sean Farley <sean.michael.farley@gmail.com> [Wed, 23 Apr 2014 15:54:26 -0500] rev 21483
localrepo: use _matchstatus of context objects
Now that we have the machinery in place, we call the context method for
changing the match object in the case of comparing the working directory with
its parent.
This removes the last of the hard-coded workingctx knowledge in localrepo
paving the way for us to remove localrepo.status completely.
Sean Farley <sean.michael.farley@gmail.com> [Thu, 24 Apr 2014 08:32:28 -0500] rev 21482
workingctx: override _matchstatus for parentworking case
This patch encapsulate the logic for changing the match.bad function when
comparing against the working directory's parent. Future patches will remove
more of the 'if ... else' blocks in localrepo.status that test for this working
directory parent case.
Sean Farley <sean.michael.farley@gmail.com> [Wed, 23 Apr 2014 15:39:30 -0500] rev 21481
basectx: add _matchstatus method for factoring out last of parentworking logic
This patch paves the way to allow a workingctx to override the match object
with a custom 'bad' method for cases where status is sent a directory pattern.
Sean Farley <sean.michael.farley@gmail.com> [Thu, 24 Apr 2014 08:34:44 -0500] rev 21480
workingctx: use inheritance for _buildstatus while keeping the fastpath
This patch maintains the fast path for workingctx which is to not build a
manifest if the working directory is being compared to its parent since, in
this case, we can just copy the parent manifest.
Sean Farley <sean.michael.farley@gmail.com> [Tue, 22 Apr 2014 19:06:37 -0500] rev 21479
localrepo: remove unneeded unpacking of r
This unpacking is unneeded now because previous patches have removed the need
for this method to modify each of these variables in favor of passing the whole
set around to pre/post hook methods of the corresponding context object.
Sean Farley <sean.michael.farley@gmail.com> [Tue, 22 Apr 2014 13:02:38 -0500] rev 21478
localrepo: call _poststatus hook instead of hardcoding symlink knowledge
Sean Farley <sean.michael.farley@gmail.com> [Tue, 22 Apr 2014 12:59:22 -0500] rev 21477
workingctx: add _poststatus method to call _filtersuspectsymlink
With this patch, we are one step closer to removing 'if ... else' logic in
localrepo.status for testing if the context is the working directory or
not. Future patches will replace those blocks of code with a call to the
context's _poststatus hook so that each context object will do the right thing
depending on the situation.
Sean Farley <sean.michael.farley@gmail.com> [Tue, 22 Apr 2014 12:51:58 -0500] rev 21476
context: add a no-op _poststatus method
This patch adds a private _poststatus method so that certain contexts, such as
workingctx, can add custom post-processing to status.
Sean Farley <sean.michael.farley@gmail.com> [Tue, 22 Apr 2014 12:43:20 -0500] rev 21475
localrepo: call _prestatus hook instead of hardcoding dirstate knowledge
Sean Farley <sean.michael.farley@gmail.com> [Mon, 21 Apr 2014 22:12:59 -0500] rev 21474
workingctx: add _prestatus method to call _dirstatestatus
With this patch, we are one step closer to removing 'if ... else' logic in
localrepo.status for testing if the context is the working directory or
not. Future patches will replace those blocks of code with a call to the
context's _prestatus hook so that each context object will do the right thing
depending on the situation.
Sean Farley <sean.michael.farley@gmail.com> [Mon, 21 Apr 2014 21:39:10 -0500] rev 21473
context: add a no-op _prestatus method
This patch adds a private _prestatus method so that certain contexts, such as
workingctx, can add custom pre-processing to status.
Sean Farley <sean.michael.farley@gmail.com> [Mon, 21 Apr 2014 21:36:14 -0500] rev 21472
localrepo: use _buildstatus from context
Sean Farley <sean.michael.farley@gmail.com> [Mon, 21 Apr 2014 21:35:36 -0500] rev 21471
context: add _buildstatus method
This method is a copy of localstatus.status's core logic. Later patches will
clean up some of the dense coditionals in the for loop.
Sean Farley <sean.michael.farley@gmail.com> [Mon, 21 Apr 2014 20:42:42 -0500] rev 21470
localrepo: remove workingctx logic already moved to context
Sean Farley <sean.michael.farley@gmail.com> [Mon, 21 Apr 2014 20:23:47 -0500] rev 21469
localrepo: move the order of local variables
This movement is a small step in getting rid of the 'if ... else' logic for
testing the current working directory with its parent. Previously, the deleted,
unknown, and ignored variables were set in a combination of before an 'if
... else' block and within the block. This moves the variables to be set
outside the loop in one common place.
Sean Farley <sean.michael.farley@gmail.com> [Tue, 15 Apr 2014 15:43:30 -0500] rev 21468
localrepo: factor out _manifestmatch logic for workingctx
Sean Farley <sean.michael.farley@gmail.com> [Tue, 11 Mar 2014 18:38:44 -0500] rev 21467
localrepo: use _manifestmatches instead of duplicating logic
Sean Farley <sean.michael.farley@gmail.com> [Wed, 23 Apr 2014 20:52:10 -0500] rev 21466
basectx: add _manifestmatches method
This method is a duplicate of localrepo.mfmatches and sets the stage for
factoring localrepo.status into a context method that will be customizable
based on inheritance and object type.
Sean Farley <sean.michael.farley@gmail.com> [Tue, 11 Mar 2014 18:35:39 -0500] rev 21465
localrepo: always set the variable r in status method
This is a small patch to help streamline keeping tracking of the list of files
for status in a variable already called 'r' ('s' is for subrepos in this
method). We now move the setting of it out of an 'if' block so that we can
later refactor more code into the context objects.
Gregory Szorc <gregory.szorc@gmail.com> [Sun, 20 Apr 2014 16:44:45 -0700] rev 21464
run-tests: execute tests via unittest
Now that unittest mode is functionally equivalent to the default mode,
we switch the default execution mode to unittest and remove the choice
of different execution modes.
Gregory Szorc <gregory.szorc@gmail.com> [Tue, 22 Apr 2014 13:37:27 -0700] rev 21463
run-tests: implement Test.__str__
unittest sometimes does a str(test) and the default implementation
doesn't do the right thing. So we override it.
Gregory Szorc <gregory.szorc@gmail.com> [Mon, 21 Apr 2014 12:15:55 -0700] rev 21462
run-tests: make failure reporting in unittest mode equivalent to default
Unlike unittest's defaults, our result formatter does not print stack
traces. Here, we change TestResult.addFailure() to be compatible with
the existing/default execution mode.