Opened 10 years ago

Closed 9 years ago

#2935 enhancement closed fixed (fixed)

Use FancyEqMixin and FancyStrMixin for all Record_* types and Name in twisted.names.dns

Reported by: Jean-Paul Calderone Owned by:
Priority: normal Milestone:
Component: names Keywords:
Cc: therve Branch: branches/dns-str-eq-2935-3
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun


Many of them use one or both of these, but not all.

Change History (8)

comment:1 Changed 10 years ago by Jean-Paul Calderone

author: exarkun
Branch: branches/dns-str-eq-2631

comment:2 Changed 10 years ago by therve

Cc: therve added

What's the status of this? It would be cool if it could land, it helps unittests a lot.

comment:3 Changed 9 years ago by Jean-Paul Calderone

Branch: branches/dns-str-eq-2631branches/dns-str-eq-2935-3

(In [23926]) Branching to 'dns-str-eq-2935-3'

comment:4 Changed 9 years ago by Jean-Paul Calderone

Keywords: review added
Owner: Jean-Paul Calderone deleted

It was waiting for #2944, but that was finished a while ago I guess. Should be finished now.

comment:5 Changed 9 years ago by therve

Keywords: review removed
Owner: set to Jean-Paul Calderone
  • in EqualityTests._equalityTest, can you add a comment about why you don't use assertEquals and friends? I probably know, but it's better to have written
  • there's an extra line after _equalityTest

Everything else is great, so please merge once both points addressed.

comment:6 Changed 9 years ago by Jean-Paul Calderone

(In [23961]) whitespace and comment about implementation

refs #2935

comment:7 Changed 9 years ago by Jean-Paul Calderone

Resolution: fixed
Status: newclosed

(In [23962]) Merge dns-str-eq-2935-3

Author: exarkun Reviewer: therve Fixes: #2935

Define string representation and equality for all record types (using FancyStrMixin and FancyEqMixin from twisted.python.util). Also add tests for these behaviors.

comment:8 Changed 7 years ago by <automation>

Owner: Jean-Paul Calderone deleted
Note: See TracTickets for help on using tickets.