[Merged by Bors] - Move x records outputed from smdk test to --verbose #3169

[TanNgocDo]

This is to add verbose flag into smdk test : #3169

[ajhunyady]

Can you please also add a test for the non-verbose case?

run $SMDK_BIN test --text '["foo", "bar"]'

[TanNgocDo]

Can you please also add a test for the non-verbose case?

run $SMDK_BIN test --text '["foo", "bar"]'

Hi @ajhunyady , Sorry, I have not paid attention that smoke test fail. I Tried to reproduce in my local machine the smoke test(I'm using Mac M1) but it seems not work(or I'm doing a wrong way). Here is the way I tried to reproduce smoke test:

export SMDK_BIN=....
bats ./tests/cli/smdk_smoke_tests/smdk-basic.bats

All the test are failed because no config in my fluvio folder:

Error: Config error: config file "/Users/tando/.fluvio/config"
   
     Caused by:
         0: config file "/Users/tando/.fluvio/config"
         1: No such file or directory (os error 2)

I'm not sure how to create such config - I'm using Mac M1.
cc @morenol
ps: All wasm modules were created but it seems to be failed at loading:

  Found SmartModule package: repo-params-aggregate-zxoua3c
     loading module at: /private/var/folders/dw/54y7x1l51v1_wjrwz5dp5cfc0000gn/T/smdk-test.XXXXX.LXXDNS0n/target/wasm32-unknown-unknown/release-lto/repo_params_aggregate_zxoua3c.wasm
     Error: Config error: config file "/Users/tando/.fluvio/config"


```

[TanNgocDo]

Update : I think I should change the Generate and test filter-map

# Test
    run $SMDK_BIN test --verbose --text '2'
    assert_output --partial "1 records outputed"
    assert_output --partial "1" #changed from 2 -> 1 (Have tested in my local the generated wasm smart module)
    assert_success

Thanks

[ajhunyady]

@morenol please review.

[morenol]

LGTM

[sehz]

Looks good. couple of minor nits

[sehz]

LGTM

[sehz]

bors r+

[bors]

Build failed:

• Done

[sehz]

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