リーチ後再度リーチのactionが送られてきている。

[nissymori]

resolve #130 #129 #128 #114 #92

[nissymori]

reach_acceptedが届いていない。

[nissymori]

検証の結果 問題1 reach_acceptedがrespond_to_actionでmjai側に送られていない

 if action.type == :reach_accepted
    p "reach_accepted_action_in_view_respond_to_action"
    p action
    p action.to_json()
    p "action.to_jsonを表示しました。"
  end
  #puts("server -> player %d\t%s" % [self.id, action.to_json()])
  @socket.puts(action.to_json())
  line = nil
  if action.type == :reach_accepted
    p "reach_accepted_action_in_view_respond_to_action_2"
  end

"reach_accepted_action_in_view_respond_to_action" は出力されているが、 "action.to_jsonを表示しました。", "reach_accepted_action_in_view_respond_to_action_2"は出力されない。

このことから、action.to_jsonがbugっている？→単体のテストは通っている。

[nissymori]

問題2

RIICHI_TEN_CHANGEを含むobservationに対してactionを返していないのに、次のobservationが送られてきている。

[nissymori]

検証の結果 問題1 reach_acceptedがrespond_to_actionでmjai側に送られていない

 if action.type == :reach_accepted
    p "reach_accepted_action_in_view_respond_to_action"
    p action
    p action.to_json()
    p "action.to_jsonを表示しました。"
  end
  #puts("server -> player %d\t%s" % [self.id, action.to_json()])
  @socket.puts(action.to_json())
  line = nil
  if action.type == :reach_accepted
    p "reach_accepted_action_in_view_respond_to_action_2"
  end

"reach_accepted_action_in_view_respond_to_action" は出力されているが、 "action.to_jsonを表示しました。", "reach_accepted_action_in_view_respond_to_action_2"は出力されない。

このことから、action.to_jsonがbugっている？→単体のテストは通っている。

同じことがryukyokuでも起こっている。

[nissymori]

TODO

• [x] 流局　to_json test作成
• [x] リーチ承認　to_json test 作成
• [x] 流局　do_action test作成
• [x] リーチ承認　do_action test 作成

[nissymori]

原因判明 どちらのcaseもprotobufのGoogle::Protobuf::RepeatedFieldをそのまま使っていたのでbugっていた。

修正後tsumogiri-playerでは連続で試合ができるようになった。

[nissymori]

@sotetsuk 上記のようにbug修正できました。 問題なければmergeします！

[sotetsuk]

ありがとうございます！マージして下さいー@nissymori

[nissymori]

@sotetsuk レビューありがとうございます！ 了解です！