872138 - mozfile.NamedTemporaryFile API updates

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Jeff Hammel]

From https://bugzilla.mozilla.org/show_bug.cgi?id=790765#c35 and
https://bugzilla.mozilla.org/show_bug.cgi?id=790765#c36 :

New file APIs are needed for mozfile.NamedTemporaryFile:

  - should be (line) iterable (i.e. a generator with __iter__ and next
    in the ususal way

  - should take delete=False

Relevent tracebacks:

======================================================================
ERROR: test_schema_version (bug785146.PermissionsTest)
----------------------------------------------------------------------
Traceback (most recent call last): File
"/home/jhammel/mozilla/src/mozilla-central/testing/mozbase/mozprofile/tests/bug785146.py",
line 41, in test_schema_version perms.write_db(self.locations_file)
File
"/home/jhammel/mozilla/src/mozilla-central/testing/mozbase/mozprofile/mozprofile/permissions.py",
line 247, in write_db for location in locations: TypeError:
'NamedTemporaryFile' object is not iterable
----------------------------------------------------------------------

======================================================================
ERROR: test_prefs_write (test_preferences.PreferencesTest) test that
the Preferences.write() method correctly serializes preferences
----------------------------------------------------------------------
Traceback (most recent call last): File
"/home/jhammel/mozilla/src/mozilla-central/testing/mozbase/mozprofile/tests/test_preferences.py",
line 237, in test_prefs_write with
mozfile.NamedTemporaryFile(suffix='.js', delete=False) as f:
TypeError: __init__() got an unexpected keyword argument 'delete'

Comment 1

[Jeff Hammel]

Following this (and ensuring m-c is okay) this should be version bumped and mirrored to m-c for bug 790765

Comment 2

[Jeff Hammel]

Found https://bugzilla.mozilla.org/show_bug.cgi?id=872230 while working.  Luckily, I don't think we care about it for this bug.

Comment 3

[Jeff Hammel]

Attachment: fix these things

a version bump and mirroring (possibly to be done with other code) is still needed following checkin

Comment 4

[Jonathan Griffin (:jgriffin)]

Comment on attachment 749497 [details] [diff] [review]
fix these things

Review of attachment 749497 [details] [diff] [review]:
-----------------------------------------------------------------

cool, lgtm!

Comment 5

[Jeff Hammel]

pushed https://github.com/mozilla/mozbase/commit/a1a7296371692e1ba765a8c4037e97e6807a47a1

Comment 6

[Jeff Hammel]

Will bump mozfile -> 0.7 for this

Comment 7

[Jeff Hammel]

bumped, tagged, ->pypi'ed: https://github.com/mozilla/mozbase/commit/b90a02a2c6a26aed9182f32bddad75bbb187e05c