Code review comments applied.

This commit is contained in:
Alexander Smorkalov 2012-11-12 17:47:20 +04:00
parent baef62b4d2
commit e06c5b6fd5
3 changed files with 15 additions and 29 deletions

View File

@ -28,7 +28,7 @@ public class OpenCVTestCase extends TestCase {
//change to 'true' to unblock fail on fail("Not yet implemented") //change to 'true' to unblock fail on fail("Not yet implemented")
public static final boolean passNYI = true; public static final boolean passNYI = true;
protected static boolean isEnabled = true; protected static boolean isTestCaseEnabled = true;
protected static final int matSize = 10; protected static final int matSize = 10;
protected static final double EPS = 0.001; protected static final double EPS = 0.001;

View File

@ -2,9 +2,6 @@ package org.opencv.test;
import java.io.File; import java.io.File;
import java.io.IOException; import java.io.IOException;
//import java.util.Iterator;
//import junit.framework.TestCase;
import junit.framework.Assert; import junit.framework.Assert;
import org.opencv.android.Utils; import org.opencv.android.Utils;
@ -74,17 +71,6 @@ public class OpenCVTestRunner extends InstrumentationTestRunner {
// Note: VideoCapture tests turned off by flag field in VideoCaptureTest class // Note: VideoCapture tests turned off by flag field in VideoCaptureTest class
// if(OpenCVTestCase.passNYI) {
// // turn off problematic camera tests
// Iterator<TestCase> it = androidTestRunner.getTestCases().iterator();
// while (it.hasNext()) {
// String name = it.next().toString();
// if (name.contains("VideoCaptureTest"))
// it.remove();
// }
// }
super.onStart(); super.onStart();
} }

View File

@ -19,13 +19,13 @@ public class VideoCaptureTest extends OpenCVTestCase {
super.setUp(); super.setUp();
capture = null; capture = null;
isEnabled = false; isTestCaseEnabled = false;
isSucceed = false; isSucceed = false;
isOpened = false; isOpened = false;
} }
public void testGet() { public void testGet() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -39,7 +39,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testGetSupportedPreviewSizes() { public void testGetSupportedPreviewSizes() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -54,7 +54,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testGrab() { public void testGrab() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -64,7 +64,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testGrabFromRealCamera() { public void testGrabFromRealCamera() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -78,7 +78,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testIsOpened() { public void testIsOpened() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -87,7 +87,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testIsOpenedRealCamera() { public void testIsOpenedRealCamera() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -101,7 +101,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testOpen() { public void testOpen() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -116,7 +116,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testRead() { public void testRead() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -132,7 +132,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testRelease() { public void testRelease() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -147,7 +147,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testRetrieveMat() { public void testRetrieveMat() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -164,7 +164,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testRetrieveMatInt() { public void testRetrieveMatInt() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -181,7 +181,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testSet() { public void testSet() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }
@ -205,7 +205,7 @@ public class VideoCaptureTest extends OpenCVTestCase {
} }
public void testVideoCaptureInt() { public void testVideoCaptureInt() {
if (!isEnabled) { if (!isTestCaseEnabled) {
fail("Not yet implemented"); fail("Not yet implemented");
} }