Merge lp://qastaging/~divmod-dev/divmod.org/829872-createnew-type-check into lp://qastaging/divmod.org

Proposed by Tristan Seligmann
Status: Superseded
Proposed branch: lp://qastaging/~divmod-dev/divmod.org/829872-createnew-type-check
Merge into: lp://qastaging/divmod.org
Prerequisite: lp://qastaging/~divmod-dev/divmod.org/829869-explicit-type-dropping
Diff against target: 202 lines (+40/-14)
3 files modified
Axiom/axiom/substore.py (+9/-0)
Axiom/axiom/test/test_batch.py (+7/-5)
Axiom/axiom/test/test_substore.py (+24/-9)
To merge this branch: bzr merge lp://qastaging/~divmod-dev/divmod.org/829872-createnew-type-check
Reviewer Review Type Date Requested Status
Jean-Paul Calderone Approve
Jonathan Jacobs Approve
Review via email: mp+72277@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2013-07-07.

To post a comment you must log in.
Revision history for this message
Jonathan Jacobs (jjacobs) wrote :

1. I think this should either be %r or you should mention the type to make figuring out what you're *actually* passing a little easier, __str__ can be deceiving:

10 + 'Received %s instead of a sequence' % (pathSegments,))

2. Can you assert a little bit more about the exception in test_createNewStringPath?

Merge when you're happy with those.

review: Approve
2678. By Tristan Seligmann

Eliminate $PYTHONPATH rewriting in code that launches python / twistd child processes.

2679. By Tristan Seligmann

Avoid relying on implicit-close-by-refcounting in the test suite.

2680. By Tristan Seligmann

Merge branch again for one last trivial deprecation fix that got missed.

2681. By Tristan Seligmann

Simplify the metaclass trickery that makes subclasses of AxiomaticCommand
automatically provide IPlugin and IAxiomaticCommand.

2682. By Tristan Seligmann

Remove the indirect-attribute trickery in TypedInterface (which no longer works
with newer versions of z.i) in favour of setting attributes directly on the
interface (which seems to have started working quite some time ago).

2683. By Jean-Paul Calderone

Give Imaginary a new basis for how the simulation graph is represented and used.

This changes the basis of the simulation to Ideas, Things, link contributors,
link annotators, and restrictions. It makes the preferred interface to use the
simulation graph `Idea.obtain` or `Thing.obtainOrReportWhyNot`. It also adds
some new application code using these APIs to implement novel behaviors.

2684. By Jean-Paul Calderone

Update the Mantissa web integration tests to work with Twisted 11.0.

2685. By Allen Short

Merge "duplicate-class-defs".

2686. By Tristan Seligmann

Merge Pyflakes 0.5.0 release branch.

2687. By Jean-Paul Calderone

Update `test_headerToUnicode` to assert better behavior in one of its cases; prior to Python 2.6, some data was lost, and the test asserted that: on Python 2.7, the data is preserved, and the new version of the test now requires that (so the test will fail on Python 2.6, as it should).

2688. By Jean-Paul Calderone

Add WindowsError as a known built-in name on all platforms, suppressing warnings about it being used but undefined when using pyflakes on a non-Windows platform.

This may suppress legitimate warnings, since using WindowsError in a code path that
will actually be evaluated on a non-Windows platform is invalid. However, it is too
hard for pyflakes to figure out which code paths will run, so trade the lost legitimate
warnings in exchange for also getting rid of some bogus warnings.

2689. By Allen Short

Merge 'amp-vertex-2580'
Author: washort
Reviewer: MostAwesomeDude

Replace JUICE with AMP in Vertex.

2690. By Allen Short

Remove 'vertex._unfortunate_defer_hack'

2691. By Allen Short

Make Vertex not depend on Epsilon.

2692. By Jean-Paul Calderone

Fix several broken uses of the deleted scheduler item type from the Quotient POP3 grabber code.

The scheduler item was removed a while ago. Several untested code paths through the POP3 grabber still used it, though. These are now changed to adapt the store to IScheduler instead. New tests are also added for them.

2693. By Jean-Paul Calderone

Fix an unhandled exception in the POP3 grabber timeout case.

Previously the POP3 grabber would fail with an AttributeError if a timeout occurred on the
POP3 connection. This was caused by trying to set the status on the grabber object after
the reference to the grabber object had been discarded. When trying to set the status, the
code now first checks to see if a grabber object is available to do that.

2694. By Jean-Paul Calderone

Add extra help for making sure the batch process is started when it is needed.

Any time a remote work item is added, check to see if the batch
process needs to be started. This ensures remote work items
are processed in a slightly more timely fashion than was
previously the case.

2695. By Jean-Paul Calderone

Start the scheduler service as soon as the store service is started.

Author: exarkun
Reviewer: mithrandi
Fixes: lp:910923

2696. By Jean-Paul Calderone

Update Quotient usage of spambayes to pass the new required `mode` argument when constructing Hammie instances.

This allows Quotient to work with the latest release of spambayes, though it breaks Quotient against older
versions of spambayes.

2697. By Jean-Paul Calderone

Switch to using a SQLite3 database for spambayes classifier training data, instead of a pickle file.

Author: exarkun
Reviewer: mithrandi

The reasons for preferring SQLite3 are manifold. Performance, reliability, inspectability,
and maintainability should all be improved by this change. An upgrader is included, of
course, to convert old data. However, the old pickle file is left in place for manual
deletion afterwards, at the discretion of a system administrator.

2698. By Jean-Paul Calderone

Improve SQLite3 Spambayes storage implementation performance

Author: exarkun
Reviewer: mithrandi

Change the SQLite3 Spambayes storage implementation to load data for all of a
message's token data in as few SQL operations as possible, instead of
loading one token's data per SQL operation.

Revision history for this message
Jean-Paul Calderone (exarkun) wrote :

Looks good, please merge.

review: Approve
2699. By Jonathan Lange

Separate logic for reporting results.

 * Author: jml
 * Reviewers: exarkun, jml

2700. By Jean-Paul Calderone

Update two Nevow unit tests to work with more recent versions of Twisted

This will introduce a test failure against Twisted < 11.0, but no implementation
changes are actually being made, so no actual functionality should be lost on
old Twisted versions.

2701. By Florent

Move Pyflakes out of the Divmod.org's repository

2702. By Allen Short

vertex moved to https://github.com/twisted/vertex/

2703. By Jonathan Jacobs

Accept varargs in Divmod.UnitTest.TestCase.assertThrows.

Author: jjacobs
Reviewer: exarkun

Pass varargs to assertThrows on to the callable, matching the way Python's assertRaises works.

2704. By Tristan Seligmann

Make __conform__ work for storeless items with in-memory powerups.

2705. By Tristan Seligmann

Reformat code.

2706. By Tristan Seligmann

Write a test, make it pass.

2707. By Tristan Seligmann

Fix incorrect uses of SubStore.createNew.

2708. By Tristan Seligmann

Reformat code and change failUnless to assertTrue.

2709. By Tristan Seligmann

Fix formatting string.

2710. By Tristan Seligmann

Assert about exception message.

2711. By Tristan Seligmann

Fix merge.

2712. By Tristan Seligmann

More fixing.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: