From 63f3412b030a194feb997f4c6b5962ecec677f3e Mon Sep 17 00:00:00 2001
From: mocabe <hisa.mocabe@gmail.com>
Date: Tue, 26 May 2020 17:17:05 +0900
Subject: [PATCH] [test] Add missing test cases for ResultValueRValue.cpp

---
 tests/ResultValueRValue/ResultValueRValue.cpp | 26 ++++++++++++-------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/tests/ResultValueRValue/ResultValueRValue.cpp b/tests/ResultValueRValue/ResultValueRValue.cpp
index d3e89ea..9d7ecb6 100644
--- a/tests/ResultValueRValue/ResultValueRValue.cpp
+++ b/tests/ResultValueRValue/ResultValueRValue.cpp
@@ -20,19 +20,25 @@
 
 VULKAN_HPP_DEFAULT_DISPATCH_LOADER_DYNAMIC_STORAGE
 
-template <typename T>
-struct Wrapper
-{
-  Wrapper( T && raw ) : raw{ raw } {}
+#if defined(VULKAN_DISABLE_IMPLICIT_RESULT_VALUE_CAST)
+  static_assert(false, "Conversions not enabled");
+#endif
 
-  T raw;
-};
+template <class T> void as_value(T)  {}
+template <class T> void as_cref(const T&) {}
+template <class T> void as_rvref(T&&)  {}
 
 void test()
 {
   auto device = vk::Device{};
-  // We should be able to move created vk objects, regardless of the
-  // result type they were returned in.
-  Wrapper<vk::Image>{ device.createImage( {} ) };
-  Wrapper<vk::Pipeline>{ device.createGraphicsPipeline( {}, {} ) };
+
+  // just example: non-result values (vk::Image)
+  as_value<vk::Image>(device.createImage({}));
+  as_cref<vk::Image>(device.createImage({}));
+  as_rvref<vk::Image>(device.createImage({}));
+
+  // vk::ResultValue<vk::Pipeline> should also work
+  as_value<vk::Pipeline>(device.createGraphicsPipeline(nullptr, {}));
+  as_cref<vk::Pipeline>(device.createGraphicsPipeline(nullptr, {}));
+  as_rvref<vk::Pipeline>(device.createGraphicsPipeline(nullptr,{}));
 }