Support for screen-recording media type about osxphotos HOT 6 OPEN

@oPromessa I'm impressed with how thorough your branch is -- nice work! A couple pointers:

The "screen_shot" key should be added inasdict() for photo PhotoInfo and the iPhoto PhotoInfo and it must be added in the if not shallow: block. This prevents new keys from messing up the comparison in osxphotos export to the stored JSON data for a photo. Only keys in the shallow=True dictionary are compared when exporting.

Additionally, in the iPhoto test, you'll need to update the test data at tests/iphoto_test_data.json to add the new key so the tests works.

from osxphotos.

Ran build.sh... some warnings related with latex and pdf (guess I don't have the tools)

The docs building needs work. Don't worry about this as I don't build the docs until I do the release. For individual PRs, just update the appropriate files but don't build the docs as they will get re-built as part of my release process.

from osxphotos.

On docsrc/README.md is there a typo: cd docs should be cd docsrc ?

Yes, you're right. I'll fix it.

from osxphotos.

Had to set property screen-recording to None for <= _PHOTOS_4_VERSION).

from osxphotos.

• Yeah! Basically RTFM 😉 Test Data -- BTW I've edited it a bit.
  • I think I mostly got it. But have it but still question what to do in these code lines (now commented. 'cause if I uncomment, tests/test_cli.py fail 😢 on assert with where 1 = <Result KeyError('screen_recording')>.exit_code
  • print_photo_info.py:L56
  • print_photo_info.py:L102
  • photoinfo.py:L2173
• All tests pass now! YAY 💯
• Ran build.sh... some warnings related with latex and pdf (guess I don't have the tools). stderr: build.txt
• On docsrc/README.md is there a typo: cd docs should be cd docsrc ? Did not want to change it, in case I was wrong. Also build.sh generates the docs. PS: Did not install MacTeX

from osxphotos.

I think I mostly got it. But have it but still question what to do in these code lines (now commented. 'cause if I uncomment, tests/test_cli.py fail 😢 on assert with where 1 = <Result KeyError('screen_recording')>.exit_code

The test_cli.py is a monster....on my to do list to break this up into smaller chunks as it has become very unwieldy.

Basically, for the tests that are failing, you need to track down where they're getting the "expected" data from. For example, in test_query_uuid(), it's reading CLI_OUTPUT_QUERY_UUID which is defined elsewhere in the file. This is a JSON string that will be reconstituted and compared to the JSON output of the query command. This JSON string will need to be modified to contain the screen_recording key.

There may be other places that need to be modified.

from osxphotos.