test: fix screenshot API impl

When redoing the website we deleted the libvirtLogo.png file
not remembering that the test driver screenshot API impl
relied on it.

Rather than having the test driver use the logo as a side
effect, give it its own dedicated image to use. This is
installed in /usr/share/libvirt/test-screenshot.png and
is taken from a NeXT Cube running WorldWideWeb[1]. The
very first web browser in existance, running on the
hardware it was originally written on.

[1] https://en.wikipedia.org/wiki/WorldWideWeb

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
This commit is contained in:
Daniel P. Berrange 2016-12-06 12:45:18 +00:00
parent 4ec981d9c2
commit 0be9cea199
5 changed files with 13 additions and 1 deletions

View File

@ -1842,6 +1842,8 @@ exit 0
%{_datadir}/libvirt/cpu_map.xml %{_datadir}/libvirt/cpu_map.xml
%{_datadir}/libvirt/test-screenshot.png
%config(noreplace) %{_sysconfdir}/sasl2/libvirt.conf %config(noreplace) %{_sysconfdir}/sasl2/libvirt.conf
%files admin %files admin

View File

@ -232,6 +232,8 @@ rm -rf $RPM_BUILD_ROOT%{mingw64_libexecdir}/libvirt-guests.sh
%{mingw32_datadir}/libvirt/cpu_map.xml %{mingw32_datadir}/libvirt/cpu_map.xml
%{mingw32_datadir}/libvirt/test-screenshot.png
%{mingw32_datadir}/locale/*/LC_MESSAGES/libvirt.mo %{mingw32_datadir}/locale/*/LC_MESSAGES/libvirt.mo
%dir %{mingw32_includedir}/libvirt %dir %{mingw32_includedir}/libvirt
@ -314,6 +316,8 @@ rm -rf $RPM_BUILD_ROOT%{mingw64_libexecdir}/libvirt-guests.sh
%{mingw64_datadir}/libvirt/cpu_map.xml %{mingw64_datadir}/libvirt/cpu_map.xml
%{mingw64_datadir}/libvirt/test-screenshot.png
%{mingw64_datadir}/locale/*/LC_MESSAGES/libvirt.mo %{mingw64_datadir}/locale/*/LC_MESSAGES/libvirt.mo
%dir %{mingw64_includedir}/libvirt %dir %{mingw64_includedir}/libvirt

View File

@ -692,6 +692,7 @@ check-local: check-protocol check-symfile check-symsorting \
# Mock driver, covering domains, storage, networks, etc # Mock driver, covering domains, storage, networks, etc
TEST_DRIVER_SOURCES = \ TEST_DRIVER_SOURCES = \
test/test_driver.c test/test_driver.h test/test_driver.c test/test_driver.h
TEST_DRIVER_ASSETS = test/test-screenshot.png
# Now the Hypervisor specific drivers # Now the Hypervisor specific drivers
XEN_DRIVER_SOURCES = \ XEN_DRIVER_SOURCES = \
@ -1195,6 +1196,10 @@ GENERATED_SYM_FILES = \
$(NULL) $(NULL)
if WITH_TEST if WITH_TEST
driver_test_asset_DATA = $(TEST_DRIVER_ASSETS)
driver_test_assetdir = $(pkgdatadir)
noinst_LTLIBRARIES += libvirt_driver_test.la noinst_LTLIBRARIES += libvirt_driver_test.la
libvirt_la_BUILT_LIBADD += libvirt_driver_test.la libvirt_la_BUILT_LIBADD += libvirt_driver_test.la
libvirt_driver_test_la_CFLAGS = \ libvirt_driver_test_la_CFLAGS = \
@ -1887,6 +1892,7 @@ access/viraccessapichecklxc.c: $(srcdir)/rpc/gendispatch.pl \
# Add all conditional sources just in case... # Add all conditional sources just in case...
EXTRA_DIST += \ EXTRA_DIST += \
$(TEST_DRIVER_SOURCES) \ $(TEST_DRIVER_SOURCES) \
$(TEST_DRIVER_ASSETS) \
$(REMOTE_DRIVER_SOURCES) \ $(REMOTE_DRIVER_SOURCES) \
$(XEN_DRIVER_SOURCES) \ $(XEN_DRIVER_SOURCES) \
$(QEMU_DRIVER_SOURCES) \ $(QEMU_DRIVER_SOURCES) \

Binary file not shown.

After

Width:  |  Height:  |  Size: 33 KiB

View File

@ -5923,7 +5923,7 @@ testDomainScreenshot(virDomainPtr dom ATTRIBUTE_UNUSED,
if (VIR_STRDUP(ret, "image/png") < 0) if (VIR_STRDUP(ret, "image/png") < 0)
return NULL; return NULL;
if (virFDStreamOpenFile(st, PKGDATADIR "/libvirtLogo.png", 0, 0, O_RDONLY) < 0) if (virFDStreamOpenFile(st, PKGDATADIR "/test-screenshot.png", 0, 0, O_RDONLY) < 0)
VIR_FREE(ret); VIR_FREE(ret);
return ret; return ret;