search

gemini-testing / gemini-gui

GUI for gemini
MIT License
69 stars 18 forks source link

feat: use last successful screenshot from retries if reuse.result failed #107

Closed miripiruni closed 6 years ago

miripiruni commented 6 years ago

See: FEI-9022

Flackus commented 6 years ago

Нужно еще портировать в html-reporter. Верно, Жека? @sipayRT

j0tunn commented 6 years ago

@Flackus там по идее неактуально, т.к. в новом gui видны все ретраи: выбираешь любой и нажимаешь Accept

miripiruni commented 6 years ago

Обсудили с @sipayRT про NoRefImageError

Данные в этом случае будут такими:

{
              "name": "ie11",
              "result": {
                "suiteUrl": "https://…",
                "name": "ie11",
                "metaInfo": {
                  "url": "…",
                  "file": "gemini/test-suites/desktop/snp.gemini.js",
                  "sessionId": "67e1a7b0611647b85f2f2a6e3c1c432691233281-0339-4049-89e8-97e4dc5daf34"
                },
                "status": "error",
                "actualPath": "images/snp/fresh/plain/ie11~current_0.png",
                "image": true,
                "reason": {
                  "message": "Can not find reference image at …/ie11.png.\nRun `gemini update` command to capture all reference images.",
                  "stack": "NoRefImageError: Can not find reference image at  .bla-bla-bla"
                }
              },
              "retries": []
            }

Так как сейчас нет возможности по человечески определить тип ошибки, то предлагаю оставить как есть.

miripiruni commented 6 years ago
sipayRT commented 6 years ago

Нужно еще портировать в html-reporter.

сорян, пропустил сообщение. Да, Тоха все правильно ответил - портировать не нужно