zcash_client_sqlite: Disallow selection of dust notes.

At present, we don't take builder padding rules into account in the
determination of whether or not we can include spends of dust notes in
grace actions. It's too complex to attempt a complete fix for Zashi 1.0,
but we can have a tolerable workaround for launch by just never
selecting dust-valued notes.

This is a temporary solution; opened #1316 to track the resolution.

Author: nuttycom

zcash_client_sqlite/src/wallet/common.rs

@@ -158,6 +158,7 @@ where
                  INNER JOIN transactions
                     ON transactions.id_tx = {table_prefix}_received_notes.tx
                  WHERE {table_prefix}_received_notes.account_id = :account
+                 AND value >= 5000 -- FIXME #1016, allow selection of a dust inputs
                  AND accounts.ufvk IS NOT NULL
                  AND recipient_key_scope IS NOT NULL
                  AND nf IS NOT NULL

zcash_client_sqlite/src/wallet/orchard.rs

@@ -596,6 +596,7 @@ pub(crate) mod tests {
     }
 
     #[test]
+    #[ignore] // FIXME: #1316 This requires support for dust outputs.
     fn zip317_spend() {
         testing::pool::zip317_spend::<OrchardPoolTester>()
     }

zcash_client_sqlite/src/wallet/sapling.rs

@@ -602,6 +602,7 @@ pub(crate) mod tests {
     }
 
     #[test]
+    #[ignore] // FIXME: #1316 This requires support for dust outputs.
     fn zip317_spend() {
         testing::pool::zip317_spend::<SaplingPoolTester>()
     }
@@ -612,9 +613,7 @@ pub(crate) mod tests {
         testing::pool::shield_transparent::<SaplingPoolTester>()
     }
 
-    // FIXME: This requires fixes to the test framework.
     #[test]
-    #[cfg(feature = "orchard")]
     fn birthday_in_anchor_shard() {
         testing::pool::birthday_in_anchor_shard::<SaplingPoolTester>()
     }