fix validation errors by flushing the upload

Summary: this test runs multiple format and when it got to the second format, it was still trying to upload the first format and freeing its resources would fail because it was in use by current frame buffer

Reviewed By: EricGriffith

Differential Revision: D62181019

fbshipit-source-id: 5a4185f38f9a890ce62896620f4b69b01a04d15d

src/igl/tests/util/TextureFormatTestBase.cpp

@@ -29,6 +29,7 @@ void TextureFormatTestBase::SetUp() {
                                            OFFSCREEN_TEX_WIDTH,
                                            OFFSCREEN_TEX_HEIGHT,
                                            TextureDesc::TextureUsageBits::Sampled);
+  texDesc.debugName = "TextureFormatTestBase rgba unorm8 sampled";
 
   Result ret;
   sampledTexture_ = iglDev_->createTexture(texDesc, &ret);
@@ -349,6 +350,11 @@ void TextureFormatTestBase::testUpload(std::shared_ptr<ITexture> texture) {
   const auto range = texture->getFullRange();
   const Result result = texture->upload(range, data.data());
   ASSERT_TRUE(result.isOk()) << texture->getProperties().name;
+  // flush upload
+  Result ret;
+  auto cmdBuf = cmdQueue_->createCommandBuffer(CommandBufferDesc{}, &ret);
+  cmdQueue_->submit(*cmdBuf);
+  cmdBuf->waitUntilCompleted();
 }
 
 // Attempts to render into texture.