Port ReadMetadataTestCase to 0.2

Bug #517298 reported by Mark Lee
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pyexiv2
Fix Released
Medium
Mark Lee

Bug Description

The ReadMetadataTestCase class should be ported to the pyexiv2 0.2 API. I need this for an enhancement to the bindings that I wrote.

Related branches

Revision history for this message
Olivier Tilloy (osomon) wrote :

Thanks for your work on this Mark. Out of curiosity, what bindings are those?
Note that for the test case to be complete I'll need to add tests for XMP metadata.

Changed in pyexiv2:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Mark Lee (malept) wrote :

I guess I should have worded that better. I meant that I enhanced *your* bindings :) I will file a bug on that shortly.

With regards to the test case, I'd be willing to write the tests for the XMP code as well. I just need to find some images (that are license-compatible with your project :) ) which contain XMP metadata.

Revision history for this message
Olivier Tilloy (osomon) wrote :

You can find sample images in exiv2's test data directory (http://dev.exiv2.org/repositories/browse/exiv2/trunk/test/data). Some of them embed XMP metadata.

Revision history for this message
Mark Lee (malept) wrote :

Ah, thanks. I will try to work on that tomorrow.

Revision history for this message
Mark Lee (malept) wrote :

The XMP read metadata test is in the branch now. Shall I file a merge request?

Revision history for this message
Olivier Tilloy (osomon) wrote :

Please do, I'll happily review it!

Revision history for this message
Olivier Tilloy (osomon) wrote :

Merged in revision 277.

Changed in pyexiv2:
assignee: nobody → Mark Lee (malept)
milestone: none → 0.2
status: Confirmed → Fix Committed
Olivier Tilloy (osomon)
Changed in pyexiv2:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.