Try to use different sessions in Examples.SessionTest

[pojiro]

@takasehideki

スカウティングディレイ 30 msec の異なるセッションを使うお試しテストを用意したので、 Mac で試してみていただけますか？

単発実行

mix test test/zenohex/examples/session_test.exs

複数回実行

for i in {1..100} ; do mix test test/zenohex/examples/session_test.exs ; done

[takasehideki]

ありがとうございます！
100回まわしまくりましたが，全件とも問題なくパスしています！

[takasehideki]

これじゃなくて良いのですが，気になったのでメモとして残させてください．
mix test の都度で recompile が走っているように思えるのですが，Rustler的に仕方ない挙動でしょうか？

Compiling 28 files (.ex)
Compiling crate zenohex_nif in release mode (native/zenohex_nif)
    Finished release [optimized] target(s) in 0.11s
Generated zenohex app

[pojiro]

mix test の都度で recompile が走っているように思えるのですが，Rustler的に仕方ない挙動でしょうか？

現状、mix test タスクに alias を作っていて、その中で mix compile --force させているので必ずコンパイルが走るようになっています。
改良できるか考えてみます！

[takasehideki]

あっ,,, 調子に乗って1000回でやってみたら，277回目の試行がコケてました,,,
確率低すぎだし想定内ではありますが，とりあえず共有しておきます 🙏

=================================================================
HEY, ZENOHEX DEVELOPER. TO REDUCE TESTING TIME,
WE COMPILE WITH API_OPEN_SESSION_DELAY=0
=================================================================
Compiling 28 files (.ex)
Compiling crate zenohex_nif in release mode (native/zenohex_nif)
    Finished release [optimized] target(s) in 0.11s
Generated zenohex app
.

  1) test delete/1 with subscriber (Zenohex.Examples.SessionTest)
     test/zenohex/examples/session_test.exs:39
     Assertion failed, no matching message after 100ms
     The process mailbox is empty.
     code: assert_receive %Sample{key_expr: "key/expression/delete", kind: :delete}
     stacktrace:
       test/zenohex/examples/session_test.exs:52: (test)

.

  2) test get/2 with queryable (Zenohex.Examples.SessionTest)
     test/zenohex/examples/session_test.exs:66
     Assertion failed, no matching message after 100ms
     The process mailbox is empty.
     code: assert_receive %Sample{}
     stacktrace:
       test/zenohex/examples/session_test.exs:83: (test)


Finished in 0.7 seconds (0.00s async, 0.7s sync)
4 tests, 2 failures

[takasehideki]

totally LGTM!!

[pojiro]

@takasehideki 同じセッションと異なるセッションの双方のテストができるように変更しました。

• 同じセッション：何も指定せずにを走らせる（ mix test ）とこちら
• 異なるセッション： USE_DIFFERENT_SESSION ありで走らせる（ USE_DIFFERENT_SESSION="" mix test ）とこちら

異なるセッションは確率的にコケることがままあることが分かったので、通常の mix test では同じセッションを使うようにしました。
そうはいっても、異なるセッションのテストは必要と思う（Zenoh のバグにも気付ける）ので、それは人間が回し忘れたとしても CI では必ず回るように ci.yaml を更新しました。

[takasehideki]

@pojiro ただの好みですみません．環境変数は定義 or 未定義よりも値で見るべきかなと思っています．export USE_DIFFERNET_SESSION=1 のときに test-with-two-session が動く感じです（ test-with-another-sessions かもですが;;
export USE_DIFFERNET_SESSION= でクリアしたと思っても環境変数としては定義されている状態になっていると認識しています．未定義に戻すには unset USE_DIFFERNET_SESSION すればいいだけですが,,,

[pojiro]

@takasehideki いえ、好み大事です！指摘事項を修正した PR を別で作ります！