Bazaar

Code review comment for lp:~doxxx/bzr/bug939605-2.5

  1. bug939605-2.5
  2. Merge into 2.5
Revision history for this message
Martin Packman (gz) wrote :

Changes look good.

+ # Internt Explorer is always registered in the App Path
+ self.assertTrue(osutils.find_executable_on_path('iexplore') is not None)

I wonder a little if one of those people who purge IE from their system will ever run our test suite. Not worrying about that seems correct, the test looks fine (though 'is not None' is perhaps not useful).

See earlier mp for initial reviews:

<https://code.launchpad.net/~doxxx/bzr/bug939605/+merge/94488>

review: Approve

« Back to merge proposal