Ticket #4347 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Screenshot-taking unit test

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.0
Component: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description (last modified by Janik Zikovsky) (diff)

Make a test or script that pops up a GUI (for me, the SliceViewer), takes a screenshot, and saves it to a report.

This will enable comparing the appearance of a GUI on several platforms, and make sure that gui looks right.

The test could also do a comparison with the previous screenshot and fail if there was a change?

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Owner set to Janik Zikovsky
  • Status changed from new to accepted
  • Component changed from Mantid to Infrastructure

comment:2 Changed 9 years ago by Janik Zikovsky

Refs #4347: Start of a system test that takes GUI screenshots, currently disabled.

The script will take a screenshot and upload it to the wiki site for easy review later. Added eclipse project files for the SystemTests project. Added a copy of the mwclient (media wiki python module).

Changeset: 042054490be9eab166ab02f01723582ea9e60e49

comment:3 Changed 9 years ago by Janik Zikovsky

Refs #4347: Fix temporarily disabled system test

Changeset: cf19d31f631f2b023d7a5f68f4d5b46b3c4034d6

comment:4 Changed 9 years ago by Janik Zikovsky

Refs #4347: Framework for a system test that takes a screenshot and posts to wiki

Test will be skipped unless a password is provided

Changeset: b16663c9695ce094ea4c770223307a9f19e63abb

comment:5 Changed 9 years ago by Janik Zikovsky

Refs #4347: minor change to appearance

Changeset: 57d483cfc6ebb2f7056461661346c41e0e4edc8f

comment:6 Changed 9 years ago by Janik Zikovsky

Refs #4347 modified code to generate a html report

instead of going to wiki site. Should be more robust, and less messing with auth etc.

Changeset: 05825ba67bb5a870ee4e27e1ef3b108c0b91896e

comment:7 Changed 9 years ago by Janik Zikovsky

Refs #4347: disable system test. Will use different method

Changeset: de1ac99a86cdee5b907b876c89b91892b67e81b7

comment:8 Changed 9 years ago by Janik Zikovsky

Refs #4347 mantidplot screenshots

Changeset: 34d779b444b481be331e56d56f1000ada8c0f5c9

comment:9 Changed 9 years ago by Janik Zikovsky

Refs #4347: MantidPlot screenshot taking and html report generation

Changeset: 3c4d01d62a067301cd3f02ea76609e85ff9513e1

comment:10 Changed 9 years ago by Janik Zikovsky

Refs #4347: configure screenshots output

Changeset: 05f0eee93cf6772f877bef06b343b1482672e9d2

comment:11 Changed 9 years ago by Janik Zikovsky

  • Description modified (diff)
  • Summary changed from Screenshot-taking system test to Screenshot-taking unit test

comment:12 Changed 9 years ago by Janik Zikovsky

Refs #4347: Screenshots look better by calling processEvents()

which allows some of the resizing events to go through properly

Changeset: 72c50145c7e87d7d3cfe6b71a8cd6e366b74b1fb

comment:13 Changed 9 years ago by Janik Zikovsky

Refs #4347 screenshot tests uses Qtest if available

Changeset: 00240f08fe398a38dafe4763c98779d60e40be1f

comment:14 Changed 9 years ago by Janik Zikovsky

Refs #4347: Avoid importing Qt in Python

and add a test helper to the test build instead of a separate target

Changeset: b0b0827dafd5223fe1161aa7e42ca1b4ac7f2cd5

comment:15 Changed 9 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to fixed

OK the screenshot report framework is working well on all platforms and is used by two MantidPlot unit tests to generate a report.

There is not currently a comparison to check for regression, so it requires human eyes to confirm that the GUI looks good. The HTML report makes this quicker, at least.

comment:16 Changed 9 years ago by Mathieu Doucet

Re #4347 Remove eclipse files from code

Changeset: e627926313bb118d4ad5f4b10b313537501f7c28

comment:17 Changed 9 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:18 Changed 9 years ago by Andrei Savici

  • Status changed from verifying to closed

Works as expected

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5194

Note: See TracTickets for help on using tickets.