Doing Simple Smoke Test - Need Help - (Moved)

[MuazIbrahim]

Hi Guys,
i'm new in AutoIt and really would appreciate your help
im doing an automated smoke test, the test is simple :

running exe - check if window active - take screenshot and write log if test passed or failed

here is my script 

Quote

 #include <File.au3>
 #include <ScreenCapture.au3>
Func _WinWaitActivate($title,$text,$timeout=0)
    WinWait($title,$text,$timeout)
    If Not WinActive($title,$text) Then WinActivate($title,$text)
    WinWaitActive($title,$text,$timeout)
 EndFunc

 Local $sMyTestLog = "C:\Autoit Logs\" &"TestOpenChrome" & @Year & "-" & @MON & "-" & @MDAY & ".log"

Func TestOpenChrome()

Run('C:\Program Files (x86)\Google\Chrome\Application\chrome.exe')

if WinExists("Neuer Tab - Google Chrome","") Then

_WinWaitActivate("Neuer Tab - Google Chrome","")
WinSetState("Neuer Tab - Google Chrome", "", @SW_MAXIMIZE)
 _ScreenCapture_CaptureWnd($sMyTestLog & "_Image.jpg", "Neuer Tab - Google Chrome")
MouseClick("left",1514,17)
_FileWriteLog($sMyTestLog,"Test 1 - Passed - ")

Else

Sleep(2000)
   _FileWriteLog($sMyTestLog,"TestOpenChrome - FAILED " & "Failure reason ?")
_ScreenCapture_Capture($sMyTestLog & "_Image.jpg")

EndIf

EndFunc

Call ("TestOpenChrome")

now the problem i'm facing that the Else(false) branch is always being executed even if the true condition was the case , am i missing something here?
another question, is there a better or more effieicent way for the verification here?
Thank you in a Advance 

[Melba23]

Moved to the appropriate forum.

Moderation Team

[Nine]

1.  When you post code, please use this tool.

2. You should not mix functions and main code in alternation, it makes the code very hard to follow.   Put your main code together at the beginning of the script (best practices).

3. As for your problem, it seems that you should put a sleep after run to let time for the program to load. 

Edited April 26, 2021 by Nine

[MuazIbrahim]

1 hour ago, Nine said:

1.  When you post code, please use this tool.

2. You should not mix functions and main code in alternation, it makes the code very hard to follow.   Put your main code together at the beginning of the script (best practices).

3. As for your problem, it seems that you should put a sleep after run to let time for the program to load. 

yes the sleep solve the problem
Thank you and i will take care of the script structure as per your notes