[Merged by Bors] - feat: Download trait for Artifact

[EstebanBorai]

Introduces download logic for Artifact resource in the Hub Util API.

Download logic is introduced through the Download trait which allow for decoupling
from the type definition.

[EstebanBorai]

...Improving to use path over passing file to get shasum

[sehz]

Nice job. LGTM

[EstebanBorai]

@digikata @sehz, looks like these tests:

• package::tests::hubutil_package_assemble
• package::tests::hubutil_package_sign

Initializes a tracing subscriber here:

fluvio/crates/fluvio-hub-util/src/package.rs

Line 620 in ad3e78f

rust_log_init();

Which conflicts with:
https://docs.rs/crate/fluvio-test-derive/0.1.1/source/src/lib.rs

        #[test]
        #test_attributes
        fn #out_fn_iden()  {

            ::fluvio_future::subscriber::init_logger(); // This subscriber

            #input

            let ft = async {
                #name().await;
            };

            
            ::fluvio_future::task::run_block_on(ft);

        }

Which is expanded by #[fluvio_future::test].

I don't see a way to avoid that subscriber::init_logger() from the macro itself, any suggestions?

[digikata]

Switch package.rs test util function to try_init?

https://docs.rs/tracing-subscriber/latest/tracing_subscriber/util/trait.SubscriberInitExt.html#method.try_init

[sehz]

Removing them from the pkg test for now. Test code should not initialize the logger explicitly. We can add a non-Sync version in the fluvio future later

[EstebanBorai]

@sehz @digikata now tests are passing! I think we are good to move into the next step!

[EstebanBorai]

bors r+

[bors]

Build failed:

• Done

[EstebanBorai]

bors retry

[bors]

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

• Done