[libcamera-devel,v2,3/8] test: camera: Remove test for bad Stream IDs

Message ID 20190405020256.22520-4-niklas.soderlund@ragnatech.se
State Superseded
Headers show
Series
  • libcamera: stream: Add basic stream usages
Related show

Commit Message

Niklas Söderlund April 5, 2019, 2:02 a.m. UTC
In preparation of reworking how a default configuration is retrieved
from a camera remove test that stream IDs must be valid as the data type
passed to Camera::streamConfiguration() will change. This change is in
preparation for an invasive change.

Signed-off-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
---
 test/camera/configuration_default.cpp | 13 -------------
 1 file changed, 13 deletions(-)

Patch

diff --git a/test/camera/configuration_default.cpp b/test/camera/configuration_default.cpp
index 53ee021d33ca39b1..856cd415f7a6aec1 100644
--- a/test/camera/configuration_default.cpp
+++ b/test/camera/configuration_default.cpp
@@ -49,19 +49,6 @@  protected:
 			return TestFail;
 		}
 
-		/*
-		 * Test that asking for configuration for an array of bad streams
-		 * returns an empty list of configurations.
-		 */
-		Stream *stream_bad = reinterpret_cast<Stream *>(0xdeadbeef);
-		std::set<Stream *> streams_bad = { stream_bad };
-		conf = camera_->streamConfiguration(streams_bad);
-		if (!conf.empty()) {
-			cout << "Failed to retrieve configuration for bad streams"
-			     << endl;
-			return TestFail;
-		}
-
 		return TestPass;
 	}
 };