search

isamu / graphai

0 stars 0 forks source link

agentのValidation #62

Open isamu opened 1 month ago

isamu commented 1 month ago

assert(!!namedInputs, "pushAgent: namedInputs is UNDEFINED!"); と、assertで実装している部分、namedInputsがdefaultで{}になるので、必ずtrueになる!!{} はtrueなので。 namedInputsの仕様が変わったときに、修正漏れっぽい。 agentはsamplesで正常系のテストはあってもエラー系のテストがないのも、これが漏れた理由。

isamu commented 1 month ago

error message含め、assertを全てに実装していく。

isamu commented 1 month ago

"pushAgent: namedInputs is UNDEFINED! Set inputs: { array: :arrayNodeId }" assertのエラーは、エラーだけでなく、どうすべきかも、メッセージに含めると開発者に優しいですね。今後はできる限り、この方針でいきましょう。