react-chatbotify icon indicating copy to clipboard operation
react-chatbotify copied to clipboard

[Task] Add integration test cases for sensitive text and markup language

Open tjtanjin opened this issue 1 year ago • 4 comments

Task Description: Currently, we run a suite of integration tests using cypress which can be found here.

However, the test cases do not cover the sensitive input feature.

Deliverable(s): New integration test cases that cover the testing of sensitive input.

Additional Context: A possible approach is to add a new block to the existing flow in App.tsx that has isSensitive set to true. Then, check that user input in textarea is masked and when message is sent, chat bubble should also be masked.

Similar approach can be referenced here: https://github.com/tjtanjin/react-chatbotify/issues/29

tjtanjin avatar May 11 '24 07:05 tjtanjin

Hi @tjtanjin, I would like to work on it, I am able to understand sensitive text one. Not able to get what markup language mean here. can you provide some reference ?

arulk8 avatar Oct 05 '24 16:10 arulk8

Hi @tjtanjin, I would like to work on it, I am able to understand sensitive text one. Not able to get what markup language mean here. can you provide some reference ?

Hey @arulk8! Go ahead and take on this task for sensitive text input 😊

For the markup language, you can reference this example. Basically, the chatbot has markup language disabled by default so tags such as <b>hello</b> will be rendered as plaintext string. If enabled, it will show as hello instead.

tjtanjin avatar Oct 05 '24 16:10 tjtanjin

Hi @tjtanjin, I would like to work on it, I am able to understand sensitive text one. Not able to get what markup language mean here. can you provide some reference ?

Hey @arulk8! Go ahead and take on this task for sensitive text input 😊

For the markup language, you can reference this example. Basically, the chatbot has markup language disabled by default so tags such as <b>hello</b> will be rendered as plaintext string. If enabled, it will show as hello instead.

Got it!

arulk8 avatar Oct 06 '24 01:10 arulk8

Hi @tjtanjin, I have raised PR for the same https://github.com/tjtanjin/react-chatbotify/pull/143 Please review.

arulk8 avatar Oct 06 '24 11:10 arulk8

Completed with #143

tjtanjin avatar Oct 18 '24 17:10 tjtanjin