Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1142)

Issue 122940043: Added a very basic test to create ticket

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 8 months ago by sksaurabhkathpalia
Modified:
9 years, 8 months ago
Reviewers:
thomas.j.waldmann, Thomas.J.Waldmann
Visibility:
Public.

Description

Added a very basic test to create ticket

Patch Set 1 #

Patch Set 2 : Added some more meta-data information specific to ticket items #

Total comments: 4

Patch Set 3 : Changed the name of file to test_ticket.py #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -0 lines) Patch
A MoinMoin/items/_tests/test_ticket.py View 1 2 1 chunk +39 lines, -0 lines 0 comments Download

Messages

Total messages: 5
Thomas.J.Waldmann
https://codereview.appspot.com/122940043/diff/20001/MoinMoin/items/_tests/test_Ticket.py File MoinMoin/items/_tests/test_Ticket.py (right): https://codereview.appspot.com/122940043/diff/20001/MoinMoin/items/_tests/test_Ticket.py#newcode1 MoinMoin/items/_tests/test_Ticket.py:1: # License: GNU GPL v2 (or any later version), ...
9 years, 8 months ago (2014-08-06 12:49:39 UTC) #1
sksaurabhkathpalia
https://codereview.appspot.com/122940043/diff/20001/MoinMoin/items/_tests/test_Ticket.py File MoinMoin/items/_tests/test_Ticket.py (right): https://codereview.appspot.com/122940043/diff/20001/MoinMoin/items/_tests/test_Ticket.py#newcode12 MoinMoin/items/_tests/test_Ticket.py:12: class TestTicket(): On 2014/08/06 12:49:38, Thomas.J.Waldmann wrote: > this ...
9 years, 8 months ago (2014-08-06 13:04:54 UTC) #2
Thomas.J.Waldmann
> > do our other tests also use old-style classes? > > Yeah test_Blog.py also ...
9 years, 8 months ago (2014-08-09 17:22:58 UTC) #3
Thomas.J.Waldmann
> NO. It does not. No reply? When you say that test_Blog.py uses old-style classes ...
9 years, 8 months ago (2014-08-17 14:52:25 UTC) #4
sksaurabhkathpalia
9 years, 8 months ago (2014-08-17 15:56:51 UTC) #5
On 2014/08/17 14:52:25, Thomas.J.Waldmann wrote:
> > NO. It does not.
> 
> No reply?
> 
> When you say that test_Blog.py uses old-style classes and I tell you that this
> is obviously not the case (see line 24 there), why is there no reaction from
> you?

sorry Thomas, Actually I was busy with some other stuff for the project so I
didn't get a time to think on it. I will try to come to it after solving issues
mentioned in the feedback.
sorry for the inconvenience.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b