Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d53b4f38 authored by Martin Geisler's avatar Martin Geisler Committed by Automerger Merge Worker
Browse files

Revert^2 "pdl: Generate canonical tests with prettyplease" am: cf5638a6 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/2596669



Change-Id: I748b172e5c03aefe08568942a11ba438c5a069aa
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 2487eb02 cf5638a6

tools/pdl/Android.bp

+5 −16
Original line number Diff line number Diff line
@@ -249,6 +249,7 @@ rust_binary_host { 
    name: "pdl_generate_tests",

    srcs: ["src/bin/generate-canonical-tests.rs"],

    rustlibs: [

        "libprettyplease",

        "libproc_macro2",

        "libquote",

        "libserde",
@@ -258,32 +259,20 @@ rust_binary_host { 
    ],

}



genrule_defaults {

    name: "pdl_rust_generator_src_defaults",

    tools: [

        ":pdl_generate_tests",

        ":rustfmt",

    ],

}



genrule {

    name: "pdl_rust_generator_tests_le_src",

    cmd: "set -o pipefail;" +

        " $(location :pdl_generate_tests) $(in) pdl_le_backend |" +

        " $(location :rustfmt) > $(out)",

    cmd: "$(location :pdl_generate_tests) $(in) pdl_le_backend > $(out)",

    srcs: ["tests/canonical/le_test_vectors.json"],

    out: ["le_canonical.rs"],

    defaults: ["pdl_rust_generator_src_defaults"],

    tools: [":pdl_generate_tests"],

}



genrule {

    name: "pdl_rust_generator_tests_be_src",

    cmd: "set -o pipefail;" +

        " $(location :pdl_generate_tests) $(in) pdl_be_backend |" +

        " $(location :rustfmt) > $(out)",

    cmd: "$(location :pdl_generate_tests) $(in) pdl_be_backend > $(out)",

    srcs: ["tests/canonical/be_test_vectors.json"],

    out: ["be_canonical.rs"],

    defaults: ["pdl_rust_generator_src_defaults"],

    tools: [":pdl_generate_tests"],

}



rust_test_host {

tools/pdl/src/bin/generate-canonical-tests.rs

+10 −11
Original line number Diff line number Diff line
@@ -128,17 +128,16 @@ fn generate_unit_tests(input: &str, packet_names: &[&str], module_name: &str) { 
    }



    // TODO(mgeisler): make the generated code clean from warnings.

    println!("#![allow(warnings, missing_docs)]");

    println!();

    println!(

        "{}",

        &quote! {

    let code = quote! {

        #![allow(warnings, missing_docs)]



        use #module::Packet;

        use serde_json::json;



        #(#tests)*

        }

    );

    };

    let syntax_tree = syn::parse2::<syn::File>(code).expect("Could not parse {code:#?}");

    println!("{}", prettyplease::unparse(&syntax_tree));

}



fn main() {