Skip to content

Conversation

@TMenting
Copy link
Contributor

No description provided.

@ferpasri
Copy link
Member

ferpasri commented Oct 1, 2021

System Under Test: https://github.com/tonis-lusmagi/QT_Notepad

  1. Master and this Qt solution branch : Seems that both branches are wrongly detecting some menu items widgets
    (Maybe this should be a different issue)

master_bad_recognition_2

  1. Master branch derives actions in some Qt widgets of the main windows but is not deriving an action in the OK button.
    Also master is correctly ignoring the action derivation of blocked widgets from the posterior panel

master_not_qt

  1. This Qt branch derives actions in the main window and also in the OK button.
    But is also deriving actions in the widgets of the posterior panel (that should be blocked)

tycho_not_blocked

@ferpasri
Copy link
Member

ferpasri commented Oct 1, 2021

@TMenting What Qt SUT did you use?
So we can test multiple Qt SUTs

@TMenting
Copy link
Contributor Author

TMenting commented Oct 1, 2021

@ferpasri That's strange.. I've tested it against my own Qt application.
I just checked it in Spy mode, and it looked fine. This PR mainly enables TESTAR to detect actions. without this fix, TESTAR will not identify available actions because everything will me marked as blocked.

@ferpasri
Copy link
Member

ferpasri commented Oct 1, 2021

Thanks, with your Qt app the difference of action derivation is clear (left master, right this Qt branch)
master_vs_tycho_qt

Think I need to test deep with more Qt examples

@TMenting
Copy link
Contributor Author

TMenting commented Apr 7, 2022

Note: This PR is embedded in PR 299.

@ferpasri
Copy link
Member

This feature is already included in the #299
We will work on integrating the complete visual validation feature, and we close this PR.

@ferpasri ferpasri closed this Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants