Remove workarounds by data_device_bins

Previously, the test has dirty hacks that force building the device
executable in correct name. Swicth to the proper solution now that we
have it.

Bug: 183749054 (not really, but related)
Test: atest AuthFsHostTest
Change-Id: Ic0f1c8aa8c05031313f0bf2caf9bd43dc6757cfd
diff --git a/authfs/tests/Android.bp b/authfs/tests/Android.bp
index 6b3a474..ca403e8 100644
--- a/authfs/tests/Android.bp
+++ b/authfs/tests/Android.bp
@@ -14,7 +14,7 @@
         "VirtualizationTestHelper",
     ],
     test_suites: ["general-tests"],
-    target_required: ["open_then_run_module"],
+    data_device_bins: ["open_then_run"],
     data: [
         ":authfs_test_files",
         ":MicrodroidTestApp.signed",
@@ -22,16 +22,7 @@
 }
 
 rust_test {
-    // PushFilePreparer can sometimes push the directory (if named "open_then_run", which contains
-    // the actual executable in a per-architecture sub-directory) instead of the executable. This
-    // makes it harder to use because the host Java test have to detect the executable path
-    // dynamically, e.g. if it's a directory, append the device's architecture to build the actual
-    // executable path. By simply renaming the module (thus the host directory), this forces
-    // PushFilePreparer to always push the executable to the destination, so that the Java test can
-    // easily locate the executable with a constant path.
-    name: "open_then_run_module",
-    stem: "open_then_run",
-
+    name: "open_then_run",
     crate_name: "open_then_run",
     srcs: ["open_then_run.rs"],
     edition: "2018",