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

Issue 10900043: suppress junit3 deprecated warnings (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by felix8a
Modified:
12 years, 8 months ago
Reviewers:
kpreid2
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

currently there are about a dozen warning messages about use of obsolete junit3 junit.framework.Assert (rather than junit4 org.junit.Assert) this suppresses the warnings until we do the full conversion to junit4.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M tests/com/google/caja/util/MoreAsserts.java View 1 chunk +2 lines, -0 lines 0 comments Download
M tests/com/google/caja/util/RhinoTestBed.java View 1 chunk +2 lines, -0 lines 1 comment Download

Messages

Total messages: 3
felix8a
12 years, 8 months ago (2013-07-03 01:37:01 UTC) #1
kpreid2
LGTM++ https://codereview.appspot.com/10900043/diff/1/tests/com/google/caja/util/RhinoTestBed.java File tests/com/google/caja/util/RhinoTestBed.java (right): https://codereview.appspot.com/10900043/diff/1/tests/com/google/caja/util/RhinoTestBed.java#newcode60 tests/com/google/caja/util/RhinoTestBed.java:60: // TODO(felix8a): remove SuppressWarnings after full conversion to ...
12 years, 8 months ago (2013-07-03 16:39:59 UTC) #2
felix8a
12 years, 8 months ago (2013-07-03 19:30:42 UTC) #3
@r5472
Sign in to reply to this message.

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