secondlife/viewer#1006: Update test plan

1 files changed, 6 insertions, 2 deletions

diff --git a/doc/testplans/pbr_terrain_composition.md b/doc/testplans/pbr_terrain_composition.md
index 05431ab4bf..0e721c064a 100644
--- a/doc/testplans/pbr_terrain_composition.md
+++ b/doc/testplans/pbr_terrain_composition.md
@@ -37,7 +37,7 @@ When toggling the "PBR Metallic Roughness" checkbox to the opposite value, which
 
 - The default textures/materials
     - For textures, this is the default terrain texture set
-    - For materials, this is blank/null, but this is subject to change
+    - For materials, this is all blank materials, but this is subject to change
 - The previously applied texture/material set
     - History is available on a best-effort basis only. In particular, the history does not persist on viewer restart.
 
@@ -47,11 +47,15 @@ When toggling back the "PBR Metallic Roughness" checkbox to the original value,
 
 A user with appropriate permissions can change and save the textures or materials to the terrain. If the "PBR Metallic Roughness" checkbox is checked, the user applies materials, otherwise the user applies textures.
 
+The user should not be allowed to set the texture or material swatches to null.
+
 Saving may fail for the following reasons:
 
-- A terrain or material texture is invalid or null
+- A terrain or material texture is invalid
 - A terrain texture is greater than the max texture upload resolution
 
+If saving the terrain fails for any reason, the terrain should not be updated.
+
 Unlike a viewer without PBR terrain support, the new viewer will no longer treat textures with alpha channels as invalid.
 
 ## Graphics Features